ci: harden Tier CI result handling and dashboard polish#3373
Merged
JeanRochCoulon merged 7 commits intoJul 7, 2026
Conversation
Contributor
|
Did you pass Black and Pylint on your python files ? |
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.
Summary
This PR is a follow-up improvement to the CVA6 Tier CI flow introduced in PR #3285. It focuses on hardening Tier CI result handling and polishing a few dashboard presentation details.
The main changes are:
Add a shared Tier regression script for result handling:
ci-results/run.log,ci-results/failure_summary.log, andci-results/exit_codeif: always()upload steps so result artifacts are still uploaded after failuresSimplify the Tier 1 / Tier 2 workflow YAML files:
.github/scripts/run-tier-regression.shPolish the Tier CI dashboard:
Not coveredtoNot in matrixMotivation
The initial Tier CI flow worked, but the result handling logic was still embedded directly in the workflow YAML files and was duplicated between Tier 1 and Tier 2.
This PR makes the result handling more explicit, more robust, and easier to maintain. It also prepares the Tier CI flow for future extensions, since future Tier 1 / Tier 2 changes can reuse the same runner logic instead of duplicating shell code in multiple workflow jobs.
Validation
I validated the updated branch on my fork:
https://alexchenic.github.io/cva6/
Validation notes:
exit_code=0.failure_summary.logcontained reported failures.*** FAILED ***lines followed by final*** SUCCESS ***; the final ISS status check correctly treats these as passing, and no final ISS FAILED status was found.