-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Check for nil cpuManager in container manager #88857
Conversation
/approve |
/kind bug |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: klueska, liggitt, nolancon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test pull-kubernetes-e2e-kind |
/retest |
@liggitt Can you manually trigger the failing ga conformance test, or do we have to wait for the next CI job to run? |
/test pull-kubernetes-conformance-kind-ga-only |
^ is serial and takes 90 minutes to run but if it gets past the cluster startup phase we'll know this resolved things |
Thanks, can you share a link to the job |
/retest |
1 similar comment
/retest |
e2e flake is #88861 |
/test pull-kubernetes-kubemark-e2e-gce-big |
/test pull-kubernetes-e2e-gce |
Thanks for the help @liggitt, now just to get past these flaking tests. |
#86318 flake /test pull-kubernetes-e2e-gce |
/test pull-kubernetes-kubemark-e2e-gce-big |
whew. all green and in the merge queue. |
What type of PR is this?
What this PR does / why we need it:
This PR is in response to #88855 with patch suggested in #87759 to check for nil cpuManager as can be the case when beta feature is disabled.
Which issue(s) this PR fixes:
Fixes #88855
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: