-
Notifications
You must be signed in to change notification settings - Fork 15
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
Monitor sidekiq process used memory #5
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very-very-very platform-dependent. I need to think about it.
We can plug the platform independent OS gem and use this method
https://github.com/rdp/os/blob/master/lib/os.rb#L139
|
@Envek do you have any thoughts here? |
I still think this feature is useful to monitor the Sidekiq process memory. Even if you use K8s, it show the actual process size inside out of the pod. |
Here is more pupular gem https://github.com/schneems/get_process_mem used as depenency in puma-worker-killer |
Still not sure whether we should add it here, to sidekiq gem. May be it is better to create one more yabeda gem just for process stats? (for the glory of god of all gems, sure!) |
Hey! In Amplifr we have some issues with sidekiq memory overlimit, so this metrics is to monitor overall memory, which is used by sidekiq process