Skip to content

🐛 fix(ha): fence cross-datacenter transfers - #1859

Merged
gaborbernat merged 2 commits into
mainfrom
fix/cross-dc-1517
Aug 30, 2026
Merged

gaborbernat merged 2 commits into
mainfrom
fix/cross-dc-1517

Conversation

@gaborbernat

Copy link
Copy Markdown
Member

Cross-datacenter replication could preserve a Pending placement across restart without enough ownership evidence to decide whether its transfer was live. That could block recovery or allow competing workers to publish the same placement.

Each staged transfer now carries the consensus-term fence and a monotonic attempt token. A live term admits one copy, a newer authoritative term can reclaim abandoned work, and checkpoint or terminal transitions must present the current token. Existing placement records remain readable with a legacy attempt value.

Closes #1517

Pending placements survived a restart without a durable way to distinguish an
abandoned transfer from a live copy. That blocked recovery or risked two owners
publishing the same placement.

Bind each transfer to the consensus term and a monotonic attempt token. A newer
term can reclaim pending work, while stale checkpoints and terminal writes fail
their ownership check.
@gaborbernat gaborbernat added the bug Something isn't working label Aug 29, 2026
@gaborbernat gaborbernat self-assigned this Aug 29, 2026
@codspeed

codspeed Bot commented Aug 30, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 23 untouched benchmarks
⏩ 139 skipped benchmarks1


Comparing fix/cross-dc-1517 (b356103) with main (60698d6)

Open in CodSpeed

Footnotes

  1. 139 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@gaborbernat
gaborbernat merged commit 584a1bc into main Aug 30, 2026
32 checks passed
@gaborbernat
gaborbernat deleted the fix/cross-dc-1517 branch August 30, 2026 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requeue abandoned pending cross-DC placements

1 participant