Skip to content

ci: preserve per-cell artifacts across failed-job reruns - #1074

Draft
kbattocchi wants to merge 1 commit into
kebatt/fix-merge-lkg-ci-conditionfrom
kebatt/artifact-attempts-repro
Draft

kbattocchi wants to merge 1 commit into
kebatt/fix-merge-lkg-ci-conditionfrom
kebatt/artifact-attempts-repro

Conversation

@kbattocchi

@kbattocchi kbattocchi commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Stacked on #1071. Set delete-merged: false in merge-artifacts so “Re-run failed jobs” can combine earlier successful cells with the newest rerun cells. Per-cell uploads retain the default retention; continue-on-error: true, merge-lkg, package publishing, and workflow triggers are unchanged. The throwaway workflow is removed; the branch contains one signed-off fix commit.

Evidence: run 39297949510, attempts 1 and 2 (--failed); attempt 2 passed. Pattern download and merge selected cell a/attempt 1 (12209657678) plus cell b/attempt 2 (12228174755), excluding old b (12207474016). By-name download selected new complete combined-kept (12228269859), not old 12213779733. The deletion control instead produced only b/attempt 2, and by-name selected new partial combined-deleted (12229165485) over old complete 12214714377. Assertions checked filenames, contents, and absence of stale files. This confirms newest-per-name selection across attempts for all three surfaces.

Local validation: YAML parsing, semantic comparison confirming only the intended boolean changes, and whitespace checks. GitHub created Run all checks 39430338009 for commit 235f3da019cb5c2e3d965737d913c87f39d4ebe1, event pull_request, associated with this stacked PR; it was queued when checked. No trigger changes were made. Coordinate with #1055's LKG redesign, which consumes requirements-* artifacts by pattern.

@kbattocchi
kbattocchi added this pull request to stack #1075 October 19, 2026 17:53
Keep merge-artifacts inputs at default retention so subsequent attempts combine successful cells from earlier attempts with the newest rerun cells. Preserve continue-on-error for genuinely absent artifact kinds; leave merge-lkg and package publishing unchanged.

Verified with throwaway run 39297949510 attempts 1 and 2: retained pattern and merge selected a1+b2; deleted inputs produced only b2. By-name downloads selected the newest merged artifact. Remove the throwaway workflow from the final change.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 373d7e7e-c5b4-4e88-9946-890209bbca88
Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
@kbattocchi
kbattocchi force-pushed the kebatt/artifact-attempts-repro branch from 60c7136 to 235f3da Compare October 20, 2026 13:25
@kbattocchi kbattocchi changed the title ci: experiment with artifacts across rerun attempts ci: preserve per-cell artifacts across failed-job reruns Oct 20, 2026

This branch has not been deployed

No deployments
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.

1 participant