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
Update kubedns and nodelocaldns to v1.22.28 #121908
Update kubedns and nodelocaldns to v1.22.28 #121908
Conversation
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/sig cloud-provider |
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
/approve
cc @DamianSawicki
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: MrHohn, mzaian The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
LGTM label has been added. Git tree hash: 167c8b39da5eaec6e7984a819179b5f5b9847efc
|
Actually, holding to check if we need to do manual promotion in https://github.com/kubernetes/k8s.io/blob/main/k8s.gcr.io/images/k8s-staging-dns/images.yaml. |
Looks like that was done in kubernetes/k8s.io#6068. https://github.com/kubernetes/k8s.io/blob/main/registry.k8s.io/images/k8s-staging-dns/images.yaml is the new image manifest location. /unhold |
Shouldn't we make a release fot the latest tag https://github.com/kubernetes/dns/releases/tag/1.22.28 and bump to that? |
Thanks, I missed that - @mzaian let's pick 1.22.28 instead as there are CVE fixes. /hold |
New release created https://github.com/kubernetes/dns/releases/tag/1.22.28. Some time is certainly needed for the images to build. |
Thanks a lot for your feedback. I created kubernetes/k8s.io#6088 to promote the images. Once merged I will update this PR to point to 1.22.28 |
59ba5fd
to
337b164
Compare
/lgtm |
LGTM label has been added. Git tree hash: f989e09320043fe9f2cfeabbb2a6b3c6ab141924
|
kubernetes/k8s.io#6088 Is merged now /unhold |
I think this is ready to be merged, thank you @mzaian! |
we are in code freeze, until master branc opens again next week this can not merge |
#121736 bump to v1.22.27. /retest |
337b164
to
259ed59
Compare
Rebase is done and resolved all the conflicts |
Thanks @mzaian! /lgtm |
LGTM label has been added. Git tree hash: 9ed8c6caaf6ab48e7178a2061b3d3ebd573f0129
|
/retest |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Update kubedns and nodelocaldns versions. See release: 1.22.28.
Which issue(s) this PR fixes:
Fixes NONE #
Does this PR introduce a user-facing change?:
NONE