Add minimum requirement for sqlalchemy to 1.4.24

gnocchixyz / gnocchi

Timeseries database

Apache License 2.0

298 stars 84 forks source link

Add minimum requirement for sqlalchemy to 1.4.24 #1350

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.

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

LazyMan145 commented 9 months ago

Fixes #1349

tobias-urdin commented 8 months ago

This will be green when it's rebased.

rafaelweingartner commented 5 months ago

Do w need this patch, and the others that are in the series? I mean, the constraint is defining a lower bound. Therefore, the installer will normally try to take the latest one possible. Only if it is not possible, that the installer would take a smaller version. Or, if it is already installed.

Are we sure this is a change that we need?

tobias-urdin commented 4 months ago

@mergifyio rebasem

tobias-urdin commented 4 months ago

@mergifyio rebase