-
Notifications
You must be signed in to change notification settings - Fork 189
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
must-gather.log spams logs about egressfirewalls and egressqoses #362
Comments
The This issue will need someone like @tssurya, @npinaeva or someone from the networking team to review. The following are lines of code that generate this error; so |
I think the source is here https://github.com/openshift/oc/blob/master/pkg/cli/admin/inspect/util.go#L118-L119 |
@soltysh / @ingvagabund can you investigate here if changing the log level in inspect is correct; or if this is a collection issue (IE: a request for a missing/bad resource)? |
The warning was introduced to avoid displaying an error when a non-existing resource is inspected. It's safe to increase the log level +1 over the default one. Just, it might be impossible to propagate the log level from the initial command down to the low level binary in the chain |
@liornoy I assume all the must-gather logs can be found in any CI job? |
@ingvagabund yes, each cnf-tests CI job of ours collects must-gather logs. But the logs about "egressfirewalls" and "egressqoses" don't reappear every time. I think I encountered that in a corner case where the cluster didn't setup correctly. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
I will close this for now. |
Looking at the must-gather.log from my latest ci-test run, it seems like we throw INFO logs which make the log harder to read. See:
I file this issue to suggest muting the
the server doesn't have..
logs to provide a cleaner log output for the must-gather.The text was updated successfully, but these errors were encountered: