Skip to content

fix(test): force sync before snapshot digest check (#731) #732

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
proerror77 merged 9 commits into from
Aug 7, 2026
Merged

fix(test): force sync before snapshot digest check (#731) #732

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
1b3c647
debug: print snapshot digest values to diagnose #731
Aug 7, 2026
eb6905a
fix(test): force sync before snapshot digest check to de-flake #731
Aug 7, 2026
21ac102
ci: stability check 1 for #731 sync fix
Aug 7, 2026
f59c26d
ci: stability check 2 for #731 sync fix
Aug 7, 2026
c4fe2dd
ci: stability check 3 for #731 sync fix
Aug 7, 2026
c70023b
ci: verify sync fix stability after PR body fix
Aug 7, 2026
1c347dd
ci: retrigger lifecycle after assignee fix
Aug 7, 2026
1ac5b9d
ci: retrigger after issue 731 labels
Aug 7, 2026
dd6b5f4
fix(test): fail closed on filesystem sync before snapshot digest check
Aug 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions deployment/aliyun/test-polymarket-raw-ops-control-plane.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2665,6 +2665,12 @@ exercise_bootstrap_snapshot() (
# shellcheck source=/dev/null
source "$snapshot_legacy_contract"
snapshot_legacy "$rollback" rust_bootstrap "$ACTIVE_BINARY" "$baseline_sha" "$candidate_sha"
# The snapshot_legacy copies ACTIVE_BINARY into the rollback dir; force a real
# sync (the test's sync() override is a no-op) so the digest check does not
# read a partially-flushed file under CI cache/IO pressure (#731 flake). Fail
# closed if sync itself fails, so the checksum check never runs on a
# potentially unflushed file.
command sync >/dev/null 2>&1 || die 'filesystem sync failed before snapshot digest check'
(cd "$rollback" && sha256sum --check --strict manifest.sha256 >/dev/null)
jq -e '.control_dir_present == false' "$rollback/state.json" >/dev/null
)
Expand Down
Loading