Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Matrix 1.71.0 #62

Merged
merged 4 commits into from
Jan 22, 2023
Merged

Update to Matrix 1.71.0 #62

merged 4 commits into from
Jan 22, 2023

Conversation

e1mo
Copy link
Member

@e1mo e1mo commented Sep 6, 2022

This has not been tested on dev yet. Due to the python3.7 requirement introduced in v1.50 an upgrade on our dev host (Ubuntu 18.04 LTS, python 3.6) requires more time than I have right now. However, prod is running on Debian 10 / python 3.7 that is fine. I would strongly prefer testing this before rolling the update out to production. However, after reading the upgrade notes I do not see any significant difficulties coming. Depending on how you see it, I think we could give directly going to production a try (Of course doing proper procedure with backups before hand and making synapse unreachable until we are assured that nothing obvious broke).

From reading the change log / upgrade notes, those changes seem as if they could affect us (but in the end don't).

  • v1.64.0
    - Deprecation of the ability to delegate e-mail verification to identity servers: Should not affect us (I think) due to our usage of SSO
  • v1.60.0
    - Possibility for failing migrations if the database is corrupted and has duplicates: We should be fine since we don't appear to have any dupes (on prod, not tested on dev):
SELECT COUNT(1) from ( SELECT state_group, prev_state_group FROM state_group_edges GROUP BY state_group, prev_state_group HAVING COUNT(ctid) > 1 ) AS dupes;
 count
-------
     0
(1 row)
  • v1.50
    - Python 3.7 and PostgresQL 10+ requirement: Fine in prod, dev runns on Python 3.6

@n0emis
Copy link
Contributor

n0emis commented Sep 6, 2022

This will break with 1.64, you'd also have to remove account_threepid_delegates from the synapse config

Copy link
Contributor

@n0emis n0emis left a comment

Choose a reason for hiding this comment

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

Please update the synapse role to n0emis/ansible-role-synapse@147d40f (see my comment above)

Co-authored-by: Ember 'n0emis' Keske <git@n0emis.eu>
@e1mo e1mo requested a review from n0emis October 16, 2022 11:57
@e1mo e1mo changed the title Update to Matrix 1.67.0 Update to Matrix 1.71.0 Jan 22, 2023
@e1mo e1mo merged commit 7e67d3b into main Jan 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants