Skip to content

fix(bounds-seeder): target clickhouse namespace + cbt-redis after raw migration#140

Merged
samcm merged 1 commit into
masterfrom
fix/bounds-seeder-clickhouse-ns
Jun 22, 2026
Merged

fix(bounds-seeder): target clickhouse namespace + cbt-redis after raw migration#140
samcm merged 1 commit into
masterfrom
fix/bounds-seeder-clickhouse-ns

Conversation

@samcm

@samcm samcm commented Jun 22, 2026

Copy link
Copy Markdown
Member

The external-bounds seeder pulled production bounds from <network>-xatu-cbt-redis-node-0 (secret <network>-xatu-cbt-redis) in the xatu namespace, but since the clickhouse-raw migration prod cbt and its Redis moved to the clickhouse namespace and dropped the xatu- prefix (<network>-cbt-redis-node-0, secret <network>-cbt-redis); the stale coordinates made SeedFromProduction fail silently (it's non-fatal), leaving local Redis empty so CBT fell back to multi-billion-row min/max scans of external tables on cold start. This points the seeder at the current location. Verified locally: lab up now seeds 58 cbt:external:* bound keys from mainnet-cbt-redis-node-0 (e.g. canonical_execution_logs min 52029 / max 25371147), so CBT reads bounds from Redis instead of scanning.

… migration

The external-bounds seeder pulled production bounds from `<network>-xatu-cbt-redis-node-0`
(secret `<network>-xatu-cbt-redis`) in the `xatu` namespace. Since the clickhouse-raw
migration, prod cbt and its Redis moved to the `clickhouse` namespace and dropped the
`xatu-` prefix (`<network>-cbt-redis-node-0`, secret `<network>-cbt-redis`). The stale
coordinates made `SeedFromProduction` fail silently (non-fatal), leaving local Redis empty
so CBT fell back to multi-billion-row min/max scans of external tables on cold start.

Verified: lab up now seeds 58 cbt:external:* bound keys into local Redis from
mainnet-cbt-redis-node-0 (e.g. canonical_execution_logs min 52029 / max 25371147).
@samcm samcm requested a review from mattevans as a code owner June 22, 2026 06:02

@mattevans mattevans left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@samcm

samcm commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

thanks @mattevans 👑

@samcm samcm merged commit 4113c5c into master Jun 22, 2026
3 checks passed
@samcm samcm deleted the fix/bounds-seeder-clickhouse-ns branch June 22, 2026 06:45
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.

2 participants