search

cetic / helm-zabbix

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

Add secrets #52

Closed plicon closed 2 years ago

plicon commented 2 years ago

What this PR does / why we need it:

This PR allows users to configure a secret for connecting to postgresql server, instead of using clear text strings.

Configurable for both server and web.

Which issue this PR fixes

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

Special notes for your reviewer:

Checklist

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

plicon commented 2 years ago

i noticed i didn't update the version of the chart in the initial PR, so updated it to 1.2.0 which now caused a conflict ..

plicon commented 2 years ago

Conflicts resolved

aeciopires commented 2 years ago

Hello @plicon!

Thank you very much for your contribution to the project. I've been away for a few weeks. I'll look at the Pull Request carefully and make my thoughts before the merge.

aeciopires commented 2 years ago

Hi @plicon!

Good job in this Pull Request. Thanks for your contributition.

Greetings.