Hi there !

We are using your module to monitor our NodeJS application and it works just fine ! Thanks !

All the collected metrics are describing processes and named accordingly pm2.processes.*. I fully understand the need of the instance tag but it would make more sense to name it process_id. That would align better with metrics' name and be more correct in my opinion : One PM2 instance is managing several NodeJS processes.

I also understand that renaming that tag would be a breaking change, even though Datadog allows you to rename your template variables to avoid modifying all your widgets in such case. Creating a duplicated tag could be a temporary solution for the transition.

I agree that this is a very opinionated suggestion. :-)

Cheers

When I run sudo pm2 install uphold/pm2-datadog I always get the error:

Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t ssh://[email protected]/uphold/pm2-datadog.git
npm ERR!
npm ERR! ERROR: Repository not found.
npm ERR! fatal: Could not read from remote repository.
npm ERR!
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR!
npm ERR! exited with error code: 128

Seems like git is trying to clone via SSH (why?).