zammad-helm icon indicating copy to clipboard operation
zammad-helm copied to clipboard

Why duplicate login credentials in values?

Open xeruf opened this issue 2 years ago • 6 comments

In the values.yaml you have zammadConfig.postgresql.pass and postgresql.auth.password, stating explicitly that they need to be the same - why not just default one to the other so filling in one is sufficient? Same for redis, but interestingly not for minio it seems.

xeruf avatar Feb 19 '24 14:02 xeruf

Because on of these values confiures the zammad chart and the other the postgres sub chart.

In the end these are just the default values so that the zammad chart can be easily deployed.

In production, where your config likely resides in a git repo, secrets should never be a part of it. Thats the reason you can define exisiting secrets.

monotek avatar Feb 19 '24 15:02 monotek

yeah mine are also deployed via gitops, still it adds opportunity to screw up in the config - and I know many other charts where you only need to add the password once so there is apparently some way to access the values of the subchart

xeruf avatar Feb 29 '24 15:02 xeruf

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Apr 30 '24 05:04 github-actions[bot]

.

xeruf avatar Apr 30 '24 19:04 xeruf

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jun 30 '24 05:06 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Aug 31 '24 05:08 github-actions[bot]