Skip to content

gha: clean up CI job summaries - #2578

Merged
bert-e merged 6 commits into
development/2.16from
improvement/ZENKO-5383-ci-summaries
Nov 12, 2026
Merged

bert-e merged 6 commits into
development/2.16from
improvement/ZENKO-5383-ci-summaries

Conversation

@francoisferrand

@francoisferrand francoisferrand commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

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

@bert-e

bert-e commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Hello francoisferrand,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Available options
name description privileged authored
/after_pull_request Wait for the given pull request id to be merged before continuing with the current one.
/bypass_author_approval Bypass the pull request author's approval ⭐
/bypass_build_status Bypass the build and test status ⭐
/bypass_commit_size Bypass the check on the size of the changeset TBA ⭐
/bypass_incompatible_branch Bypass the check on the source branch prefix ⭐
/bypass_jira_check Bypass the Jira issue check ⭐
/bypass_peer_approval Bypass the pull request peers' approval ⭐
/bypass_leader_approval Bypass the pull request leaders' approval ⭐
/bypass_source_branch_lineage Bypass the cross-branch contamination check ⭐
/approve Instruct Bert-E that the author has approved the pull request. ✍️
/create_pull_requests Allow the creation of integration pull requests.
/create_integration_branches Allow the creation of integration branches.
/no_octopus Prevent Wall-E from doing any octopus merge and use multiple consecutive merge instead
/unanimity Change review acceptance criteria from one reviewer at least to all reviewers
/wait Instruct Bert-E not to run until further notice.
Available commands
name description privileged
/help Print Bert-E's manual in the pull request.
/status Print Bert-E's current status in the pull request.
/clear Remove all comments from Bert-E from the history TBA
/retry Re-start a fresh build TBA
/build Re-start a fresh build TBA
/force_reset Delete integration branches & pull requests, and restart merge process from the beginning.
/reset Try to remove integration branches unless there are commits on them which do not appear on the source branch.

Status report is not available.

@SylvainSenechal

SylvainSenechal commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The repo is going private soon right ?

image

We will be able to remove the docs thing

@scality scality deleted a comment from bert-e Nov 12, 2026
@bert-e

bert-e commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Waiting for approval

The following approvals are needed before I can proceed with the merge:

  • the author

  • 2 peers

@francoisferrand
francoisferrand force-pushed the improvement/ZENKO-5383-ci-summaries branch from 0c8f091 to cf65bf7 Compare November 12, 2026 13:42
Comment thread .github/workflows/end2end.yaml Resolved
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
@francoisferrand
francoisferrand force-pushed the improvement/ZENKO-5383-ci-summaries branch from cf65bf7 to f29d8e1 Compare November 12, 2026 16:29
@francoisferrand
francoisferrand changed the base branch from development/2.16 to improvement/ZENKO-5389-dev-tag-deployments November 12, 2026 16:30
@francoisferrand

Copy link
Copy Markdown
Contributor Author

/approve

Base automatically changed from improvement/ZENKO-5389-dev-tag-deployments to development/2.16 November 12, 2026 18:43
@bert-e

bert-e commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Build failed

The build for commit did not succeed in branch improvement/ZENKO-5383-ci-summaries

The following options are set: approve

@bert-e

bert-e commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

I have successfully merged the changeset of this pull request
into targetted development branches:

  • ✔️ development/2.16

The following branches have NOT changed:

  • development/2.10
  • development/2.11
  • development/2.12
  • development/2.13
  • development/2.14
  • development/2.15
  • development/2.5
  • development/2.6
  • development/2.7
  • development/2.8
  • development/2.9

This pull request did not target the following hotfix branch(es) so they
were left untouched:

  • hotfix/2.13.5

Please check the status of the associated issue ZENKO-5383.

Goodbye francoisferrand.

The following options are set: approve

@bert-e
bert-e merged commit f29d8e1 into development/2.16 Nov 12, 2026
45 of 46 checks passed
@bert-e
bert-e deleted the improvement/ZENKO-5383-ci-summaries branch November 12, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants