Skip to content

chore(ci): public nightlies skip scenario tests and the next tag#23984

Merged
AztecBot merged 1 commit into
nextfrom
cb/public-nightly-no-scenario-no-next-tag
Jun 11, 2026
Merged

chore(ci): public nightlies skip scenario tests and the next tag#23984
AztecBot merged 1 commit into
nextfrom
cb/public-nightly-no-scenario-no-next-tag

Conversation

@AztecBot

@AztecBot AztecBot commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

What

Two changes scoped to the public repo (AztecProtocol/aztec-packages) nightly flow, plus a follow-up tightening of the scenario-test trigger. Private tagging is unchanged.

1. Network scenario tests run only on the private v5-next nightly

ci3.yml's ci-network-scenario job fired on any current nightly tag in both repos. Private produces both a next (v6) and a v5-next (v5) nightly tag, so simply gating to the private repo still ran scenarios against the v6 nightly. The nightly-triggered path is now gated to private repo + a v5. nightly tag:

(
  needs.validate-nightly-tag.outputs.is_current == 'true'
  && github.repository == 'AztecProtocol/aztec-packages-private'
  && startsWith(github.ref_name, 'v5.')
)
|| contains(github.event.pull_request.labels.*.name, 'ci-network-scenario')

v5-next is at 5.x.x (tag v5.x.x-nightly.*) and next is at 6.x.x (tag v6.x.x-nightly.*), so startsWith(github.ref_name, 'v5.') selects the v5-next nightly only. The manual PR-label path (ci-network-scenario) is preserved for ad-hoc dev runs.

2. Stop tagging next with a nightly tag in public

nightly-release-tag.yml's matrix tagged [next, v5-next] in both repos. The branch list is now repo-dependent: private keeps [next, v5-next], public tags only v5-next (and v4-next via its existing dedicated job). Net result: public tags v4-next + v5-next only, private is untouched.

Why

Nightly network scenario tests should run only against the private v5-next nightly, and public should not produce a next nightly tag.

@AztecBot AztecBot added ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR. labels Jun 10, 2026
AztecBot added a commit that referenced this pull request Jun 10, 2026
@alexghr alexghr changed the base branch from next to merge-train/spartan June 10, 2026 10:30
@alexghr alexghr changed the base branch from merge-train/spartan to next June 11, 2026 05:49
@alexghr alexghr marked this pull request as ready for review June 11, 2026 05:50
@alexghr alexghr requested a review from charlielye as a code owner June 11, 2026 05:50
@AztecBot AztecBot force-pushed the cb/public-nightly-no-scenario-no-next-tag branch from 63540bc to be3cc49 Compare June 11, 2026 05:53
@AztecBot AztecBot enabled auto-merge June 11, 2026 05:53
## What

Two changes scoped to the **public** repo (`AztecProtocol/aztec-packages`) nightly flow, plus a follow-up tightening of the scenario-test trigger. Private tagging is unchanged.

### 1. Network scenario tests run only on the private v5-next nightly
`ci3.yml`'s `ci-network-scenario` job fired on any current nightly tag in both repos. Private produces both a `next` (v6) and a `v5-next` (v5) nightly tag, so simply gating to the private repo still ran scenarios against the v6 nightly. The nightly-triggered path is now gated to **private repo + a `v5.` nightly tag**:

```yaml
(
  needs.validate-nightly-tag.outputs.is_current == 'true'
  && github.repository == 'AztecProtocol/aztec-packages-private'
  && startsWith(github.ref_name, 'v5.')
)
|| contains(github.event.pull_request.labels.*.name, 'ci-network-scenario')
```

`v5-next` is at `5.x.x` (tag `v5.x.x-nightly.*`) and `next` is at `6.x.x` (tag `v6.x.x-nightly.*`), so `startsWith(github.ref_name, 'v5.')` selects the v5-next nightly only. The manual PR-label path (`ci-network-scenario`) is preserved for ad-hoc dev runs.

### 2. Stop tagging `next` with a nightly tag in public
`nightly-release-tag.yml`'s matrix tagged `[next, v5-next]` in both repos. The branch list is now repo-dependent: private keeps `[next, v5-next]`, public tags only `v5-next` (and `v4-next` via its existing dedicated job). Net result: **public tags `v4-next` + `v5-next` only**, private is untouched.

## Why
Nightly network scenario tests should run only against the private v5-next nightly, and public should not produce a `next` nightly tag.
@AztecBot AztecBot force-pushed the cb/public-nightly-no-scenario-no-next-tag branch from be3cc49 to 56168b3 Compare June 11, 2026 05:56
@AztecBot AztecBot added this pull request to the merge queue Jun 11, 2026
@AztecBot

Copy link
Copy Markdown
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/e9fbff36c5782f29�e9fbff36c5782f298;;�): yarn-project/kv-store/scripts/run_test.sh src/sqlite-opfs/internal/ordered-binary-browser.test.ts (1s) (code: 0)

Merged via the queue into next with commit 0f2b8e1 Jun 11, 2026
19 checks passed
@AztecBot AztecBot deleted the cb/public-nightly-no-scenario-no-next-tag branch June 11, 2026 06:40
@AztecBot

Copy link
Copy Markdown
Collaborator Author

❌ Failed to cherry-pick to v5-next due to conflicts. (🤖) View backport run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v5-next ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure ci-skip claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants