Skip to content

Add postgres interpreter for UserStore#4951

Open
akshaymankar wants to merge 28 commits into
developfrom
user-pg-migration
Open

Add postgres interpreter for UserStore#4951
akshaymankar wants to merge 28 commits into
developfrom
user-pg-migration

Conversation

@akshaymankar
Copy link
Copy Markdown
Member

@akshaymankar akshaymankar commented Jan 13, 2026 •
edited
Loading

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

Checklist

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

@akshaymankar akshaymankar changed the title WIP Add postgres interpreter for UserStore Jan 13, 2026
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 13, 2026
@akshaymankar akshaymankar force-pushed the user-pg-migration branch 8 times, most recently from 7735235 to edefb6e Compare January 20, 2026 16:01
@akshaymankar akshaymankar force-pushed the user-pg-migration branch 3 times, most recently from 58edc5f to 272456f Compare January 26, 2026 14:24
@akshaymankar akshaymankar force-pushed the user-pg-migration branch 8 times, most recently from b5cd210 to 68b2d01 Compare March 2, 2026 09:13
@akshaymankar akshaymankar force-pushed the user-pg-migration branch 9 times, most recently from 68ef4d0 to b2719f5 Compare March 5, 2026 13:20
akshaymankar and others added 23 commits June 3, 2026 09:59
@akshaymankar
UserStore.Postgres: Implement the interpreter for UserStore
fb6d561
@akshaymankar
brig: Allow selecting postgres storage for user
63b0688
@akshaymankar
UserStore.Postgres: Allow getting deleted users
d731d02
@akshaymankar
UserStore.Postgres: Make sure lookupStatus returns Deleted for delete…
591bf44 
…d users
@akshaymankar
UserStore.Postgres: Makes no sense to say a deleted user is activated
8bce241 
This makes postgres behave differently from Cassandra, but this is more correct.
@akshaymankar
UserStore.Postgres: Keep track of team id of a deleted user
f28befe
@akshaymankar
UserStore.Postgres: Remove TODOs about making things work for deleted…
8555e50 
… users

These actions shouldn't work for deleted users.
@akshaymankar
Test 6 letter passwords in unit tests
b61b196
@akshaymankar
Fix queries for service user lookup
f34e2f3
@akshaymankar
tab -> spaces
135b287
@akshaymankar
docs and changelog
5626945
@akshaymankar
More fixes for queries
7c426d2
@akshaymankar
Use serializable transaction to update handle
0e97350
@akshaymankar @supersven
Typos and better words
27ce126 
Co-authored-by: Sven Tennie <sven.tennie@gmail.com>
@akshaymankar
hlint and fix import
0a3c823
@akshaymankar
background-worker.integration.yaml: Add postgresMigration settings back
a6e2737
@akshaymankar
UserStore.Postgres: Log when inconsistence is found between wire_user…
a963b15 
… and deleted_user tables
@akshaymankar
UserStore.Postgres: Reduce scope of a statement only used in one impl
c70ac53
@akshaymankar
UserStore.Postgres: select password as the correct type
3476c73
@akshaymankar
integration: Add assertions for updates to actually happen, add some …
c84c5a0 
…cases for locale update

The test are failing for Postgres due to a bug.
@akshaymankar
UserStore.Postgres: Correctly update locale
25146dc 
Override country with `NULL` when it is not provided
@akshaymankar
hlint
391c486
@akshaymankar
Add failing test to ensure reindexing deletes missed deletes
faebac5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@supersven supersven supersven left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@akshaymankar @supersven @zebot