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

main exit #912

Open
wants to merge 3 commits into
base: devel
Choose a base branch
from
Open

main exit #912

wants to merge 3 commits into from
+133 −13

Conversation

baum
Copy link
Collaborator

@baum baum commented Oct 22, 2024 •
edited
Loading

main exit

  1. do not catch System exit
  2. return exit code 0 from TERM signal handler
  3. tweak wait for gateways ha test

partial revert of

@baum baum force-pushed the main_exit branch 3 times, most recently from 2cdae48 to 6551853 Compare October 22, 2024 11:44
main exit
95af819 
1. Do not catch System exit
2. Return exit code 0 from TERM signal handler

partial revert of
- 6c120cf
- 042cf8c

Signed-off-by: Alexander Indenbaum <[email protected]>
@baum baum force-pushed the main_exit branch 2 times, most recently from 25f8504 to 5522e1d Compare October 22, 2024 12:42
Alexander Indenbaum added 2 commits October 22, 2024 12:51
initial main exit system test
e2eecc5 
Signed-off-by: Alexander Indenbaum <[email protected]>
tweak wait for gateways ha test
88d9021 
Signed-off-by: Alexander Indenbaum <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oritwas oritwas oritwas approved these changes

@gbregman gbregman gbregman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
NVMe-oF
Status: 👍 Ready to merge
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@baum @oritwas @gbregman