Skip to content
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

fix(robot): skip kubelet terminated pod in get_workload_pods #1902

Conversation

c3y1huang
Copy link
Collaborator

Which issue(s) this PR fixes:

Issue longhorn/longhorn#8550

What this PR does / why we need it:

longhorn/longhorn#8550 (comment)

Special notes for your reviewer:

Additional documentation or context

@c3y1huang c3y1huang self-assigned this May 14, 2024
@c3y1huang c3y1huang force-pushed the fix-wait_for_workload_pods_stable-filter-pod-terminated-by-kubelet branch 3 times, most recently from 0b8b99b to 6a76ec4 Compare May 14, 2024 08:43
@c3y1huang c3y1huang marked this pull request as ready for review May 14, 2024 23:35
@c3y1huang c3y1huang requested a review from a team as a code owner May 14, 2024 23:35
@c3y1huang c3y1huang force-pushed the fix-wait_for_workload_pods_stable-filter-pod-terminated-by-kubelet branch 2 times, most recently from 6a7c6e0 to 302e1d8 Compare May 15, 2024 01:30
longhorn/longhorn-8550

Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
@c3y1huang c3y1huang force-pushed the fix-wait_for_workload_pods_stable-filter-pod-terminated-by-kubelet branch from 302e1d8 to b16a2b5 Compare May 15, 2024 01:39
Copy link
Member

@derekbit derekbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@derekbit derekbit requested a review from yangchiu May 15, 2024 02:15
Copy link
Contributor

@roger-ryao roger-ryao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@yangchiu yangchiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://ci.longhorn.io/job/private/job/longhorn-e2e-test/573/ failed.
Need to clarify if it's affected by this PR.

@c3y1huang c3y1huang requested a review from yangchiu May 15, 2024 02:33
@c3y1huang c3y1huang dismissed yangchiu’s stale review May 15, 2024 02:35

Requesting review again

@c3y1huang
Copy link
Collaborator Author

Copy link
Member

@yangchiu yangchiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yangchiu yangchiu merged commit 615bb5f into longhorn:master May 15, 2024
7 of 8 checks passed
@c3y1huang c3y1huang deleted the fix-wait_for_workload_pods_stable-filter-pod-terminated-by-kubelet branch May 16, 2024 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants