Skip to content

wip ray version update#11184

Draft
mszadkow wants to merge 4 commits into
kubernetes-sigs:mainfrom
epam:cleanup/update-ray-version
Draft

wip ray version update#11184
mszadkow wants to merge 4 commits into
kubernetes-sigs:mainfrom
epam:cleanup/update-ray-version

Conversation

@mszadkow
Copy link
Copy Markdown
Contributor

@mszadkow mszadkow commented May 14, 2026

What type of PR is this?

What this PR does / why we need it:

Added .RayStartParam(rayv1.HeadNode, "object-store-memory", "100000000") to both failing InTreeAutoscaling RayJob tests. This limits GCS object store memory to 100MB, reducing GCS load so worker raylets can register within the 30-second timeout — matching the fix already present in the passing RayService InTreeAutoscaling test.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

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-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels May 14, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for kubernetes-sigs-kueue canceled.

Name Link
🔨 Latest commit 87e6e6f
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-sigs-kueue/deploys/6a05ff016329760008c9fc7c

@mszadkow
Copy link
Copy Markdown
Contributor Author

/ok-to-test

@k8s-ci-robot k8s-ci-robot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label May 14, 2026
@k8s-ci-robot k8s-ci-robot requested review from mimowo and tenzen-y May 14, 2026 11:21
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 14, 2026
@mszadkow mszadkow force-pushed the cleanup/update-ray-version branch from 1e63f8e to 34c7ce3 Compare May 14, 2026 11:30
@mszadkow
Copy link
Copy Markdown
Contributor Author

/test pull-kueue-test-e2e-main-1-35

@mszadkow
Copy link
Copy Markdown
Contributor Author

/test pull-kueue-test-e2e-main-1-34
/test pull-kueue-test-e2e-main-1-33

@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 14, 2026
@mszadkow
Copy link
Copy Markdown
Contributor Author

/test pull-kueue-test-e2e-main-1-34
/test pull-kueue-test-e2e-main-1-33
/test pull-kueue-test-e2e-main-1-35

@mszadkow mszadkow force-pushed the cleanup/update-ray-version branch from 2936589 to a61ee2b Compare May 14, 2026 16:36
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mszadkow
Once this PR has been reviewed and has the lgtm label, please assign mimowo for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mszadkow
Copy link
Copy Markdown
Contributor Author

/test pull-kueue-test-e2e-main-1-35

@mszadkow
Copy link
Copy Markdown
Contributor Author

/test pull-kueue-test-e2e-main-1-35

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 14, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

@mszadkow: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-kueue-test-e2e-main-1-33 16f674e link true /test pull-kueue-test-e2e-main-1-33
pull-kueue-test-e2e-main-1-34 16f674e link true /test pull-kueue-test-e2e-main-1-34
pull-kueue-test-e2e-main-1-35 87e6e6f link true /test pull-kueue-test-e2e-main-1-35

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

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-sigs/prow repository. I understand the commands that are listed here.

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

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants