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
tests: Spawn poststart / prestop pods on the same node as the http pod #101063
tests: Spawn poststart / prestop pods on the same node as the http pod #101063
Conversation
@claudiubelu: The label(s) In 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/test-infra repository. |
15fef69
to
ef556f1
Compare
/retest |
/triage accepted /retest |
/assign @fromanirh |
/test pull-kubernetes-integration |
Only in Windows Overlay networks (ie Flannel where these tests are flaking) does the pod have to be on the same node. These tests are not flaky on l2bridge setup. I would be more inclined to disable this test for Windows Overlay test setups only and add a sig-windows test for the overlay functionality that requires it to be on the same node (but not sure if that is useful in an meaningful way?) |
62359bd
to
90cd3d6
Compare
In the case of multinode clusters, the http server pod and the test cluster can spawn on different nodes, which can be problematic for poststart / prestop hooks, as they are executed by the kubelet itself, and the cross-node lifecycle hook might fail (according to the Kubernetes network model, it is not mandatory for kubelet to be able to access pods on a different node). This commit ensures that the test pod spawns on the same node as the http server pod.
d252fe5
to
f9e4a01
Compare
/hold cancel |
what is this conformance pod doing?
|
/test pull-kubernetes-conformance-image-test this job is very cryptic, I can't see any meaningful log, however, the kind jobs passed so I assume it should be a problem with that specific job |
Needs milestone. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/lgtm
/milestone v1.23
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: claudiubelu, spiffxp 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 |
What type of PR is this?
/kind flake
/sig testing
/sig networking
/priority important-soon
What this PR does / why we need it:
In the case of multinode clusters, the http server pod and the test cluster can spawn on different nodes, which can be problematic for poststart / prestop hooks, as they are executed by the kubelet itself, and the cross-node lifecycle hook might
fail (according to the Kubernetes network model, it is not mandatory for kubelet to be able to access pods on a different node).
This PR ensures that the test pod spawns on the same node as the http server pod.
Which issue(s) this PR fixes:
Fixes #101062
Special notes for your reviewer:
Flakes:
https://prow.k8s.io/view/gs/k8s-ovn/logs/k8s-e2e-ltsc2019-containerd-flannel-sdnoverlay-master/1380264207156514816
https://prow.k8s.io/view/gs/k8s-ovn/logs/k8s-e2e-ltsc2019-containerd-flannel-sdnoverlay-master/1380083012074475520
Not flaking:
https://testgrid.k8s.io/sig-windows-networking#sac1909-containerd-flannel-sdnbridge-stable
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: