Exclude topology.kubernetes.io labels by default#43725
Conversation
Starting from version 1.35 Kuberentes propagetes topology.kubernetes.io labels from nodes to pods. This causes CID duplication and can lead to CID exhaustion. This patch adds !topology.kubernetes.io label filter to default label filters Fixes: cilium#43723 Signed-off-by: Maciej Moscicki <mmoscicki@google.com>
|
I'll at least nominate this for 1.19 backport, but we may want to think about backports to earlier releases as well (while balancing the risk). That said this is not a review, will defer to others on that for now. |
|
Commit a542840 does not match "(?m)^Signed-off-by:". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
a542840 to
c61bdf0
Compare
|
/test |
|
I'm strongly in favor of backporting this; it's not really a "code change" as much as it is a configuration update. Unexpected identity churn can quickly destabilize a cluster. |
|
/test |
Additionally, labels in the |
|
What a change! Good job @moscicky |
Starting from version 1.35 Kuberentes propagetes topology.kubernetes.io labels from nodes to pods. This causes CID duplication and can lead to CID exhaustion.
This patch adds !topology.kubernetes.io label filter to default label filters
Fixes: #43723