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

[FINE][POC] Store unique set size (USS) in the PSS column #313

Merged
merged 1 commit into from
Jan 3, 2018

Conversation

jrafanie
Copy link
Member

@jrafanie jrafanie commented Nov 15, 2017

https://bugzilla.redhat.com/show_bug.cgi?id=1479356
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1526473

Unique set size is a better way to detect workers that are growing
unbounded since any memory/reference leaks would be shown in their
uss. If the server process is large when forking, new workers would
inherit a big pss immediately.

We should really rename the column/hash key to uss.

Related PRs:
master [Fixed correctly with schema changes] via ManageIQ/manageiq#16569 ManageIQ/manageiq#16570
gaprindashvili [Fixed correctly with schema changes] via ManageIQ/manageiq#16569 ManageIQ/manageiq#16570
fine
euwe

https://bugzilla.redhat.com/show_bug.cgi?id=1479356
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1526473

Unique set size is a better way to detect workers that are growing
unbounded since any memory/reference leaks would be shown in their
uss.  If the server process is large when forking, new workers would
inherit a big pss immediately.

We should really rename the column/hash key to uss.
@miq-bot
Copy link
Member

miq-bot commented Dec 15, 2017

Checked commit jrafanie@d51285c with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

Copy link
Member

@kbrock kbrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@simaishi simaishi merged commit 3d9e6e2 into ManageIQ:fine Jan 3, 2018
@simaishi simaishi added this to the Sprint 77 Ending Jan 15, 2018 milestone Jan 3, 2018
@jrafanie jrafanie deleted the fine_hack_uss_in_pss_column branch January 3, 2018 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants