-
Notifications
You must be signed in to change notification settings - Fork 898
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
Do not store whole container env. in the reporting worker forever #14807
Do not store whole container env. in the reporting worker forever #14807
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.
👍 Looks good
This pull request is not mergeable. Please rebase and repush. |
229093f
to
0e506da
Compare
In big openshift environments do not use Class variables to store whole environemnt forever. Ugly but it plugs memory leak.
0e506da
to
0114f3d
Compare
Checked commit isimluk@0114f3d with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
@gtanzillo improved the patch a little bit and added bz. |
Do not store whole container env. in the reporting worker forever (cherry picked from commit 45ac9b6) https://bugzilla.redhat.com/show_bug.cgi?id=1444050
Fine backport details:
|
Do not store whole container env. in the reporting worker forever (cherry picked from commit 45ac9b6) https://bugzilla.redhat.com/show_bug.cgi?id=1444052
Euwe backport details:
|
In big openshift environments do not use Class variables to store whole environment forever. Plugs memory leak.
https://bugzilla.redhat.com/show_bug.cgi?id=1443606