From 6418b0379fe8a7fcd0561444b1388bd70379155a Mon Sep 17 00:00:00 2001 From: Patrick Cloke Date: Wed, 17 Jun 2020 10:01:18 -0400 Subject: [PATCH] Ignore the UI Auth sessions when porting from sqlite to postgresql (#7711) --- changelog.d/7711.bugfix | 1 + scripts/synapse_port_db | 19 ++++++++++++------- 2 files changed, 13 insertions(+), 7 deletions(-) create mode 100644 changelog.d/7711.bugfix diff --git a/changelog.d/7711.bugfix b/changelog.d/7711.bugfix new file mode 100644 index 0000000000..180de08405 --- /dev/null +++ b/changelog.d/7711.bugfix @@ -0,0 +1 @@ +The `synapse_port_db` script no longer fails when the `ui_auth_sessions` table is non-empty. This bug has existed since v1.13.0rc1. diff --git a/scripts/synapse_port_db b/scripts/synapse_port_db index a0d81c77c2..810e08beb5 100755 --- a/scripts/synapse_port_db +++ b/scripts/synapse_port_db @@ -127,6 +127,16 @@ APPEND_ONLY_TABLES = [ ] +IGNORED_TABLES = { + "user_directory", + "user_directory_search", + "users_who_share_rooms", + "users_in_pubic_room", + "ui_auth_sessions", + "ui_auth_sessions_credentials", +} + + # Error returned by the run function. Used at the top-level part of the script to # handle errors and return codes. end_error = None @@ -289,13 +299,8 @@ class Porter(object): ) return - if table in ( - "user_directory", - "user_directory_search", - "users_who_share_rooms", - "users_in_pubic_room", - ): - # We don't port these tables, as they're a faff and we can regenreate + if table in IGNORED_TABLES: + # We don't port these tables, as they're a faff and we can regenerate # them anyway. self.progress.update(table, table_size) # Mark table as done return