search

schneems / puma_auto_tune

Puma performance without all the (T)pain
227 stars 19 forks source link

Memory usage probably wrong #6

Closed MarceloCajueiro closed 7 years ago

MarceloCajueiro commented 10 years ago

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:

correlate___librato

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 commented 10 years ago

You are correct https://github.com/schneems/get_process_mem/pull/2 on linux we should be using PSS, i'm working on updating this.

MarceloCajueiro commented 10 years ago

Thanks @schneems.

bdmac commented 10 years ago

:+1:

jeremyhaile commented 10 years ago

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

schneems commented 10 years ago

I wouldn't consider this gem safe for production just quite yet. Turns out PSS is also not correct https://github.com/schneems/get_process_mem/issues/7

jeremyhaile commented 10 years ago

OK thanks! What does unicorn worker killer use?

schneems commented 10 years ago

same thing :(

On Mon, Apr 28, 2014 at 6:04 PM, Jeremy Haile notifications@github.comwrote:

OK thanks! What does unicorn worker killer use?

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

grosser commented 8 years ago

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

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

schneems commented 8 years ago

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 commented 8 years ago

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 < notifications@github.com> 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 https://github.com/schneems/puma_auto_tune/issues/6#issuecomment-160727158 .