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
Poll for stats until Windows kubelet present it in the stats endpoint #116110
Poll for stats until Windows kubelet present it in the stats endpoint #116110
Conversation
cc @jsturtevant |
/test pull-kubernetes-e2e-capz-windows-containerd startup failure |
1 similar comment
/test pull-kubernetes-e2e-capz-windows-containerd startup failure |
/test pull-kubernetes-e2e-capz-windows-containerd
|
/test pull-kubernetes-e2e-capz-windows-containerd |
1 similar comment
/test pull-kubernetes-e2e-capz-windows-containerd |
/triage accepted |
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.
lgtm generally, one minor suggest for readability. Due to the infrequency of this flake, we will have to merge and monitor.
test/e2e/windows/host_process.go
Outdated
|
||
for _, containerStats := range podStats.Containers { | ||
statsChecked = true | ||
for _, podStats := range nodeStats.Pods { |
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.
could we move this validation to a function? It might make reading this easier.
0f207f7
to
677d849
Compare
677d849
to
3fd3a76
Compare
/lgtm |
LGTM label has been added. Git tree hash: 739a05b3cc707ba6a1ddf9f9cf8724365b41d40a
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jsturtevant, knabben 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
/kind failing-test
/sig windows
What this PR does / why we need it:
Polling the Kubelet stats endpoint until a CPU/Mem value appears for the pod, or fail close.
Which issue(s) this PR fixes:
Fixes #115932
Special notes for your reviewer:
Does this PR introduce a user-facing change?