[PR #901/914a16ec backport][stable-5] Add helm insecure skip tls verify #925
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a backport of PR #901 as merged into main (914a16e).
SUMMARY
Added the option
insecure_skip_tls_verify
to the following helm modules:helm_repository
helm
helm_pull
For
helm
, added the option to thehelm diff
call, as it got fixed upstream.Upstream Issue: databus23/helm-diff#503
Fixed with: helm/helm#12856
Fixes #694
ISSUE TYPE
COMPONENT NAME
kubernetes.core.helm
kubernetes.core.helm_repository
kubernetes.core.helm_pull
ADDITIONAL INFORMATION
Basically the option was added in the parameters set in the ansible job, in the docs and then injected in the
helm
andhelm diff
binary calls if set. Defaults toFalse
.Example
Note
Might need an alias for
telm_template
, as the option is calledinsecure_registry
, in the manual and docs of helm it would be--insecure-skip-tls-verify
as well though.Not included, as it was recently merged with #805