-
Notifications
You must be signed in to change notification settings - Fork 40.1k
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
kubeadm: update coredns to 1.8.4 #102466
kubeadm: update coredns to 1.8.4 #102466
Conversation
Skipping CI for Draft Pull Request. |
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.
You would want to add the changes in #101955 in this PR to add permissions for endpointslices
@rajansandeep I cherry-picked it to this PR. |
the coredns migration library was just updated in this PR: |
no worries, thanks for updating the PR description, I mostly wanted to avoid automatic issue closure in this case 😅 |
/assign @fabriziopandini |
/cc neolit123 |
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.
/approve
/hold
/priority important-soon
LGTM, please update the release note to something like:
kubeadm: update CoreDNS to 1.8.4. Grant CoreDNS permissions to "list" and "watch" EndpointSlice objects to accommodate dual-stack support.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, neolit123, pacoxu 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 |
/retitle kubeadm: update coredns to 1.8.4 |
/triage accepted |
Updated |
/hold cancel |
/retest Review the full test history for this PR. Silence the bot with an |
/retest |
FYI: @neolit123 Last week a regression was reported for 1.8.4 for the rewrite plugin. There is a workaround. FYI AFAIK, AKS has decided to defer rolling out 1.8.4, skipping over the version, since a fix will be in 1.8.5 (already merged).
FWIW, this has always been the case for some clients (some clients are extra picky about question mismatch), but the regression caused it to affect most clients (if not all). My attempt to explain in more detail here. Also, regarding the workaround, |
@chrisohaver Kubeadm default coredns corefile doesn’t include ‘rewrite’ plug-in. However, it Will affect user using rewrite plugin. |
Yes. That's correct. Hard to know how many people use it, and how many people who use it are not already doing a response rewrite (as would be prudent). |
What type of PR is this?
/kind feature
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
xref #99751
Special notes for your reviewer:
Does this PR introduce a user-facing change?