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

[bitnami/vault] Parametize caBundle in mutatingwebhook #30709

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tspiri26
Copy link

@tspiri26 tspiri26 commented Dec 2, 2024

Description of the change

We have a cluster with multiple Vaults, each located in a different namespace. ArgoCD attempts to apply the caBundle with an empty string as its value. However, the cluster automatically populates the caBundle with a certificate, causing ArgoCD to fail when it tries to reapply the object with the empty string value.

Benefits

Mentioned above.

Possible drawbacks

N/A

Applicable issues

N/A

Additional information

N/A

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

@github-actions github-actions bot added vault triage Triage is needed labels Dec 2, 2024
@javsalgar javsalgar changed the title bitnami/vault Parametize caBundle in mutatingwebhook [bitnami/vault] Parametize caBundle in mutatingwebhook Dec 3, 2024
@javsalgar javsalgar added verify Execute verification workflow for these changes in-progress labels Dec 3, 2024
@github-actions github-actions bot removed the triage Triage is needed label Dec 3, 2024
@github-actions github-actions bot removed the request for review from javsalgar December 3, 2024 09:16
Signed-off-by: teodor.spiridonescu <[email protected]>
Signed-off-by: teodor.spiridonescu <[email protected]>
Signed-off-by: Bitnami Containers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress vault verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants