Bug 263193

Summary: net-im/py-matrix-synapse: update to 1.56.0
Product: Ports & Packages Reporter: Ashish SHUKLA <ashish>
Component: Individual Port(s)Assignee: Ashish SHUKLA <ashish>
Status: Closed FIXED    
Severity: Affects Only Me CC: ports
Priority: --- Flags: ports: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://github.com/matrix-org/synapse/releases/tag/v1.56.0
Attachments:
Description Flags
Update to 1.56.0
none
Update to 1.56.0
none
net-im/py-matrix-synapse: Update to 1.56.0 (w/ dependency adjustments)
ports: maintainer-approval+
Update to 1.56.0 v2 ashish: maintainer-approval?

Description Ashish SHUKLA freebsd_committer freebsd_triage 2022-04-10 12:58:37 UTC
Created attachment 233119 [details]
Update to 1.56.0

- Update to 1.56.0
- Note important changes since 1.55.2 in pkg-message.in



Following is an excerpt from test suite:

tests.replication.slave.storage.test_events.SlavedEventStoreTestCase.test_redactions                                 tests.replication.slave.storage.test_receipts.SlavedReceiptTestCase.test_receipt                                     tests.replication.tcp.streams.test_account_data.AccountDataStreamTestCase.test_update_function_global_account_data_limit                                                                                                                   tests.replication.tcp.streams.test_account_data.AccountDataStreamTestCase.test_update_function_room_account_data_limit                                                                                                                     tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_backwards_stream_id                              tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_update_function_event_row_limit                  tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_update_function_huge_state_change
tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_update_function_state_row_limit
tests.replication.tcp.streams.test_federation.FederationStreamTestCase.test_catchup
tests.replication.tcp.streams.test_receipts.ReceiptsStreamTestCase.test_receipt
tests.replication.tcp.streams.test_typing.TypingStreamTestCase.test_reset
tests.replication.tcp.streams.test_typing.TypingStreamTestCase.test_typing
-------------------------------------------------------------------------------                                      Ran 2278 tests in 1216.386s

PASSED (skips=40, successes=2238)
Comment 1 Ashish SHUKLA freebsd_committer freebsd_triage 2022-04-11 21:20:20 UTC
Created attachment 233156 [details]
Update to 1.56.0

Sorry about the previous patch.
Comment 2 Sascha Biberhofer 2022-04-12 09:22:22 UTC
Patch looks good, but misses two minor dependency changes:

* We now have a dependency on jinja2>=3.0.0.
* We have a version cap on signedjson (<=1.1.1).

The latter is only necessary for typechecking during developement and thus doesn't really matter to us (see [1]). I'd suggest removing that check from synapse's python_dependencies.py checking, like we do for the artificial cryptography check. Otherwise this will bite us if someone updates signedjson beyond 1.1.1. I've also poked upstream about this and they agree. :D 

I've updated your patch with these two minor changes applied. Still passes tests, builds fine and runs fine. :)

Since this release includes changes that will cause synapse to refuse starting unless the configuration is adjusted and/or a complete (and time consuming) DB migration takes place, I think it would be worthwhile to also mention this in UPGRADING. :D

Cheers,
Sascha


[1] https://github.com/matrix-org/synapse/issues/12324
Comment 3 Sascha Biberhofer 2022-04-12 09:23:07 UTC
Created attachment 233162 [details]
net-im/py-matrix-synapse: Update to 1.56.0 (w/ dependency adjustments)
Comment 4 Ashish SHUKLA freebsd_committer freebsd_triage 2022-04-12 21:37:54 UTC
Created attachment 233177 [details]
Update to 1.56.0 v2

It includes all the changes in the previous diffs + UPDATING entry.


Thanks!
Comment 5 Sascha Biberhofer 2022-04-13 07:10:01 UTC
Looks good to me, let's get this merged. :)
Comment 6 commit-hook freebsd_committer freebsd_triage 2022-04-13 13:59:51 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=76ad2b55b71f9ba18a205d9ecdb248b78d626a0c

commit 76ad2b55b71f9ba18a205d9ecdb248b78d626a0c
Author:     Ashish SHUKLA <ashish@FreeBSD.org>
AuthorDate: 2022-04-10 11:45:30 +0000
Commit:     Ashish SHUKLA <ashish@FreeBSD.org>
CommitDate: 2022-04-13 13:58:07 +0000

    net-im/py-matrix-synapse: Update to 1.56.0

    - Mention important changes related to the upgrade to UPDATING
    - Update the dependency constraints

    Co-authored-by: Sascha Biberhofer <ports at skyforge dot at>
    PR:             263193
    Approved by:    Sascha Biberhofer <ports at skyforge dot at> (maintainer)

 UPDATING                                           | 22 ++++++++++++++++++++++
 net-im/py-matrix-synapse/Makefile                  |  4 ++--
 net-im/py-matrix-synapse/distinfo                  |  6 +++---
 .../files/patch-synapse_python__dependencies.py    | 13 +++++++++++--
 net-im/py-matrix-synapse/files/pkg-message.in      | 17 +++++++++++++++++
 5 files changed, 55 insertions(+), 7 deletions(-)
Comment 7 Ashish SHUKLA freebsd_committer freebsd_triage 2022-04-13 14:01:21 UTC
Committed, thanks!
Comment 8 commit-hook freebsd_committer freebsd_triage 2022-06-29 04:30:17 UTC
A commit in branch 2022Q2 references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=9a15ef04b4cc306bf68abfcd474eb4a1f726ccf2

commit 9a15ef04b4cc306bf68abfcd474eb4a1f726ccf2
Author:     Ashish SHUKLA <ashish@FreeBSD.org>
AuthorDate: 2022-04-10 11:45:30 +0000
Commit:     Ashish SHUKLA <ashish@FreeBSD.org>
CommitDate: 2022-06-29 04:09:54 +0000

    net-im/py-matrix-synapse: Update to 1.56.0

    - Mention important changes related to the upgrade to UPDATING
    - Update the dependency constraints

    Co-authored-by: Sascha Biberhofer <ports at skyforge dot at>
    PR:             263193
    Approved by:    Sascha Biberhofer <ports at skyforge dot at> (maintainer)

    (cherry picked from commit 76ad2b55b71f9ba18a205d9ecdb248b78d626a0c)

 net-im/py-matrix-synapse/Makefile                       |  4 ++--
 net-im/py-matrix-synapse/distinfo                       |  6 +++---
 .../files/patch-synapse_python__dependencies.py         | 13 +++++++++++--
 net-im/py-matrix-synapse/files/pkg-message.in           | 17 +++++++++++++++++
 4 files changed, 33 insertions(+), 7 deletions(-)