Skip to content

Conversation

@andrew-anyscale
Copy link
Contributor

@andrew-anyscale andrew-anyscale commented Jan 27, 2026

Now handled in single source of truth at rayci.env. That envfile should be the only placeholder we update.

Topic: cleanup-manylinux
Relative: rayci-envfile-27
Signed-off-by: andrew andrew@anyscale.com

@andrew-anyscale
Copy link
Contributor Author

andrew-anyscale commented Jan 27, 2026

Reviews in this chain:
#60540 cleanup(ci): don't set default MANYLINUX_VERSION build_arg

@andrew-anyscale
Copy link
Contributor Author

andrew-anyscale commented Jan 27, 2026

# head base diff date summary
0 2691a366 3813b4f8 diff Jan 27 10:51 AM 4 files changed, 4 insertions(+), 4 deletions(-)
1 92674ade c9b78801 diff Jan 27 13:56 PM 0 files changed
2 40ef2098 c9b78801 diff Jan 27 15:07 PM 0 files changed

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the hardcoded default MANYLINUX_VERSION from several CI configuration files. By relying on a single source of truth from the environment (rayci.env), this change improves maintainability and consistency. The refactoring is applied correctly and consistently across all relevant files. This is a good cleanup.

Base automatically changed from andrew/revup/master/rayci-envfile-27 to master January 27, 2026 21:46
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/cleanup-manylinux branch from 2691a36 to 92674ad Compare January 27, 2026 21:56
Now handled in single source of truth at rayci.env. That envfile should be the only placeholder we update.

Topic: cleanup-manylinux
Relative: rayci-envfile-27
Signed-off-by: andrew <andrew@anyscale.com>
@andrew-anyscale andrew-anyscale marked this pull request as ready for review January 27, 2026 23:07
@andrew-anyscale andrew-anyscale force-pushed the andrew/revup/master/cleanup-manylinux branch from 92674ad to 40ef209 Compare January 27, 2026 23:07
@andrew-anyscale andrew-anyscale requested review from a team as code owners January 27, 2026 23:07
@andrew-anyscale andrew-anyscale added the go add ONLY when ready to merge, run all tests label Jan 27, 2026
@aslonnie aslonnie self-requested a review January 27, 2026 23:10
@aslonnie aslonnie self-requested a review January 27, 2026 23:12
@ray-gardener ray-gardener bot added the devprod label Jan 28, 2026
@aslonnie aslonnie merged commit aa34029 into master Jan 29, 2026
7 checks passed
@aslonnie aslonnie deleted the andrew/revup/master/cleanup-manylinux branch January 29, 2026 13:24
limarkdcunha pushed a commit to limarkdcunha/ray that referenced this pull request Jan 29, 2026
…ct#60540)

Now handled in single source of truth at rayci.env. That envfile should
be the only placeholder we update.

Topic: cleanup-manylinux
Relative: rayci-envfile-27
Signed-off-by: andrew <andrew@anyscale.com>

Signed-off-by: andrew <andrew@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devprod go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants