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/memcached] Set usePasswordFiles=true by default #32362

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

Conversation

migruiz4
Copy link
Member

@migruiz4 migruiz4 commented Mar 7, 2025

Description of the change

Sets default value for usePasswordFiles to true.

Benefits

With this change, the chart will mount the secrets as files by default instead of directly setting them into environment variables.

Possible drawbacks

None

Applicable issues

None

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 memcached verify Execute verification workflow for these changes bitnami labels Mar 7, 2025
@github-actions github-actions bot requested a review from gongomgra March 7, 2025 10:00
Bitnami Containers and others added 4 commits March 7, 2025 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami memcached verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants