Memory usage probably wrong

[MarceloCajueiro]

Hey guys,

I'm using puma auto tune in an api with heroku and it looks that auto tune is seeing the wrong amount of used memory.

Here's a librato chart with the date:

The current configuration is:

PumaAutoTune.config do |config|
  config.ram = 1024 # 2x dynos
end

PumaAutoTune.start

The app is using only one thread.

Is this right?

Thanks!

[schneems]

You are correct zombocom/get_process_mem#2 on linux we should be using PSS, i'm working on updating this.

[MarceloCajueiro]

Thanks @schneems.

[bdmac]

👍

[jeremyhaile]

Should I hold off on using this gem until this is fixed? Was thinking about deploying it, but this issue worries me!

[schneems]

I wouldn't consider this gem safe for production just quite yet. Turns out PSS is also not correct zombocom/get_process_mem#7

[jeremyhaile]

OK thanks! What does unicorn worker killer use?

[schneems]

same thing :(

On Mon, Apr 28, 2014 at 6:04 PM, Jeremy Haile [email protected]:

OK thanks! What does unicorn worker killer use?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/6#issuecomment-41625090
.

[grosser]

could this help ?
heroku labs:enable log-runtime-metrics --app YOUR_APP_NAME

see https://github.com/arches/whacamole

[schneems]

whacamole is pretty cool concept, have you tried it? I'm not in love with requiring people to embed an API token in their app, stuff like that is a bit messy ATM. Maybe we should link people to that from the readme?

[grosser]

Did not try it, using restart every 2 hours now :D
http://stackoverflow.com/questions/8986259/restart-my-heroku-application-automatically/33943132#33943132

But adding it to the readme sounds like a good idea ... lot's of desperate
people coming her ...

On Mon, Nov 30, 2015 at 11:07 AM, Richard Schneeman <
[email protected]> wrote:

whacamole is pretty cool concept, have you tried it? I'm not in love with
requiring people to embed an API token in their app, stuff like that is a
bit messy ATM. Maybe we should link people to that from the readme?

—
Reply to this email directly or view it on GitHub
#6 (comment)
.