-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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.
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
[Serve] fix control+c after serve run
doesn't shutdown serve components
#45087
[Serve] fix control+c after serve run
doesn't shutdown serve components
#45087
Conversation
Signed-off-by: Gene Su <e870252314@gmail.com>
Signed-off-by: Gene Su <e870252314@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Gene!
@@ -964,5 +964,27 @@ def test_serve_run_mount_to_correct_deployment_route_prefix(ray_start_stop): | |||
) | |||
|
|||
|
|||
@pytest.mark.skipif(sys.platform == "win32", reason="File path incorrect on Windows.") | |||
def test_control_c_shutdown_serve_components(ray_start_stop): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I presume you tested that this fails w/o your change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, I purposely push a commit with just the test trying to see this failing in CI with the serve component still alive as well https://buildkite.com/ray-project/premerge/builds/25316#018f3652-b00a-45a2-aadc-ca46fb8d4f5d/182-3706
Signed-off-by: Gene Su <e870252314@gmail.com>
…ents (ray-project#45087) Added a test to ensure this is captured and fix it.
…ents (ray-project#45087) Added a test to ensure this is captured and fix it. Signed-off-by: pdmurray <peynmurray@gmail.com>
…ents (ray-project#45087) Added a test to ensure this is captured and fix it. Signed-off-by: Ryan O'Leary <ryanaoleary@google.com>
…ents (ray-project#45087) Added a test to ensure this is captured and fix it.
Why are these changes needed?
Added a test to ensure this is captured and fix it.
Related issue number
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.