Skip to content

Conversation

pchaigno
Copy link
Member

@pchaigno pchaigno commented Nov 4, 2024

Sysdumps collected on failures are currently not named in any special way in the ClusterMesh workflow. They end up having names like:

cilium-sysdump-20241104-095153.zip

which, among all sysdumps collected can be a bit painful to navigate.

This commit fixes that by including the name of the matrix entry in the sysdump name.

@pchaigno pchaigno added area/clustermesh Relates to multi-cluster routing functionality in Cilium. release-note/ci This PR makes changes to the CI. labels Nov 4, 2024
@pchaigno pchaigno marked this pull request as ready for review November 4, 2024 19:59
@pchaigno pchaigno requested review from a team as code owners November 4, 2024 19:59
@pchaigno pchaigno requested review from giorio94 and brlbil November 4, 2024 19:59
@pchaigno pchaigno force-pushed the pr/pchaigno/workflows-clustermesh-improve-sysdump-name branch 2 times, most recently from bb21af0 to e9ed00a Compare November 4, 2024 20:05
Sysdumps collected on failures are currently not named in any special
way in the ClusterMesh workflow. They end up having names like:

    cilium-sysdump-20241104-095153.zip

which, among all sysdumps collected can be a bit painful to navigate.

This commit fixes that by including the name of the matrix entry in the
sysdump name.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno force-pushed the pr/pchaigno/workflows-clustermesh-improve-sysdump-name branch from e9ed00a to 2ec90ea Compare November 4, 2024 22:05
@pchaigno pchaigno enabled auto-merge November 4, 2024 22:05
@pchaigno
Copy link
Member Author

pchaigno commented Nov 4, 2024

/test

@pchaigno pchaigno added this pull request to the merge queue Nov 5, 2024
Merged via the queue into main with commit f4154f9 Nov 5, 2024
78 checks passed
@pchaigno pchaigno deleted the pr/pchaigno/workflows-clustermesh-improve-sysdump-name branch November 5, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/clustermesh Relates to multi-cluster routing functionality in Cilium. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants