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
Bump cilium images #3802
Bump cilium images #3802
Conversation
702c41d
to
a9bf5bc
Compare
scripts/build-images
Outdated
${REGISTRY}/rancher/mirrored-cilium-hubble-relay:v1.12.5 | ||
${REGISTRY}/rancher/mirrored-cilium-hubble-ui:v0.10.0 | ||
${REGISTRY}/rancher/mirrored-cilium-hubble-ui-backend:v0.10.0 | ||
${REGISTRY}/rancher/mirrored-cilium-operator:v1.12.5 |
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.
I don't believe this image is used by the chart, is it? It only uses the generic/aws/azure version?
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.
Thank you for the review, @brandond. I have removed it accordingly and updated the description. PTAL
a9bf5bc
to
39e835d
Compare
39e835d
to
fe753c2
Compare
You need to update the chart version in rke2-charts and then re-run CI here once that is merged. |
Signed-off-by: Paulo Gomes <paulo.gomes@suse.com>
fe753c2
to
6d74764
Compare
LGTM. I think we are still in code freeze and we can't merge this yet. |
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.
LGTM. We can merge this after the January release code freeze has been lifted.
This should probably be backported to the 1.23-1.25 branches; can you open PRs to do so?
Proposed Changes
Patch release bumps for Cilium and minor release bump for Cilium Hubble.
Depends on: