Skip to content

ci: re-enable the nightly PreflightCheckRollback platform check - #38711

Merged
ggevay merged 1 commit into
MaterializeInc:mainfrom
ggevay:gabor/db-85-unskip-preflight-rollback
Sep 9, 2026
Merged

ci: re-enable the nightly PreflightCheckRollback platform check#38711
ggevay merged 1 commit into
MaterializeInc:mainfrom
ggevay:gabor/db-85-unskip-preflight-rollback

Conversation

@ggevay

@ggevay ggevay commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The nightly job checks-preflight-check-rollback has been skip:-ped since 2026-02-23 (#35166) because of DB-85, where the 0dt preflight timed out waiting for the new environmentd to reach ReadyToPromote.

Evidence that the mechanism is fixed:

  • compute: fix output frontier for read-only collections #35634 (2026-03-26) stops the output frontier of read-only collections from being held back by the write frontier, which is the value controller hydration compares against.
  • The sibling release-qualification scenario PreflightCheckContinue, which shares the scenario prefix and kept running, went from about 7 hits in 113 builds (February to April 2026) to 0 in 176 builds since 2026-04-12.

This PR only removes the skip: line so the rollback path gets the same confirmation on nightlies. DB-85 closes after a few green runs; if the timeout comes back, the job goes back to skipped and DB-85 gets the new build linked.

Part of: DB-85

🤖 Generated with Claude Code

The job was skipped on 2026-02-23 (MaterializeInc#35166) because of DB-85, where the
0dt preflight timed out waiting for the new environmentd to reach
ReadyToPromote. MaterializeInc#35634 (2026-03-26) fixed the frontier that controller
hydration compares against, and the sibling release-qualification
scenario PreflightCheckContinue has been clean for 176 builds since
2026-04-12. Re-enabling the nightly job confirms that on the rollback
path; DB-85 closes after a few green runs.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@ggevay
ggevay marked this pull request as ready for review September 8, 2026 10:10
@ggevay
ggevay requested a review from a team as a code owner September 8, 2026 10:10
@ggevay ggevay added the T-testing Theme: tests or test infrastructure label Sep 8, 2026
@ggevay
ggevay merged commit 78ab53a into MaterializeInc:main Sep 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-testing Theme: tests or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants