-
Notifications
You must be signed in to change notification settings - Fork 879
promote dns v1.26.0 images #8095
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
base: main
Are you sure you want to change the base?
Conversation
/cc @DamianSawicki |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: pacoxu The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The second commit is to add @DamianSawicki to approver list since he is owner of dns. https://github.com/kubernetes/dns/blob/master/OWNERS |
I was just going to do the promotion, so thanks so much for taking care of it instead! One request: there was a vulnerability PR merged yesterday, so I already cut 1.26.1. Could you please promote it instead (or both)? |
OK. I will add 1.26.1 ASAP. |
Added. |
Thanks so much for your prompt reaction! Actually, there's lots of activity on kubernetes/dns right now, and I've just cut another tag 1.26.2. The images are building right now, and I think it would be great if we could include 1.26.2 in this PR as well. @pacoxu Could you possibly add 1.26.2 images when they become available? In any case, I've reviewed the current version, and the changes to Update: Actually, it seems that some follow-up on kubernetes/dns#685 will be needed to comprehensively address vulnerabilities, so 1.26.2 is not the desired state yet. We can merge the current PR as it is or wait for another PR in kubernetes/dns to cut 1.26.3 and include it here. /lgtm |
New changes are detected. LGTM label has been removed. |
1.26.3 is not ready yet. I updated 1.26.2 in last commit.
|
Hi @pacoxu, thanks for following up!
As I wrote above, 1.26.2 is not the desired state yet, so I don't see much value in promoting it. Ideally, I would like to get kubernetes/dns#681 merged, cut 1.26.3, and then promote it. To avoid too much mess, I'd prefer to promote 1.26.0 and 1.26.3 or just 1.26.3. |
Got it. Dropped v1.26.1/v1.26.2 and wait for your 1.26.3 release. |
https://github.com/kubernetes/dns/releases/tag/1.26.0