[v1.19] ci: e2e: improve GitHub action readability#44127
Merged
Conversation
e5b3a68 to
c19bfdb
Compare
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>
c19bfdb to
3ca11a0
Compare
Contributor
Author
|
/test |
aanm
approved these changes
Feb 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
kernelto workflow job names #44291 (@smagnani96)Once this PR is merged, a GitHub action will update the labels of these PRs: