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

Improve events log format #300

Merged
merged 2 commits into from
Aug 31, 2023
Merged

Improve events log format #300

merged 2 commits into from
Aug 31, 2023

Conversation

gctucker
Copy link
Contributor

Print the events log directly on stdout and add titles for the columns to improve readability.

@gctucker
src/notifier.py: log directly on stdout
2ed4ea0 
Send the events log output directly on stdout to improve readability.
These are not really logging messages from the service but monitoring
for the API events.

Signed-off-by: Guillaume Tucker <[email protected]>
@gctucker
src/notifier.py: add log format and titles
8158d81 
Add a line with the events log format and titles for the different
columns.  This helps understanding what is being logged.

Signed-off-by: Guillaume Tucker <[email protected]>
@gctucker gctucker mentioned this pull request Aug 30, 2023
Merged
@gctucker
Copy link
Contributor Author

Tested OK on staging, via Dozzle:

$ zcat kernelci-pipeline-monitor-2023-08-30T07-47-13.log.gz | cut -d\  -f 2-
08/30/2023 07:24:31 AM UTC [INFO] Listening for events... 
08/30/2023 07:24:31 AM UTC [INFO] Press Ctrl-C to stop.
Time                        Commit        Node Id                  State      Result    Name
2023-08-30 07:26:29.344958  9ad614477bb9  64eeef2550b18f8c24a1b3df Running    -         checkout
2023-08-30 07:31:52.479021  9ad614477bb9  64eeef2550b18f8c24a1b3df Available  -         checkout
2023-08-30 07:31:52.637653  9ad614477bb9  64eef06850b18f8c24a1b3e0 Running    -         kbuild-gcc-10-x86
2023-08-30 07:31:52.739037  9ad614477bb9  64eef06850b18f8c24a1b3e1 Running    -         kunit
2023-08-30 07:31:52.749531  9ad614477bb9  64eef06850b18f8c24a1b3e2 Running    -         kver
2023-08-30 07:31:56.136140  9ad614477bb9  64eef06c50b18f8c24a1b3e3 Running    -         kunit-x86_64
2023-08-30 07:32:19.567149  9ad614477bb9  64eef06850b18f8c24a1b3e2 Done       Pass      kver
2023-08-30 07:35:14.119961  9ad614477bb9  64eef06c50b18f8c24a1b3e3 Done       Pass      kunit-x86_64
2023-08-30 07:37:54.184568  9ad614477bb9  64eef06850b18f8c24a1b3e0 Done       Pass      kbuild-gcc-10-x86
2023-08-30 07:37:54.449877  9ad614477bb9  64eef1d250b18f8c24a1b527 Running    -         baseline-x86
2023-08-30 07:38:47.559100  9ad614477bb9  64eef1d250b18f8c24a1b527 Done       Pass      baseline-x86
2023-08-30 07:42:31.928746  9ad614477bb9  64eeef2550b18f8c24a1b3df Closing    -         checkout

@gctucker gctucker added this pull request to the merge queue Aug 31, 2023
Merged via the queue into kernelci:main with commit c6fa5ed Aug 31, 2023
3 checks passed
@gctucker gctucker deleted the notifier-format branch August 31, 2023 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gctucker