-
Notifications
You must be signed in to change notification settings - Fork 35
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 print all RDMA resources to logs #33
Conversation
Pull Request Test Coverage Report for Build 107
💛 - Coveralls |
9008ba4
to
e159fd7
Compare
Can you print one of the resources then their count, also check the failing ci |
hmm not sure why would CI fail on this change.
overall im LGTM on this |
Hmm, im also seeing a gzip header issue which i think comes from we encountered this in sriov-network-operator. @e0ne could you try adding to .travis ci file in
|
Thanks for the help, @adrianchiris. I'll update my PR with it |
@e0ne can you update the PR (e.g tweak commit msg) to trigger GithubActions ? once it passes we can merge this IMO |
RDMA resources output doesn't contain a lot of useful information but floods a logs with a huge amount of similar records. This patch changes logs output to print only resources count. Signed-off-by: Ivan Kolodyazhny <ikolodiazhny@nvidia.com>
@adrianchiris could you please take a look on this once more? |
RDMA resources output doesn't contain a lot of useful information
but floods a logs with thousands of similar records. This patch
changes logs output to print only resources count.
Signed-off-by: Ivan Kolodyazhny ikolodiazhny@nvidia.com