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

Add id to title #15

Closed
AdriVanHoudt opened this issue May 12, 2017 · 4 comments
Closed

Add id to title #15

AdriVanHoudt opened this issue May 12, 2017 · 4 comments

Comments

@AdriVanHoudt
Copy link

Would it be possible to add the id of the process to the message title?
If you have 4 processes with the same name this helps to see where an event happened.

@mattpker
Copy link
Owner

@ma-zal seems like something easy to add... would you like to tackle this one?

@abawchen
Copy link
Contributor

abawchen commented Nov 7, 2017

@ma-zal & @mattpker
Which process id do you want to add? pm_id(pm2 id) or pid(os process id)? I guess pm_id is enough, right?

I think I can help this easy one -- does it need to be configrable(show or hide)? or just display as name[id] by default.

@mattpker : this tool does save my day :)

@mattpker
Copy link
Owner

mattpker commented Nov 8, 2017

@abawchen I assume they would just need the pm2 process id. That way when they do a pm2 status they can see which pm2 process specifically is having issues.

I would vote for it just to be defaulted on with no config to turn it off. I don't see a reason why someone would specifically not want to display that in the slack messages.

@mattpker
Copy link
Owner

Fixed by #23

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

3 participants