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

High memory usage #45

Closed
DevBrent opened this issue Feb 27, 2017 · 9 comments
Closed

High memory usage #45

DevBrent opened this issue Feb 27, 2017 · 9 comments

Comments

@DevBrent
Copy link

DevBrent commented Feb 27, 2017

First use-case, ubuntu14.04
pm2-logrotate 2.0.0
PMX version 0.6.8
Global logs size 20.87 GB (yeah, we should probably turn on cleanup)
Files count 8.4K
Memory Usage: 958.6 MB

Second use-case, amazon linux-ami
pm2-logrotate 2.0.0
PMX version 0.6.8
Global logs size 147.11 MB
Files count 836
Memory usage: 579.4 MB

@vmarchaud vmarchaud self-assigned this Feb 27, 2017
@vmarchaud
Copy link
Contributor

Are you using the default configuration ? If not could you please provide the configuration you are using (you can get it by running pm2 conf)

@FredrikSeidl
Copy link

FredrikSeidl commented Mar 7, 2017

We have the same issue in our environment, also on 2.0.0. Our settings are below

retain=5
max_size=100M

@korentsov
Copy link

korentsov commented Apr 7, 2017

We have same issue, version 2.2.0, pm2 version 2.4.0

Module activated ┌───────────────┬─────────┬────────────┬────────┬─────────┬─────┬──────────────┐ │ Module │ version │ target PID │ status │ restart │ cpu │ memory │ ├───────────────┼─────────┼────────────┼────────┼─────────┼─────┼──────────────┤ │ pm2-logrotate │ 2.2.0 │ N/A │ online │ 0 │ 0% │ 251.965 MB │ └───────────────┴─────────┴────────────┴────────┴─────────┴─────┴──────────────┘ Usepm2 show <id|name> to get more details about an app

Our settings:
== module-db == │ pm2-logrotate │ true == pm2-logrotate == │ max_size │ 10M │ interval_unit │ mm │ interval │ 60 │ retain │ 50

@ma-zal
Copy link
Contributor

ma-zal commented Oct 26, 2017

I found a memory leak.

You can try my commit with fix: ma-zal@0723af9 (temporary branch fix-unlink-enoent).

In newer pm2, you can install it by:

pm2 install ma-zal/pm2-logrotate#fix-unlink-enoent

Note: Older PM2 cannot install specific branch (Unitech/pm2#2809)

@amit777
Copy link

amit777 commented Dec 7, 2017

I'm running pm2 2.8.0 with pm2-logrotate 2.2.0. I think we are experiencing a memory leak with pm2-logrotate. Would our version numbers be consistent with this potential memory leak?

@ipengyo
Copy link

ipengyo commented Jan 22, 2018

master branch READM.ME How the recommended version is 2.2?

@nitrocode
Copy link
Contributor

nitrocode commented Mar 17, 2018

@amit777 @ipengyo the current version is 2.4.0. The command you're using in the README is to install a specific version. If you drop the @2.2.0 from the README command, it installs the latest version. I can see how it's confusing.

You can see the latest version on the npmjs website.

@dskrvk
Copy link

dskrvk commented May 30, 2019

Happened today with pm2@3.5.0 and pm2-logrotate@2.6.0 (latest).

@Unitech
Copy link
Member

Unitech commented May 31, 2019

https://github.com/keymetrics/pm2-logrotate/blob/master/package.json#L30
pm2-logrotate is auto restarted in case of usage of more than 500mo

Please upgrade to the latest version, I've also updated the documentation to not display "2.2.0"

@Unitech Unitech closed this as completed May 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants