arches / whacamole

restart heroku dynos that run out of RAM instead of swapping to disk
115 stars 12 forks source link

Add swap memory size threshold #12

Closed pomartel closed 9 years ago

pomartel commented 9 years ago

This addresses the issue #6 where the dyno swap size should also be monitored. With this pull request, you can set a swap memory size threshold and the dyno will restart if it reaches the threshold.

I have not yet tested it in production. I added some specs though.

pomartel commented 9 years ago

I am no longer sure this is as useful as I thought it would be. Trying to control the swap size might not be such a great idea. I am going to close this.

michaelbaudino commented 9 years ago

I'm actually testing this at the moment, why did you close it? It looks like a very interesting feature to have, doesn't it?

pomartel commented 9 years ago

It's not the right approach. Your swap size can increase without impacting the dyno performance. Try to find what's causing the swap memory to increase instead, that's a better way to go.