fix(cronjob): add api version check

cetic / helm-zabbix

Helm Chart For Zabbix

https://artifacthub.io/packages/helm/cetic/zabbix

Apache License 2.0

56 stars 57 forks source link

fix(cronjob): add api version check #69

Closed tomsozolins closed 2 years ago

tomsozolins commented 2 years ago

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md

aeciopires commented 2 years ago

Hello @tomsozolins!

Thanks for your contributition in this PR.

I'm publishing a new version of the chart in PR #67, I found it simpler to do git cherry-pick to merge your commit here than to have to publish another version just for that.

Reference:

https://gist.github.com/rxhanson/df0f2b49f402ea181f5ed61b20f9cd8d
https://poanchen.github.io/blog/2017/11/12/How-to-create-a-GitHub-pull-request-with-a-specific-commits