Skip to content

[RayService][e2e] fix flaky test TestAutoscalingRayService #4702

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

Merged
andrewsykim merged 4 commits into from
Apr 15, 2026
+2 −2
Merged

[RayService][e2e] fix flaky test TestAutoscalingRayService #4702

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
80ff116
fix: increase head pod CPU limits in e2e test YAMLs to fix dashboard …
AndySung320 Apr 13, 2026
25891ee
Merge remote-tracking branch 'upstream/master' into flaky/e2e-rayservice
Future-Outlier Apr 15, 2026
350878e
update
Future-Outlier Apr 15, 2026
c436dc0
Merge remote-tracking branch 'upstream/master' into flaky/e2e-rayservice
Future-Outlier Apr 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions ray-operator/test/e2erayservice/testdata/rayservice.autoscaling.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ spec:
image: rayproject/ray:2.52.0
resources:
requests:
cpu: 300m
cpu: "1"
memory: 1G
limits:
cpu: 500m
cpu: "1"
Copy link
Copy Markdown
Member

@andrewsykim andrewsykim Apr 15, 2026

Choose a reason for hiding this comment

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

Can we also consider removing limits entirely? Or does the autoscaling test require it?

Copy link
Copy Markdown
Member

@Future-Outlier Future-Outlier Apr 16, 2026

Choose a reason for hiding this comment

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

cc @AndySung320 to do the investigation, and update the yaml manifest if needed

memory: 2G
ports:
- containerPort: 6379
Expand Down
Loading