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.

Sign up for GitHub

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

Jump to bottom

Reduce verbosity of logs in CI runs #54

Open
pranavgaikwad opened this issue Nov 9, 2023 · 1 comment · May be fixed by #58
Open

Reduce verbosity of logs in CI runs #54

pranavgaikwad opened this issue Nov 9, 2023 · 1 comment · May be fixed by #58
Assignees
@aufi

Comments

@pranavgaikwad
Copy link
Contributor

With current logging level set to Info logs, finding failures in the logs becomes hard:

 time=2023-11-08T18:55:41Z level=info msg=[binding] |201|  POST /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |201|  POST /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |201|  POST /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |201|  POST /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |201|  POST /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |201|  POST /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |200|  GET /hub/identities
time=2023-11-08T18:55:41Z level=info msg=[binding] |204|  DELETE /hub/identities/1
time=2023-11-08T18:55:41Z level=info msg=[binding] |204|  DELETE /hub/identities/2
time=2023-11-08T18:55:41Z level=info msg=[binding] |204|  DELETE /hub/identities/3
time=2023-11-08T18:55:41Z level=info msg=[binding] |204|  DELETE /hub/identities/4
time=2023-11-08T18:55:41Z level=info msg=[binding] |204|  DELETE /hub/identities/5
time=2023-11-08T18:55:41Z level=info msg=[binding] |204|  DELETE /hub/identities/6
--- PASS: TestIdentityList (0.10s)

The logs themselves don't seem too helpful. Can we reduce the verbosity or have an option to do it?
@aufi aufi self-assigned this Nov 13, 2023
@aufi
Copy link
Member

aufi commented Nov 13, 2023

These lines are valid in API test, but I agree it is too verbose for CI runs, will reduce it.

aufi added a commit to aufi/go-konveyor-tests that referenced this issue Nov 13, 2023
@aufi
Decrease Hub API tests verbosity in CI
9967c5a 
Hub API tests use binding.RichClient which logs every API call, this
could introduce unwanted verbosity in CI test execution.

Updating API test execution script to grep failures only (with some
context) to make CI output cleaner and easier to debug. Full API test
output is still part of Hub's own API test execution.

Fixes konveyor#54

Signed-off-by: Marek Aufart <maufart@redhat.com>
@aufi aufi linked a pull request Nov 13, 2023 that will close this issue
Draft
@mguetta1 mguetta1 mentioned this issue Dec 14, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aufi aufi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Decrease Hub API tests verbosity in CI aufi/go-konveyor-tests
2 participants
@aufi @pranavgaikwad