Skip to content

test(ci): publish lease ready marker atomically - #691

Merged
zackees merged 1 commit into
mainfrom
fix/647-atomic-lease-ready
Oct 18, 2026
Merged

zackees merged 1 commit into
mainfrom
fix/647-atomic-lease-ready

Conversation

@zackees

@zackees zackees commented Oct 18, 2026

Copy link
Copy Markdown
Owner

Closes #647. Part of #668.

Publish the embedded test child's JSON ready marker through a same-directory temporary file and os.replace. The parent still waits for marker existence, which now means the complete payload is available to parse.

Verification: ./lint, managed bosn run --task rust-test (including shared_lease_spans_exec_and_releases_after_last_server_dies), and the local gate's rust and tests lanes passed on committed tree 845f2fb4e597. Pre-push clud-review was clean.

Local-Gate: v1 tree=845f2fb4e597acad6e8c163c4c186617e4db54e0 secs=320 lanes=rust:run,tests:run
Ci-Attestation: {"at":1792299985,"gate":"rust/all/fmt","host":"linux-x86_64","key":"97550911dc57dcc03b39b671673fdff2719cc4dfdd012d02a70ab08bf88759da","lane":"rust","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":320,"stamp":"e83c81ca3b840fc45f98614915b492a2","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
Ci-Attestation: {"at":1792299985,"gate":"rust/all/boundary","host":"linux-x86_64","key":"97550911dc57dcc03b39b671673fdff2719cc4dfdd012d02a70ab08bf88759da","lane":"rust","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":320,"stamp":"7844091df0583a22c438cac60a7f750c","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
Ci-Attestation: {"at":1792299985,"gate":"rust/x86_64-unknown-linux-gnu/clippy","host":"linux-x86_64","key":"97550911dc57dcc03b39b671673fdff2719cc4dfdd012d02a70ab08bf88759da","lane":"rust","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":320,"stamp":"b34853c84dc4d4928bd90be81b971ee7","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
Ci-Attestation: {"at":1792299985,"gate":"rust/x86_64-unknown-linux-gnu/test","host":"linux-x86_64","key":"97550911dc57dcc03b39b671673fdff2719cc4dfdd012d02a70ab08bf88759da","lane":"rust","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":320,"stamp":"7cc16a6738b4e714934dfd40cc3ba63d","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
Ci-Attestation: {"at":1792299985,"gate":"python/all/static","host":"linux-x86_64","key":"96dca758f37509c14808e0812f6fca31b5d530542ed11b5e1c93b03392a9a42a","lane":"tests","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":306,"stamp":"d2e7c2167bdd85c7304f16c9cdf11234","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
Ci-Attestation: {"at":1792299985,"gate":"general/all/ci-policy","host":"linux-x86_64","key":"96dca758f37509c14808e0812f6fca31b5d530542ed11b5e1c93b03392a9a42a","lane":"tests","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":306,"stamp":"7a32e5cf51011d1866468ccd79c491a3","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
Ci-Attestation: {"at":1792299985,"gate":"python/x86_64-unknown-linux-gnu/test","host":"linux-x86_64","key":"96dca758f37509c14808e0812f6fca31b5d530542ed11b5e1c93b03392a9a42a","lane":"tests","parents":["e08bfcac1eaf3b690b5a64266e263f64068164c5"],"secs":306,"stamp":"dfa539e4bf1e35987646cb7b1d496ff7","tree":"845f2fb4e597acad6e8c163c4c186617e4db54e0","v":1,"via":"run"}
@zackees
zackees merged commit 4bf62b3 into main Oct 18, 2026
14 checks passed
@zackees
zackees deleted the fix/647-atomic-lease-ready branch October 18, 2026 05:07
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.

Flaky: legacy_lease shared_lease_spans_exec test reads the ready file before it is written

1 participant