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.

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

[NBS] The vhost-server can catch erroneous SIGUSR2 #2013

Closed
komarevtsev-d opened this issue Sep 12, 2024 · 0 comments · Fixed by #2014, #2022, #2032, #2057 or #2064
Closed

[NBS] The vhost-server can catch erroneous SIGUSR2 #2013

komarevtsev-d opened this issue Sep 12, 2024 · 0 comments · Fixed by #2014, #2022, #2032, #2057 or #2064
Assignees
@komarevtsev-d
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR bug Something isn't working

Comments

@komarevtsev-d
Copy link
Collaborator

Since the vhost-server uses PR_SET_PDEATHSIG it tracks the aliveness of the parent thread. The NBS spawns the vhost-server from a coroutine thread which has an unspecified time of life.
@komarevtsev-d komarevtsev-d added bug Something isn't working blockstore Add this label to run only cloud/blockstore build and tests on PR labels Sep 12, 2024
@komarevtsev-d komarevtsev-d self-assigned this Sep 12, 2024
@komarevtsev-d komarevtsev-d linked a pull request Sep 12, 2024 that will close this issue
Spawn the vhost-server from special thread #2014
Merged
This was linked to pull requests Sep 13, 2024
Merge to stable 23-3 #2022
Merged
Switch coroutine wait to CondVar #2032
Merged
Simlify the vhost server spawn #2057
Merged
@komarevtsev-d komarevtsev-d mentioned this issue Sep 18, 2024
Merged
@komarevtsev-d komarevtsev-d linked a pull request Sep 18, 2024 that will close this issue
[stable-23-3] Merge to stable #2064
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@komarevtsev-d komarevtsev-d

Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@komarevtsev-d