feat(rapid-response,sysdig-deploy): Verify that image.tag is a string #894
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.
Signed-off-by: Daniele De Lorenzi daniele.delorenzi@sysdig.com
What this PR does / why we need it:
We give the possibility to our customers to customise Rapid Response image for adding more tools inside it and therefore they will build the image to be hosted on a their repo with a their own image tag.
When specifying
rapidResponse.image.tag
to a value like1
,1.0
or20161216
this gets converted by helm into float64 instead of string and causes to have insidedaemonset.yaml
manifest a badimage:
value, ex:This seems to be a known issue on helm, see:
helm/helm#1707
helm/helm#11130
helm/helm#8978
I have updated the helper for verifying that
rapidResponse.image.tag
is a string and I have updated the values.yaml comments for image.tag parameter. A unit test for this scenario has been added as well.Potentially this issue is present in all our charts, shall we need to extend this change to all of them? even though usually customer don't customise the other components image they can still upload our image on a their private registry and tag them differently. What are your though on this? @aroberts87 @ironashram @lilx1ao
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
Check Contribution guidelines in README.md for more insight.