Skip to content

ci: drop redundant glamsterdam serial exec leg from kurtosis-assertoor#21572

Merged
taratorio merged 1 commit into
mainfrom
remove_duplicate_serial_exec_runs
Jun 2, 2026
Merged

ci: drop redundant glamsterdam serial exec leg from kurtosis-assertoor#21572
taratorio merged 1 commit into
mainfrom
remove_duplicate_serial_exec_runs

Conversation

@taratorio

Copy link
Copy Markdown
Member

Removes the glamsterdam exec_mode: serial entry from the test-kurtosis-assertoor matrix, leaving glamsterdam parallel-only.

Why

glamsterdam.io passes --experimental.bal, and executor selection is dbg.Exec3Parallel || cfg.experimentalBAL (execution/stagedsync/stage_execute.go). So the BAL flag forces parallel exec regardless of the ERIGON_EXEC3_PARALLEL=false that the serial leg sets — the serial entry never actually ran serial, it just duplicated the parallel run (doubling runner-minutes for zero added signal). Serial exec also hard-fails on Amsterdam blocks by design (execution/stagedsync/exec3_serial.go).

This matches test-hive-eest.yml, whose glamsterdam-devnet shard is already parallel-only.

Validation

  • actionlint clean on the changed workflow
  • YAML re-parses; matrix is now regular(×2), pectra(×2), glamsterdam(parallel), caplin-minimal(×2)

glamsterdam.io passes --experimental.bal, and executor selection is
`dbg.Exec3Parallel || cfg.experimentalBAL` (stage_execute.go), so the BAL
flag forces parallel exec regardless of the ERIGON_EXEC3_PARALLEL=false the
serial leg sets. The serial entry never exercised serial exec — it just
duplicated the parallel run. Matches test-hive-eest.yml, whose
glamsterdam-devnet shard is already parallel-only.
@taratorio taratorio force-pushed the remove_duplicate_serial_exec_runs branch from 4d19ad9 to 5014d4a Compare June 2, 2026 05:03
@taratorio taratorio enabled auto-merge June 2, 2026 05:04
@taratorio taratorio added this pull request to the merge queue Jun 2, 2026
Merged via the queue into main with commit ec23b03 Jun 2, 2026
89 checks passed
@taratorio taratorio deleted the remove_duplicate_serial_exec_runs branch June 2, 2026 11:44
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.

2 participants