Skip to content

WPB-22961: migrate service (old bot) to PostGreSQL - #5414

Open
blackheaven wants to merge 2 commits into
developfrom
gdifolco/WPB-22961-migration-postgres-service
Open

blackheaven wants to merge 2 commits into
developfrom
gdifolco/WPB-22961-migration-postgres-service

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-22961

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven
blackheaven requested review from a team as code owners August 4, 2026 14:14
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Aug 4, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22961-migration-postgres-service branch 2 times, most recently from e42e5ae to d4a4cc4 Compare August 4, 2026 14:16
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22961-migration-postgres-service branch 2 times, most recently from a90e887 to 783f632 Compare September 11, 2026 16:13
@blackheaven blackheaven added the ready-for-review looking actively for reviewer label Sep 11, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22961-migration-postgres-service branch from 783f632 to 7f29308 Compare September 11, 2026 18:20
Migrate galley's ServiceStore (old bot provider/service connection data) from Cassandra to PostgreSQL using the dual-write + background-worker pattern: a Postgres interpreter, a DualWrite interpreter (reads from Cassandra, writes to both), and a copy-only backfill loop. Adds the service StorageLocation config field, a service table, marshall instances for Fingerprint Rsa and ServiceToken, and an integration test.
@blackheaven
blackheaven force-pushed the gdifolco/WPB-22961-migration-postgres-service branch from 7f29308 to bcca48c Compare September 24, 2026 07:24
…ce in background-worker

Without this, background-worker kept reading Cassandra after cutover, so
ExternalAccess.deliver saw newly created services as gone and deleted
their bots.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist ready-for-review looking actively for reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants