Skip to content

WPB-29580: migrate provider passwords to PostGreSQL - #5641

Open
blackheaven wants to merge 9 commits into
developfrom
gdifolco/WPB-29580-migration-postgres-password
Open

blackheaven wants to merge 9 commits into
developfrom
gdifolco/WPB-29580-migration-postgres-password

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

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

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 October 22, 2026 17:47
@blackheaven
blackheaven force-pushed the gdifolco/WPB-29580-migration-postgres-password branch from c202f45 to ca5b807 Compare October 29, 2026 15:41
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Oct 29, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-29580-migration-postgres-password branch 3 times, most recently from 59d7a8c to 060b9c2 Compare November 10, 2026 08:52
…cstring

- insertIfNotDeletedStatement: DO UPDATE -> DO NOTHING per ADR-0005 (a replayed
  scan page must not clobber newer dual-written credentials nor resurrect a
  password whose provider row was deleted).
- ProviderPassword migration test docstring claimed a copy-only backfill with
  no dual-write interpreter; the PR ships Wire.PasswordStore.DualWrite.
@blackheaven
blackheaven force-pushed the gdifolco/WPB-29580-migration-postgres-password branch from 060b9c2 to 672e5d5 Compare November 13, 2026 11:18

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants