Repository navigation
gha: clean up CI job summaries - #2578
Conversation
Hello francoisferrand,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
|
Looking at the dashboard here, lots of little warning with easy fixes : https://github.com/scality/Zenko/actions/runs/43099650616 Like bumping actions versions that uses node 20 instead of node 24, or deprecaterd app-id |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
0c8f091 to
cf65bf7
Compare
The cucumber report links were meant for ctst jobs only, but the check compared junit-paths against the action's default value, which no job uses: so every end2end job got them in its summary, passed or failed. Only add them when cucumber actually produced a report, and only to the summary when the job failed, like the junit summary. They are still printed in the log, to look at a passing run if needed. Issue: ZENKO-5383
Each docker build added its own summary to its job, so a run ended up with one per image, spread across the jobs. Build jobs now upload their build records along with a short description, and a dedicated job aggregates them into a single table (status, cache usage, duration and link to the build record), in both end2end and cache-warmer runs. Issue: ZENKO-5383
Build history does not report the image size, but it keeps the manifests of the images produced by the build: sum their layers to get the size of each image, per platform. Issue: ZENKO-5383
Images built in a run share some layers (base images, kafka-connect on top of kafka...), so their sizes cannot just be added up. Add a total row to the summary, where each layer is only counted once. Issue: ZENKO-5383
Every run was littered with warnings about the app-id input of create-github-app-token (now client-id, which also accepts the app ID) and about actions still running on Node.js 20. Issue: ZENKO-5383
Multi-platform images had their sizes stacked in a single cell, next to single-platform ones, which made the table hard to scan. With a column per platform, sizes line up and can be compared with the total. Issue: ZENKO-5383
cf65bf7 to
f29d8e1
Compare
|
/approve |
Build failedThe build for commit did not succeed in branch improvement/ZENKO-5383-ci-summaries The following options are set: approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
This pull request did not target the following hotfix branch(es) so they
Please check the status of the associated issue ZENKO-5383. Goodbye francoisferrand. The following options are set: approve |

The summary page of a CI run has become cluttered: each image we build adds its own Docker build summary, and every end2end job adds links to the cucumber reports, even when it didn't run cucumber or when everything passed.
Docker builds now get a single table for the whole run, with each image's status, cache usage, build time and size, a link to its build record, and the total size of the images we ship. Cucumber report links only show up when there's a failure to look at.
Images are still listed by their build folder rather than their name, like docker build action does. It may be improved later, if/when we introduce a generic Docker build workflow.
Issue: ZENKO-5383