Skip to content
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

Cherry pick PRs #11784, #11757 into release 1.20 bp #35

Merged

Conversation

ederst
Copy link

@ederst ederst commented Oct 13, 2021

This cherry picks changes of kubernetes#11757 and kubernetes#11784 into our release branch (release-1.20-bp).

This sorts the security groups of the port model and when querying the
cloud, which will result in tasks comparing those slices
deterministically.
This fixes the rendering of the Corefile of the NodeLocal DNSCache
template when setting forwardToKubeDNS to false (or not setting it).

Previously, due to not dereferencing the bool pointer, the Corefile was
always rendered with the true clause, due to checking the address
instead of the actual value of the variable.

With this fix, the templating mechanism will actually respect the value
of forwardToKubeDNS and render it accordingly.
This enables to use config drives instead of the metadata service as a
source for the user data (cloudinit).
@ederst ederst changed the title Cherrypick PRs #11784, #11757 into release 1.20 bp Cherry pick PRs #11784, #11757 into release 1.20 bp Oct 13, 2021
@ederst ederst added the merge label Oct 13, 2021
@github-actions github-actions bot enabled auto-merge October 13, 2021 12:08
@github-actions github-actions bot merged commit bcd3dd5 into release-1.20-bp Oct 13, 2021
@ederst ederst deleted the cherrypick-pr11784-pr11743-into-release-1.20-bp branch October 13, 2021 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants