-
Notifications
You must be signed in to change notification settings - Fork 8
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
PM2 running more than 1 clustered process #1
Comments
You're absolutely right, it should filter out other pm2 processes.
It might be a couple of days before I can update this though.
…On Wed, Dec 12, 2018, 10:51 rodenj1 ***@***.***> wrote:
First off, just wanted to say I have been looking for something like this
for a while! Nice work!
It looks like as part of your metric gathering process that you send a
message to all PM2 controlled process to report their stats and this works
great if you only have 1 clustered process running in PM2. If you have
multiple, it sends a request to all and waits for a reply. If the other
process is not running your module, it does not respond causing a timeout.
If it is running your module, it will inadvertently report stats for a
process a different process. I was wondering if you could add a filter to
only send a message to process with the same process name "
process.env.name". I figure a quick filter under getProcList would do the
trick. :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/ADAoMdi0_DTyoURi1UbH9YVN64Jfof-Eks5u4GEcgaJpZM4ZOlGp>
.
|
A couple of days, or 7 hours. NPM package version is 1.0.6. Please let me know if this works for you now. |
Awesome! Thank you!
John
… On Dec 12, 2018, at 1:12 AM, Jake ***@***.***> wrote:
A couple of days, or 7 hours.
NPM package version is 1.0.5. Please let me know if this works for you now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AH7heiQe94ZPWqQ0qq2kvDu-CuM_NcoNks5u4MiUgaJpZM4ZOlGp>.
|
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
First off, just wanted to say I have been looking for something like this for a while! Nice work!
It looks like as part of your metric gathering process that you send a message to all PM2 controlled process to report their stats and this works great if you only have 1 clustered process running in PM2. If you have multiple, it sends a request to all and waits for a reply. If the other process is not running your module, it does not respond causing a timeout. If it is running your module, it will inadvertently report stats for a process a different process. I was wondering if you could add a filter to only send a message to process with the same process name "process.env.name". I figure a quick filter under getProcList would do the trick. :)
The text was updated successfully, but these errors were encountered: