[release-0.47] optimize init map ns#557
[release-0.47] optimize init map ns#557RamLavi merged 4 commits intok8snetworkplumbingwg:release-0.47from
Conversation
Currently kubemacpool scans all pods cluster-wide and performs expensive IsPodManaged() checks for each pod during startup. This causes slow initialization in large clusters with many unmanaged namespaces. Pre-compute managed namespaces once per webhook type and only scan pods from those namespaces, eliminating redundant API calls and reducing startup time. Signed-off-by: Ram Lavi <ralavi@redhat.com>
Currently kubemacpool scans all VMs cluster-wide and performs expensive IsVMManaged() checks for each VM during startup. This causes slow initialization in large clusters with many unmanaged namespaces. Pre-compute managed namespaces once per webhook type and only scan VMs from those namespaces, eliminating redundant API calls and reducing startup time. Signed-off-by: Ram Lavi <ralavi@redhat.com>
To anticipate big clusters with 10K VMs, increasing the readiness probe settle time to 3m. Signed-off-by: Ram Lavi <ralavi@redhat.com>
|
/hold trying to debug the new flake |
|
issue is not relevant to this failed test. |
|
@RamLavi: Overrode contexts on behalf of RamLavi: pull-kubemacpool-e2e-k8s DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/hold cancel |
|
/lgtm |
|
@RamLavi: you cannot LGTM your own PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: RamLavi The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@maiqueb: changing LGTM is restricted to collaborators DetailsIn response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
RamLavi
merged commit d610744
into
k8snetworkplumbingwg:release-0.47
3 participants
What this PR does / why we need it:
This is a manual cherry-pick of #556
Special notes for your reviewer:
Release note: