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: add kill_retry_time argument #3539

Merged
merged 2 commits into from
Mar 23, 2018
Merged

Conversation

KimSeongIl
Copy link
Contributor

@KimSeongIl KimSeongIl commented Mar 16, 2018

Please always submit pull requests on the development branch.

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #1234, #3524
License MIT
Doc PR https://github.com/pm2-hive/pm2-hive.github.io/pulls

Please update this template with something that matches your PR

Too many logs are output at shutdown with graceful shutdown.


PM2        | pid=4382 msg=failed to kill - retrying in 100ms
PM2        | pid=4399 msg=failed to kill - retrying in 100ms
PM2        | pid=4364 msg=failed to kill - retrying in 100ms
PM2        | pid=4365 msg=failed to kill - retrying in 100ms
PM2        | pid=4382 msg=failed to kill - retrying in 100ms
PM2        | pid=4399 msg=failed to kill - retrying in 100ms
PM2        | pid=4364 msg=failed to kill - retrying in 100ms
PM2        | pid=4365 msg=failed to kill - retrying in 100ms
PM2        | pid=4382 msg=failed to kill - retrying in 100ms
PM2        | pid=4399 msg=failed to kill - retrying in 100ms
PM2        | pid=4364 msg=failed to kill - retrying in 100ms
PM2        | pid=4365 msg=failed to kill - retrying in 100ms
...........
...........

I think we should be able to set the retry time.

@wallet77 wallet77 merged commit 1325704 into Unitech:development Mar 23, 2018
inerc pushed a commit to inerc/pm2 that referenced this pull request Feb 11, 2020
feat: add kill_retry_time argument
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants