)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5e8375e6815dbc380965a9ecf19a036932ebbb32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"19341efa_dd935d88","updated":"2024-10-15 12:46:22.000000000","message":"The u-c takes care of what we\u0027re testing with, but requirements.txt sets the minimum version used.  Do we need to change that as well?","commit_id":"833451d66ecfdc5c16ba29c0c7fddd1a6e066270"}],"requirements.txt":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5e8375e6815dbc380965a9ecf19a036932ebbb32","unresolved":true,"context_lines":[{"line_number":44,"context_line":"Routes\u003e\u003d2.4.1 # MIT"},{"line_number":45,"context_line":"taskflow\u003e\u003d4.5.0 # Apache-2.0"},{"line_number":46,"context_line":"rtslib-fb\u003e\u003d2.1.74 # Apache-2.0"},{"line_number":47,"context_line":"SQLAlchemy\u003e\u003d1.4.23 # MIT"},{"line_number":48,"context_line":"stevedore\u003e\u003d3.2.2 # Apache-2.0"},{"line_number":49,"context_line":"tabulate\u003e\u003d0.8.7 # MIT"},{"line_number":50,"context_line":"tenacity\u003e\u003d6.3.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"ba1da111_f031bfa7","line":47,"range":{"start_line":47,"start_character":12,"end_line":47,"end_character":18},"updated":"2024-10-15 12:46:22.000000000","message":"Should we bump this to some suitable version of 2.0?","commit_id":"833451d66ecfdc5c16ba29c0c7fddd1a6e066270"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"36053838647681c552a18f763e4448af5a4eb254","unresolved":true,"context_lines":[{"line_number":44,"context_line":"Routes\u003e\u003d2.4.1 # MIT"},{"line_number":45,"context_line":"taskflow\u003e\u003d4.5.0 # Apache-2.0"},{"line_number":46,"context_line":"rtslib-fb\u003e\u003d2.1.74 # Apache-2.0"},{"line_number":47,"context_line":"SQLAlchemy\u003e\u003d1.4.23 # MIT"},{"line_number":48,"context_line":"stevedore\u003e\u003d3.2.2 # Apache-2.0"},{"line_number":49,"context_line":"tabulate\u003e\u003d0.8.7 # MIT"},{"line_number":50,"context_line":"tenacity\u003e\u003d6.3.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"5cce67b9_1968c7f9","line":47,"range":{"start_line":47,"start_character":12,"end_line":47,"end_character":18},"in_reply_to":"ba1da111_f031bfa7","updated":"2024-10-15 13:07:52.000000000","message":"I got a similar question in https://review.opendev.org/c/openstack/glance/+/929391 and answer to it there, but I don\u0027t think we need to bump the min version because removing SQLALCHEMY_WARN_20 does not remove compatibility with SQLAlchemy \u003c 2.0. It was added to catch any potential problems in SQLAlchemy 2.0 support work but does not affect actual deployments or our current job.\n\nI think the question mostly asks if we actively bump min to reflect what we test, but general guidance about min version maintenance is that avoud bump as much as possible unless we are aware that actual logic compatibility is broken.","commit_id":"833451d66ecfdc5c16ba29c0c7fddd1a6e066270"}]}