Skip to content

[CI] Publish artifacts when a branch finishes#2362

Open
leo-automation wants to merge 2 commits intodevelopfrom
leo/push-objects-unfinished-jobs
Open

[CI] Publish artifacts when a branch finishes#2362
leo-automation wants to merge 2 commits intodevelopfrom
leo/push-objects-unfinished-jobs

Conversation

@leo-automation
Copy link
Copy Markdown
Contributor

@leo-automation leo-automation commented May 5, 2026

Per-arch tuning DBs are currently only published at the end of the weekly Tuning matrix, after every parallel branch (gfx908/gfx90a/gfx942/gfx950/gfx1100/gfx1201) finishes. A single slow or failing branch hides the DBs that the rest already produced.

This adds an archiveArtifacts 'build/*.tsv' in the matrix row's finally block, so each branch publishes its own DBs as soon as it finishes independent if it is success or failure

@leo-automation leo-automation requested a review from umangyadav May 5, 2026 14:06
@leo-automation leo-automation requested a review from causten as a code owner May 5, 2026 14:06
@leo-automation leo-automation changed the title CI: Publish artifacts when a branch finishes [CI] Publish artifacts when a branch finishes May 5, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Jenkins CI pipeline to publish tuning database artifacts (*.tsv) as soon as each matrix branch finishes (and best-effort on failure), so artifacts become visible without waiting for all parallel branches to complete.

Changes:

  • Archive build/*.tsv immediately after stashing in the “Stash Databases” stage.
  • Add a best-effort archiveArtifacts call in the finally block to publish partial DBs even if the branch fails/aborts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread mlir/utils/jenkins/Jenkinsfile Outdated
Comment thread mlir/utils/jenkins/Jenkinsfile Outdated
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.

3 participants