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.

Sign up for GitHub

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

Jump to bottom

[bitnami/redis] Fix preExecCmds parameter #29898

Merged
merged 2 commits into from
Oct 15, 2024
Merged

[bitnami/redis] Fix preExecCmds parameter #29898

merged 2 commits into from
Oct 15, 2024
+19 −8

Conversation

dgomezleon
Copy link
Member

Description of the change

Fix preExecCmds to work as array.

Benefits

Allow to use preExecCmds parameter.

Possible drawbacks

N/A

Applicable issues

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)

Sorry, something went wrong.

@dgomezleon
[bitnami/redis] Fix preExecCmds parameter
Loading
Loading status checks…
240c87b 
Signed-off-by: David Gomez <dgomezleon@vmware.com>
@github-actions github-actions bot added redis verify Execute verification workflow for these changes bitnami labels Oct 15, 2024
@github-actions github-actions bot requested a review from jotamartos October 15, 2024 12:42
Update CHANGELOG.md
Loading
Loading status checks…
50ecee8 
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
@dgomezleon dgomezleon merged commit 6db8a2e into main Oct 15, 2024
10 checks passed
@dgomezleon dgomezleon deleted the Redis-preExecCmds branch October 15, 2024 14:00
@pcorpet
Copy link

pcorpet commented Oct 15, 2024

Thanks @dgomezleon for the quick fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@javsalgar javsalgar

@jotamartos jotamartos

Assignees

@jotamartos jotamartos

Labels
bitnami redis solved verify Execute verification workflow for these changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bitnami/redis] preExecsCmd type conflict
4 participants
@dgomezleon @pcorpet @javsalgar @jotamartos