-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
e2e don't set pod.Spec.NodeName directly #101184
Conversation
@aojea: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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/test-infra repository. |
/assign @wojtek-t @BenTheElder |
/test pull-kubernetes-e2e-ubuntu-gce-network-policies |
@aojea - I'm going to approve and hold - if you disagree with the comment, feel free to hold cancel :) /approve |
/lgtm |
great catch BTW |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, wojtek-t 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 |
based on this comment in https://github.com/kubernetes/kubernetes/blob/ea0764452222146c47ec826977f49d7001b0ea8c/test/e2e/framework/pod/node_selection.go#L96-L101 // pod.Spec.NodeName should not be set directly because // it will bypass the scheduler, potentially causing // kubelet to Fail the pod immediately if it's out of // resources. Instead, we want the pod to remain // pending in the scheduler until the node has resources // freed up.
/lgtm Thanks! |
/hold cancel |
/kind flake
based on this comment in
kubernetes/test/e2e/framework/pod/node_selection.go
Lines 96 to 101 in ea07644
// pod.Spec.NodeName should not be set directly because
// it will bypass the scheduler, potentially causing
// kubelet to Fail the pod immediately if it's out of
// resources. Instead, we want the pod to remain
// pending in the scheduler until the node has resources
// freed up.
and this commit from sig-storage to solve flakes on tests
d71829a