Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[vault-operator] add OPERATOR_LOG_LEVEL setting to values for the vault-operator chart #1490

Open
jkirkham-ratehub opened this issue Dec 8, 2021 · 2 comments
Labels
lifecycle/stale Denotes an issue or PR that has become stale and will be auto-closed.

Comments

@jkirkham-ratehub
Copy link

Is your feature request related to a problem? Please describe.
It appears that the vault-operator (as installed by the Helm chart) has hard-coded the log-level to debug. https://github.com/banzaicloud/bank-vaults/blob/1b3a8b7480c061866616a3563838d2214682479d/charts/vault-operator/templates/deployment.yaml#L50-L51
This is producing a lot of unnecessary log data for us. It would be better to make this configurable.

Describe the solution you'd like
Expose OPERATOR_LOG_LEVEL in the values.yaml of the Helm chart for vault-operator so it can be changed as needed by end users.

Describe alternatives you've considered
I haven't tried this yet, but it may be possible to use a kustomize overlay to patch the rendered Helm chart with the desired log level. Though having this as a configurable part of the Helm chart would be easier.

@SCLogo
Copy link

SCLogo commented Sep 6, 2022

any update on this?

Copy link

github-actions bot commented Jun 9, 2024

Thank you for your contribution! This issue has been automatically marked as stale because it has no recent activity in the last 60 days. It will be closed in 20 days, if no further activity occurs. If this issue is still relevant, please leave a comment to let us know, and the stale label will be automatically removed.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR that has become stale and will be auto-closed. label Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/stale Denotes an issue or PR that has become stale and will be auto-closed.
Projects
None yet
Development

No branches or pull requests

2 participants