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

labelsfilter: ignore Job completion index label by default for CID creation #28897

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

tosi3k
Copy link
Contributor

@tosi3k tosi3k commented Oct 31, 2023

This prevents creating a CiliumIdentity object per Pod that is owned by an Indexed Job. This was especially costly in clusters with large quantities of Indexed Job-controlled Pods and basically limited the number of such Pods in a single cluster to a theoretical max number of CID objects (2^16 ~ 65k).

This is a follow-up to #28003 after the relevant change in the upstream K8s 1.28+.

Ignore Indexed Job-specific label by default for CID creation `batch.kubernetes.io/job-completion-index`.

…eation

This prevents creating a CiliumIdentity object per Pod that is owned by
an Indexed Job. This was especially costly in clusters with large
quantities of Indexed Job-controlled Pods and basically limited the
number of such Pods in a single cluster to a theoretical max number of
CID objects (2^16 ~ 65k).

Signed-off-by: Antoni Zawodny <zawodny@google.com>
@tosi3k tosi3k requested review from a team as code owners October 31, 2023 10:55
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 31, 2023
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Oct 31, 2023
@qmonnet
Copy link
Member

qmonnet commented Oct 31, 2023

/test

Copy link
Member

@nebril nebril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! LGTM.

@qmonnet qmonnet added sig/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. labels Oct 31, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 31, 2023
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you

@aanm aanm merged commit f1ae3ce into cilium:main Oct 31, 2023
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/community-contribution This was a contribution made by a community member. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. sig/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants