search

gnocchixyz / gnocchi

Timeseries database
Apache License 2.0
298 stars 84 forks source link

Add minimum requirement for sqlalchemy to 1.4.24 #1353

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

The jobs will be green when this is rebased.

tobias-urdin commented 4 months ago

@mergifyio rebase

mergify[bot] commented 4 months ago

rebase

✅ Branch has been successfully rebased