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
helm: Bump default spire image version #25444
Conversation
Testing was done locally to make sure there is no breaking change. ``` $ cilium connectivity test --test auth ... [=] Test [echo-ingress-auth-always-fail] ........ [=] Test [echo-ingress-auth-mtls-spiffe] ........ ✅ All 2 tests (16 actions) successful, 45 tests skipped, 0 scenarios skipped. ``` Signed-off-by: Tam Mach <tam.mach@cilium.io>
391b323
to
84fe5fd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but please rework the description of the second commit. The motivation is not clear to me (Do you mean that you add this line to the config file to have the bot update the SPIRE images using the info from values.yaml.tmpl
, and that unless we add more config this is the only image it should update? If so, please mention explicitly that the intent is to handle updates for the SPIRE images. Or do you mean something else?)
+ Typo in commit title, configu
.
This is to make sure that spire images mentnioned in values.yaml.tmpl are updated automatically by renovate bot. While there are other image values, renovate bot should only update spire image values as these are following one liner format. Signed-off-by: Tam Mach <tam.mach@cilium.io>
84fe5fd
to
6dc63c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reads much better, thank you!
Travis simply had a hiccup: |
/test |
run-time failed for below issue
|
/mlh test-runtime |
/mlh new-flake Cilium-PR-Runtime-net-next |
/test-runtime |
Description
Testing was done locally to make sure there is no breaking change.
Signed-off-by: Tam Mach tam.mach@cilium.io