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.

Sign up for GitHub

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

Jump to bottom

[Bug] kubectl plugin e2e test is flaky #3140

Closed
1 of 2 tasks
kevin85421 opened this issue Mar 3, 2025 · 4 comments · Fixed by #3147
Closed
1 of 2 tasks

[Bug] kubectl plugin e2e test is flaky #3140

kevin85421 opened this issue Mar 3, 2025 · 4 comments · Fixed by #3147
Assignees
@MortalHappiness
Labels
bug Something isn't working cli kubectl plugin P0 Critical issue that should be fixed ASAP

Comments

@kevin85421
Copy link
Member

Search before asking

  • I searched the issues and found no similar issues.

KubeRay Component

ci

What happened + What you expected to happen

https://buildkite.com/ray-project/ray-ecosystem-ci-kuberay-ci/builds/7280#01955c7f-fd4e-4752-a4c7-c1e0862f7a29

Reproduction script

https://buildkite.com/ray-project/ray-ecosystem-ci-kuberay-ci/builds/7280#01955c7f-fd4e-4752-a4c7-c1e0862f7a29

Anything else

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!
@kevin85421 kevin85421 added bug Something isn't working triage P0 Critical issue that should be fixed ASAP cli kubectl plugin and removed triage labels Mar 3, 2025
@kevin85421
Copy link
Member Author

cc @davidxia @chiayi @MortalHappiness would you mind taking a look?

@MortalHappiness MortalHappiness self-assigned this Mar 3, 2025
MortalHappiness added a commit to MortalHappiness/kuberay that referenced this issue Mar 3, 2025
@MortalHappiness
[Bug] kubectl plugin e2e test is flaky
eca6bbf 
Closes: ray-project#3140
Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
@MortalHappiness MortalHappiness mentioned this issue Mar 3, 2025
Merged
4 tasks
@davidxia
Copy link
Contributor

davidxia commented Mar 3, 2025

Looks like #3004 broke the kubectl-plugin E2E tests on master. How did they pass in the PR?

@kevin85421
Copy link
Member Author

kevin85421 commented Mar 3, 2025 •
edited
Loading

I reran the failed test. If it still fails, I will revert #3004. cc @andrewsykim

@kevin85421
Copy link
Member Author

@MortalHappiness has already fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MortalHappiness MortalHappiness

Labels
bug Something isn't working cli kubectl plugin P0 Critical issue that should be fixed ASAP
Projects
None yet
Milestone
No milestone
3 participants
@davidxia @kevin85421 @MortalHappiness