Skip to content

fix(ci): skip acceptance test for canary -commit. tags#22951

Merged
vezenovm merged 1 commit into
merge-train/fairiesfrom
acceptance-test-failure
May 5, 2026
Merged

fix(ci): skip acceptance test for canary -commit. tags#22951
vezenovm merged 1 commit into
merge-train/fairiesfrom
acceptance-test-failure

Conversation

@nchamo

@nchamo nchamo commented May 5, 2026

Copy link
Copy Markdown
Contributor

Problem

The acceptance test workflow fires whenever CI3 succeeds on any v* branch, including development canary tags like v0.0.1-commit.<sha>. However, those canary tags are intentionally excluded from the publishing pipeline (!contains(github.ref_name, '-commit.')), so the installer probe hits the install endpoint for an artifact that was never published and fails with a 404.

Fix

Add a matching !contains(..., '-commit.') guard to the acceptance test job's if: condition so the workflow only runs for refs whose artifacts are actually published.

@nchamo nchamo requested a review from charlielye as a code owner May 5, 2026 13:51
@nchamo nchamo self-assigned this May 5, 2026
@nchamo nchamo requested a review from vezenovm May 5, 2026 13:58
@vezenovm vezenovm merged commit 86096b6 into merge-train/fairies May 5, 2026
14 checks passed
@vezenovm vezenovm deleted the acceptance-test-failure branch May 5, 2026 14:17
AztecBot pushed a commit that referenced this pull request May 5, 2026
## Problem

The acceptance test workflow fires whenever CI3 succeeds on any `v*`
branch, including development canary tags like `v0.0.1-commit.<sha>`.
However, those canary tags are intentionally excluded from the
publishing pipeline (`!contains(github.ref_name, '-commit.')`), so the
installer probe hits the install endpoint for an artifact that was never
published and fails with a 404.

## Fix

Add a matching `!contains(..., '-commit.')` guard to the acceptance test
job's `if:` condition so the workflow only runs for refs whose artifacts
are actually published.
@AztecBot

AztecBot commented May 5, 2026

Copy link
Copy Markdown
Collaborator

✅ Successfully backported to backport-to-v4-next-staging #22924.

AztecBot added a commit that referenced this pull request May 5, 2026
BEGIN_COMMIT_OVERRIDE
docs: add map and state variable docs  (#22824)
fix: e2e compat should not fail for contracts added after legacy stables
(#22900)
chore: fix kv-store browser tests hangs (#22721)
feat: kv-store sqlite backend with page level encryption (#22759)
fix: install node 22 for aztec-cli acceptance test (#22917)
feat: backport kv-store sqlite encryption (#22759) to v4-next (#22927)
fix(docs): correct llms.txt links for versioned developer docs (#22819)
feat(docs): improve discoverability of Aztec.nr API reference docs
(#22861)
feat(docs): backport improve discoverability of Aztec.nr API reference
docs (#22861) to v4-next (#22931)
feat(aztec-nr): add call_self stubs for utility functions (#22885)
docs: add map and state variable docs (backport #22824) (#22880)
refactor: `getPackageVersion` fn cleanup (#22941)
fix(ci): skip acceptance test for canary -commit. tags (#22951)
fix: closing db, correct stub side effects (#22939)
END_COMMIT_OVERRIDE
rangozd pushed a commit to rangozd/aztec-packages that referenced this pull request May 16, 2026
BEGIN_COMMIT_OVERRIDE
refactor: `getPackageVersion` fn cleanup (AztecProtocol#22941)
fix(ci): skip acceptance test for canary -commit. tags (AztecProtocol#22951)
fix: closing db, correct stub side effects (AztecProtocol#22939)
feat: wallet-sdk heartbeat (AztecProtocol#22948)
chore: pxe db schema compatibility test (AztecProtocol#22872)
feat: stamping aztec version into contract artifacts (AztecProtocol#22550)
fix: add aztecVersion to PXE schema_tests fixture (AztecProtocol#22960)
feat(pxe): deduplicate class ID verification per contract (AztecProtocol#22966)
chore: add noirfmt.toml to noir-contracts and run nargo fmt (AztecProtocol#22971)
feat(aztec-nr): Initial handshake registry contract with non interactive
handshake function (AztecProtocol#22854)
chore: merge next into merge-train/fairies (AztecProtocol#22991)
fix(aztec-up): Aztec installer does not shadow user installed binaries
on PATH (AztecProtocol#22902)
test(ci): drop e2e_kernelless_simulation from backwards-compat e2e
(AztecProtocol#23005)
feat(ci): notify #team-fairies when backwards-compat e2e fails on
nightly (AztecProtocol#23006)
chore: merge next into merge-train/fairies (AztecProtocol#23021)
fix: better DeployMethod (AztecProtocol#22985)
refactor(pxe): batch RPC calls for note and event validation (AztecProtocol#22988)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants