Skip to content

fix: ensure graceful shutdown for Ray head container when GCS fault tolerance is enabled #4079

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
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

fix: ensure graceful shutdown for Ray head container when GCS fault tolerance is enabled #4079

wants to merge 1 commit into from

Conversation

Future-Outlier
Copy link
Member

@Future-Outlier Future-Outlier commented Sep 18, 2025 •
edited
Loading

Why are these changes needed?

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

@Future-Outlier Future-Outlier changed the title fix: ensure graceful shutdown for Ray head pods when GCS fault tolerance is enabled fix: ensure graceful shutdown for Ray head container when GCS fault tolerance is enabled Sep 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevin85421 kevin85421 Awaiting requested review from kevin85421 kevin85421 will be requested when the pull request is marked ready for review kevin85421 is a code owner

@andrewsykim andrewsykim Awaiting requested review from andrewsykim andrewsykim will be requested when the pull request is marked ready for review andrewsykim is a code owner

@rueian rueian Awaiting requested review from rueian rueian will be requested when the pull request is marked ready for review rueian is a code owner

@MortalHappiness MortalHappiness Awaiting requested review from MortalHappiness MortalHappiness will be requested when the pull request is marked ready for review MortalHappiness is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Future-Outlier