Skip to content

[v1.19] ci: e2e: improve GitHub action readability#44127

Merged
aanm merged 3 commits intov1.19from
pr/smagnani96/ci-e2e-readability-v1.19
Feb 16, 2026
Merged

[v1.19] ci: e2e: improve GitHub action readability#44127
aanm merged 3 commits intov1.19from
pr/smagnani96/ci-e2e-readability-v1.19

Conversation

@smagnani96
Copy link
Copy Markdown
Contributor

@smagnani96 smagnani96 commented Feb 2, 2026

Once this PR is merged, a GitHub action will update the labels of these PRs:

 44126 44291

@smagnani96 smagnani96 self-assigned this Feb 2, 2026
@smagnani96 smagnani96 added kind/backports This PR provides functionality previously merged into master. backport/1.19 This PR represents a backport for Cilium 1.19.x of a PR that was merged to main. labels Feb 2, 2026
@smagnani96 smagnani96 force-pushed the pr/smagnani96/ci-e2e-readability-v1.19 branch from e5b3a68 to c19bfdb Compare February 5, 2026 15:13
@smagnani96
Copy link
Copy Markdown
Contributor Author

/test

[ upstream commit 070be0c ]

This commit updates the name of the "Setup & Test" job in the
GitHub Actions workflow for e2e upgrade tests to include only the matrix
parameters "name" and "mode". This change improves the readability
of the workflow runs by providing more context about the specific
configuration being tested.

Prior to this, the name of each job contained the whole matrix combination,
which in the UI resulted to be cut off and not readable. Given that now
we use the same workflow file for running both `minor` and `patch` upgrades,
let's make the displayed name simpler.

The result will be `Setup & Test (ipsec-1, minor)`.

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
[ upstream commit e5c6372 ]

This commits adds as a first step of the `Setup & Test` job for e2e-upgrade
a simple step to dump the current matrix configuration being tested.

The previous commit, modified the title to simply display the matrix entry
name and mode (e.g., `Setup & Test (ipsec-1, minor)`) rather than the
whole configuration. In UI, that would result to be truncated anyway.

It is true that, given the matrix.name (e.g., ipsec-1), a user can open the
specific file and lookup the configuration required, but I think that
having a step where we dump it would speed up and easy debuggability in CI.

The output would be similar to:

```
> Log Matrix Configuration
Current matrix configuration:
{
  "name": "wireguard-1",
  "kernel": "5.10",
  "kube-proxy": "iptables",
  "kpr": "true",
  "devices": "{eth0,eth1}",
  "secondary-network": "true",
  "tunnel": "vxlan",
  "encryption": "wireguard",
  "encryption-node": "false",
  "lb-mode": "snat",
  "endpoint-routes": "true",
  "egress-gateway": "true",
  "ingress-controller": "true",
  "mode": "minor"
}
```

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
[ upstream commit 315441c ]

As a follow-up of changes introduced in #44126
to improve the readability of workflow job names in the GitHub UI, this
commit adds the `kernel` parameter to the name too. In OpenSearch,
it would be good to differentiate these jobs by the used kernel.
Given the kernel is a parameter that may vary frequently, it is good to be
able to differentiate runs that used the same configuration but different
kernels. That's also good in case of regressions when changing kernel.

The result will be `Setup & Test (ipsec-1, minor, 5.10)`.

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
@smagnani96 smagnani96 force-pushed the pr/smagnani96/ci-e2e-readability-v1.19 branch from c19bfdb to 3ca11a0 Compare February 13, 2026 11:13
@smagnani96
Copy link
Copy Markdown
Contributor Author

/test

@smagnani96 smagnani96 requested a review from aanm February 13, 2026 12:25
@smagnani96 smagnani96 marked this pull request as ready for review February 13, 2026 12:25
@smagnani96 smagnani96 requested review from a team as code owners February 13, 2026 12:26
@aanm aanm added this pull request to the merge queue Feb 16, 2026
Merged via the queue into v1.19 with commit fc74a67 Feb 16, 2026
81 checks passed
@aanm aanm deleted the pr/smagnani96/ci-e2e-readability-v1.19 branch February 16, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.19 This PR represents a backport for Cilium 1.19.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants