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
v1.14 Backports 2023-07-25 #27055
v1.14 Backports 2023-07-25 #27055
Conversation
[ upstream commit 6d6d1eb ] As a suggested follow up, further clarify that the MetalLB based BGP control plane is being deprecated. Signed-off-by: Louis DeLosSantos <louis.delos@isovalent.com> Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit 33c2726 ] Make it clear that this option applies to iptables-driven masquerading, not BPF-driven masquerading. Signed-off-by: Julian Wiedmann <jwi@isovalent.com> Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit 4863f81 ] In the example policy for l2-announcements, the example was using a label selector that said: `role: worker`. While it is possible to add such a label, most people will not do that and want to rely on the existing `node-role.kubernetes.io/<role>` labels added by kubeadm and similar tools. If an inexperienced user copies and pastes our example without modifying the node label selector, it will likely not work. This commit updates the example to be more realistic. Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com> Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
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.
Acking the multi-pool changes
This quick backport round was fast-tracked in preparation for the 1.14 release, it only makes changes to the documentation and helper strings, so we do not need to run any additional CI for this one. |
Documentation CI and Helm checks CI passed. This has got to be the fastest backport round in the history of backport rounds, maybe ever. Marking as |
Once this PR is merged, you can update the PR labels via:
or with