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
Copy link
Copy Markdown
Contributor

@nchamo nchamo commented 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.

@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
Copy link
Copy Markdown
Collaborator

AztecBot commented May 5, 2026

✅ 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
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