Eric Engestrom
60d7c3f32f
marge/pipeline_message: print details for any pipeline that hasn't succeeded
...
Not just the ones that have failed.
Again, this now includes "still running past timeout" and "blocked on a manual job".
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/39815 >
2026-02-13 14:44:30 +00:00
Eric Engestrom
95eb133c17
marge/pipeline_message: consider any job that hasn't succeded as problematic
...
That includes jobs that are still running past the timeout, and jobs
that are waiting on a manual action.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/39815 >
2026-02-13 14:44:30 +00:00
Eric Engestrom
c4afecfb28
marge/pipeline_message: print job status for jobs still running or waiting for manual action
...
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/39815 >
2026-02-13 14:44:30 +00:00
Deborah Brouwer
3c441191d9
ci: pipeline_message: reset empty errors
...
Currently marge will ignore an error message if it is just the word
"error" without any further information. However, if she never finds a
more informative message, then she will just print that meaningless
error message.
Instead of an empty error message, just leave the message blank.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:10 +00:00
Deborah Brouwer
011a867fbe
ci: pipeline_message: parse fatal messages
...
Currently marge only parses a failed job log for error messages, which can
cause her to miss fatal messages.
Example:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32557#note_2696196
https://gitlab.freedesktop.org/mesa/mesa/-/pipelines/1327177
If a `fatal` message is in the error log, add it to the pipeline message.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:09 +00:00
Deborah Brouwer
3bcb29cfcc
ci: pipeline_message: ignore generated errors
...
Currently marge does not ignore the word `generated` when she is searching
the logs for a relevant error message. So, marge's comments on a failed
pipeline say something unhelpful like "errors generated"
Example:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32687#note_2706792
https://gitlab.freedesktop.org/mesa/mesa/-/pipelines/1332129
Ignore the word `generated` so marge will provide more useful comments.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:09 +00:00
Deborah Brouwer
6dc094decd
ci: pipeline_message: ignore harmless build logs
...
Currently marge gets confused when parsing a build log and mistakes
innocuous lines that have the word "error" in them as actual issues.
Example:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/31093#note_2705442
https://gitlab.freedesktop.org/mesa/mesa/-/pipelines/1331453
Ignore lines with the word `error` in them that are not actual issues,
so that marge can provide more useful comments.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:09 +00:00
Deborah Brouwer
2a477f7df2
ci: pipeline_message: ignore error_type errors
...
Currently marge only ignores `error_type` when the type is None.
But other error_types are equally uninteresting e.g. `error_type: Jobs`.
Example:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32677#note_2711470
https://gitlab.freedesktop.org/mesa/mesa/-/pipelines/1333563
Ignore the phrase `error_type` so marge will provide more useful comments.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:09 +00:00
Deborah Brouwer
4039043f72
ci: pipeline_message: improve job list formatting
...
If marge can't find an error in a failed-job log, the formatting of her
pipeline message has extra spaces between the jobs.
Example:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32577#note_2699071
https://gitlab.freedesktop.org/mesa/mesa/-/pipelines/1328702
Add a line break after jobs even if marge can't find an error message,
so that the job list is easier to read.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:09 +00:00
Deborah Brouwer
819f9f8425
ci: pipeline_message: catch module loading errors
...
Currently, if the pipeline_message script runs in an an env missing the
aiohttp package, the pipeline_message script will throw a
ModuleNotFoundError and crash marge.
Make sure to catch and ignore these errors since the pipeline message
should never interfere with a merge request.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32773 >
2025-01-07 05:08:09 +00:00
Deborah Brouwer
caa6ccd7d6
ci: move pipeline_summary tool to .marge/hooks
...
Move the tool to summarize a failed pipeline to a generic .marge/hooks
directory. This will allow the fdo-bots repo to handle all marge hooks in
a consistent way across repositories that use this service.
Add a symlink to the bin/ci directory so that the pipeline summary tool
can still be run locally as well.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32413 >
2024-12-02 19:22:59 -08:00
