cetic / helm-zabbix

Helm Chart For Zabbix
https://artifacthub.io/packages/helm/cetic/zabbix
Apache License 2.0
56 stars 57 forks source link

Issue 65 #67

Closed aeciopires closed 2 years ago

aeciopires commented 2 years ago

What this PR does / why we need it:

Example of config:

  extraPodSpecs:
    dnsPolicy: "None"
    dnsConfig:
      nameservers:
        - 1.2.3.4
      searches:
        - ns1.svc.cluster-domain.example
        - my.dns.search.suffix
      options:
        - name: ndots
          value: "2"
        - name: edns0
  extraInitContainers:
  - name: testing
    image: "busybox:latest"
    command: 
    - /bin/sh
    - -c
    - wget -O- https://zabbix.com
    resources: {}

Which issue this PR fixes

Checklist

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

aeciopires commented 2 years ago

Hi @maxenced!

Could you test these changes and see if it helps you?

Hi @sa-ChristianAnton! What do you think of this implementation?

david-moreau commented 2 years ago

Hi @aeciopires, Thanks for your works I just test your change and it's works great :+1:

aeciopires commented 2 years ago

Hello @david-moreau!

Thanks for your feedback. I will do merge.

aeciopires commented 2 years ago

Hello @tomsozolins!

Thanks for your contributition in PR #69.

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