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

Petal processes kill #464

Closed
nsosio opened this issue Nov 1, 2023 · 1 comment · Fixed by #468
Closed

Petal processes kill #464

nsosio opened this issue Nov 1, 2023 · 1 comment · Fixed by #468
Assignees
Labels
enhancement New feature or request

Comments

@nsosio
Copy link
Contributor

nsosio commented Nov 1, 2023

Replace kill -s SIGKILL of petal processes with:

  • SIGINT first with a timeout
  • SIGTERM first with a timeout
  • SIGKILL
@nsosio nsosio added the enhancement New feature or request label Nov 1, 2023
@nsosio nsosio added this to Prem App Nov 1, 2023
@nsosio nsosio mentioned this issue Nov 1, 2023
4 tasks
@nsosio nsosio self-assigned this Nov 1, 2023
@swarnimarun
Copy link
Contributor

Note: SIGINT is windows whereas SIGTERM is unix.

If if just want timeout on unix then, --timeout flag exists on /usr/bin/kill (not on shell kill).

You can write it as,

/usr/bin/kill --timeout 1000 TERM --signal QUIT ${PID_TO_KILL}

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 a pull request may close this issue.

2 participants