Ensure signals are propagated to all nodejs processes#5910
Ensure signals are propagated to all nodejs processes#5910bert-e merged 1 commit intodevelopment/8.8from
Conversation
Pass the `-g` flag to tini, so that when a signal is received (typically from k8s' pod lifecycle management) it gets propagated to all node.js processes that were spawned. This behavior is similar to Ctrl+C in a terminal: > In macOS, as in other Unix-like operating systems (like Linux and BSD), > when you press Ctrl+C in your terminal, the terminal driver sends the > SIGINT (Signal Interrupt) signal to the entire foreground process group, > not just the single process you might see running. Issue: CLDSRV-727
Hello francoisferrand,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## development/8.8 #5910 +/- ##
================================================
Coverage 65.48% 65.48%
================================================
Files 188 188
Lines 12071 12071
================================================
Hits 7905 7905
Misses 4166 4166
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
|
/approve |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option The following options are set: approve |
Build failedThe build for commit did not succeed in branch w/9.1/bugfix/CLDSRV-727 The following options are set: approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-727. Goodbye francoisferrand. The following options are set: approve |
Pass the
-gflag to tini, so that when a signal is received (typicallyfrom k8s' pod lifecycle management) it gets propagated to all node.js
processes that were spawned.
This behavior is similar to Ctrl+C in a terminal:
Issue: CLDSRV-727