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
Bug 1851492: [release-4.5] clusterconfig.go: limit the size of collected logs #128
Bug 1851492: [release-4.5] clusterconfig.go: limit the size of collected logs #128
Conversation
@vrutkovs: This pull request references Bugzilla bug 1851492, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test e2e-aws |
/test e2e-aws |
Thank you @vrutkovs |
/approve |
/bugzilla refresh Recalculating validity in case the underlying Bugzilla bug has changed. |
@openshift-bot: This pull request references Bugzilla bug 1851492, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest Please review the full test history for this PR and help us cut down flakes. |
9 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
This commit ensures clusterconfig won't collect large logs. GatherClusterOperators would use maximum available compressed tarball size (set in `diskrecorder`), multiply it by `logCompressionRatio` (set to 2 initially) and split this budget between unhealthy pods. `collectContainerLogs` would limit the amount of fetched bytes so that log lines would not exceed container log budget.
5e38d06
to
6deebfd
Compare
@vrutkovs why the force push, is this now out of sync with master? |
@sdodson comment update from https://github.com/openshift/insights-operator/pull/127/files#diff-397c7d21d587358413d334b87528a7cbL224-R224 caused merge conflict |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: martinkunc, sdodson, vrutkovs The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@vrutkovs: All pull requests linked via external trackers have merged: openshift/insights-operator#128. Bugzilla bug 1851492 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This commit ensures clusterconfig won't collect large logs.
GatherClusterOperators would use maximum available compressed tarball
size (set in
diskrecorder
), multiply it bylogCompressionRatio
(set to 2 initially) and split this budget between unhealthy pods.
collectContainerLogs
would limit the amount of fetched bytes sothat log lines would not exceed container log budget.
Cherry-pick of #126