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

Show original parameters on process info/describe #1044

Open
CGastrell opened this issue Feb 26, 2015 · 3 comments
Open

Show original parameters on process info/describe #1044

CGastrell opened this issue Feb 26, 2015 · 3 comments

Comments

@CGastrell
Copy link

This is just a suggestion I'd consider and enhancement (I read the contribution guidelines and I think it fits).

I think it would be helpful to know which parameters were used to start a process. I find myself deleting/stopping and recreating processes manually just to ensure I use the correct startup parameters (watch, ignore-watch, user, log-date-format, name, etc).

I know all these can be saved in a startup script, but I also think a little entry with the original start arguments wouldn't hurt and will come in handy when issuing:

pm2 describe <pid>
# or even
pm2 [prettylist | jlist]

Great work, cheers.

@soyuka
Copy link
Collaborator

soyuka commented Feb 26, 2015

I can see two ways to do this:

  • enable debugging with the environment variable: export DEBUG="pm2:monit"
  • pm2 desc, prettylist, jlist as you mentioned

I don't think we would want to "pollute" the pm2 list with more parameters than we already have. Were would you like to see these parameters?

@CGastrell
Copy link
Author

Well, pm2 describe <pid> would be a nice place, since it simply looks coherent to see these kind of information on a process while "describing" it.

I agree it wouldn't be wise to pollute pm2 [list | ls] anymore.

@Unitech
Copy link
Owner

Unitech commented Feb 28, 2015

+1 to add this metadata to pm2 describe. This is straightforward, we just need to add some cols there to display more value of the pm2_env object: https://github.com/Unitech/PM2/blob/master/lib/CliUx.js#L63

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