Skip to content
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

feat(core): Log runtime names during Server shutdown #512

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

theduke
Copy link

@theduke theduke commented Jan 12, 2025

Log the names of runtimes during Server shutdown.

Motivation: server shutdown can take quite a long time if the runtimes
don't exit.
It's hard to debug why this is the case, because there is no indiciation
in the logs which service is pending.

Same goes for shutdown failures.

This small change modifies the server to log the name of the service
whose runtime is being waited for.

Log the names of runtimes during Server shutdown.

Motivation: server shutdown can take quite a long time if the runtimes
don't exit.
It's hard to debug why this is the case, because there is no indiciation
in the logs which service is pending.

This small change modifies the server to log the name of the service
whose runtime is being waited for.
@theduke theduke force-pushed the server-log-runtime-names-during-shutdown branch from be31479 to 278730c Compare January 12, 2025 16:53
@gumpt gumpt added the enhancement New feature or request label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants