bedrock-kv / bedrock / 61a4dcd8c18adf906201889e2b8e99c534e06ac2

74%

develop: 66%

Build Type

push

github

Committed by web-flow

Commit Message

fix: make consistent-hash recovery routing deterministic (#72)

Closes #68.

## Summary

This PR fixes the fresh-cluster consistent-hash routing bug by making
empty-descriptor routing deterministic and shared across
recovery/bootstrap paths.

This PR keeps scope intentionally narrow:
- adds a shared internal `LayoutRouting` helper for deterministic log
ordering and shard-to-log selection
- uses that helper in materializer bootstrap for empty-descriptor
layouts
- uses the same helper when recovery rebuilds commit-proxy routing data
and when live routing data adds/removes logs
- rejects mixed empty/non-empty log descriptor modes so routing
semantics are not ambiguous

It does **not** include the Olivine waitlist timeout hardening or the
persisted restart recovery investigation.

## Why

Fresh-cluster recovery stores log descriptors as empty lists under the
consistent-hashing model. The old bootstrap path still treated
descriptors like shard-tag filters, so materializers could be unlocked
with the wrong log set.

Separately, recovery-time topology reconstruction and live commit-proxy
routing were rebuilding log order independently. This PR makes those
paths agree on one deterministic routing model for empty-descriptor
layouts.

## Tests

- `mix test test/bedrock/internal/layout_routing_test.exs
test/bedrock/control_plane/director/recovery/materializer_bootstrap_phase_test.exs
test/bedrock/control_plane/director/recovery/topology_phase_test.exs
test/bedrock/control_plane/director/recovery/tsl_validation_phase_test.exs
test/bedrock/data_plane/commit_proxy/routing_data_test.exs`

Run Details

43 of 43 new or added lines in 5 files covered. (100.0%)

1 existing line in 1 file now uncovered.

5213 of 7870 relevant lines covered (66.24%)

1951.96 hits per line