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

Error message is not formatted properly #3735

Closed
vivex opened this issue Jun 25, 2018 · 1 comment
Closed

Error message is not formatted properly #3735

vivex opened this issue Jun 25, 2018 · 1 comment

Comments

@vivex
Copy link

vivex commented Jun 25, 2018

What's going wrong?

While stopping a pm2 --no-daemon process it prints some console message, one of them prints %s directly without putting process name in it.

[Watch] Process name %s is being stopped so I won't restart it

How could we reproduce this issue?

Start any nodejs script with pm2 start app.js --watch --no-daemon , and then stop it by sending CTRL + C Signal.

Supporting information

18:19:32 PM2        | pm2 has been killed by signal, dumping process list before exit...
18:19:32 PM2        | Deleting process 0
18:19:32 PM2        | Stopping app:app id:0
2018-06-25 18:19:32: App [app] with id [0] and pid [88962], exited with code [0] via signal [SIGINT]
18:19:32 PM2        | App [app] with id [0] and pid [88962], exited with code [0] via signal [SIGINT]
2018-06-25 18:19:32: Change detected on path newrelic_agent.log for app app - restarting
2018-06-25 18:19:32: [Watch] Process name %s is being stopped so I won't restart it
18:19:32 PM2        | Change detected on path newrelic_agent.log for app app - restarting
18:19:32 PM2        | [Watch] Process name %s is being stopped so I won't restart it
2018-06-25 18:19:32: pid=88962 msg=process killed
2018-06-25 18:19:32: [PM2] Exited peacefully
$ pm2 report
vivex pushed a commit to vivex/pm2 that referenced this issue Jun 25, 2018
@vivex vivex mentioned this issue Jun 25, 2018
@vivex
Copy link
Author

vivex commented Jun 25, 2018

although its a minor thing, i have created a pr for it: #3736

@vivex vivex mentioned this issue Jun 27, 2018
wallet77 added a commit that referenced this issue Jul 4, 2018
inerc pushed a commit to inerc/pm2 that referenced this issue Feb 11, 2020
inerc pushed a commit to inerc/pm2 that referenced this issue Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants