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

Limit configmap cache&watch #2033

Closed
wants to merge 1 commit into from
Closed

Limit configmap cache&watch #2033

wants to merge 1 commit into from

Conversation

manno
Copy link
Member

@manno manno commented Dec 21, 2023

Refers to #2018

TODO

  • this seems to limit Get for configmaps, too. Fails when gitrepo reconciler tries to CreateOrUpdate the target.yaml configmap
  • DefaultNamespaces doesn't seem to work for : "unable to list: test-7af124d because of unknown namespace for the cache"

Opened: kubernetes-sigs/controller-runtime#2628

@manno manno requested a review from a team as a code owner December 21, 2023 15:41
@manno manno marked this pull request as draft December 21, 2023 16:00
@manno manno force-pushed the cr-limit-configmap-cache branch 2 times, most recently from 0050a8f to 42c7e7f Compare December 22, 2023 14:11
@manno manno force-pushed the cr-limit-configmap-cache branch 2 times, most recently from a42f11a to 768be38 Compare January 10, 2024 17:52
@manno manno force-pushed the cr-limit-configmap-cache branch from 768be38 to 2db7fab Compare May 29, 2024 12:23
@manno
Copy link
Member Author

manno commented May 29, 2024

The upstream issue has been closed. metav1.NamespaceAll use in CacheOptions seems to be intended to define different cache options for namespaces only.

However, using a simpler cache config seems to limit the cache, which can be proven with some benchmarking.

@manno manno force-pushed the cr-limit-configmap-cache branch from 2db7fab to b7fdd83 Compare May 29, 2024 12:49
@manno manno closed this May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant