search

gnocchixyz / gnocchi

Timeseries database
Apache License 2.0
298 stars 84 forks source link

Add minimum requirement for sqlalchemy to 1.4.24 #1351

Closed LazyMan145 closed 4 months ago

LazyMan145 commented 9 months ago

Some of the recent sqlalchemy 2 changes used features that were added in 1.4.24 sqlalchemy (session.scalar) [1]. This change fixes the issue with gnocchi-upgrade built with wrong constraints.

  1. https://docs.sqlalchemy.org/en/20/changelog/changelog_14.html#change-e619b9bd2796d7e0b948631fac5b01e9

(cherry picked from commit 680ac01512935f7c7bef3e99bd42963c949d6229)

LazyMan145 commented 9 months ago

Fixes #1349

tobias-urdin commented 8 months ago

This can be closed a that code is not present in stable/4.4 series.