generated from ansible-collections/collection_template
-
Notifications
You must be signed in to change notification settings - Fork 143
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
Installed Helm diff plugin may break deploy #669
Labels
Comments
patchback bot
pushed a commit
that referenced
this issue
Mar 1, 2024
Align `helmdiff_check` behavior with the `deploy` function SUMMARY Align helmdiff_check behavior with the deploy function Fixes #638 helmdiff_check respects set_values parameter Fixes #669 helmdiff_check command line parameters sequence aligned to the deploy function ISSUE TYPE Bugfix Pull Request COMPONENT NAME kubernetes.core.helm Reviewed-by: Mike Graves <[email protected]> (cherry picked from commit 7c4ec3b)
abikouo
pushed a commit
that referenced
this issue
Mar 1, 2024
Align `helmdiff_check` behavior with the `deploy` function SUMMARY Align helmdiff_check behavior with the deploy function Fixes #638 helmdiff_check respects set_values parameter Fixes #669 helmdiff_check command line parameters sequence aligned to the deploy function ISSUE TYPE Bugfix Pull Request COMPONENT NAME kubernetes.core.helm Reviewed-by: Mike Graves <[email protected]> (cherry picked from commit 7c4ec3b) Co-authored-by: psmolkin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
SUMMARY
When you have the Helm Diff plugin installed, the
kubernetes.core.helm
module run thediff
command before deploy. But, the command line generation logic works different and put generated yaml of a[release_]values
dict to the first place beforevalues_files
, that means that these generated values become hidden.This is quite similar to the #638 issue.
I'm going to raise a PR.
ISSUE TYPE
COMPONENT NAME
kubernetes.core.helm
ANSIBLE VERSION
COLLECTION VERSION
CONFIGURATION
OS / ENVIRONMENT
doesn't matter
STEPS TO REPRODUCE
where
values-commons.yaml
is:EXPECTED RESULTS
The result
region:
value rendered tohidden_one
ACTUAL RESULTS
The result
region
value forhelm diff upgrade
will bedefault
instead ofhidden_one
The text was updated successfully, but these errors were encountered: