)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"5d82b7820df001721eeba10596be0f2cdeefca70","unresolved":true,"context_lines":[{"line_number":7,"context_line":"fix: Upgrade the version of databases"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For this commit https://review.opendev.org/c/openstack/requirements/+/900435"},{"line_number":10,"context_line":"The version of SQLAlchemy has been as 1.4.50"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"From the install_requires of databases, we need to upgrade it."},{"line_number":13,"context_line":"For 0.8.0 version of databases, sqlalchemy\u003e\u003d1.4.42,\u003c1.5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f95843b7_3f34b2ab","line":10,"updated":"2024-01-31 06:42:24.000000000","message":"Note there is another update already pending https://review.opendev.org/c/openstack/requirements/+/907285\n\nIt would also be good if you could prepare for using SQLAlchemy 2.x by adding a dedicated test job for that. See https://review.opendev.org/c/openstack/placement/+/886229/ for an example of how to do this.","commit_id":"73e4855b869207138233201d7d2ccac5a14fbf88"},{"author":{"_account_id":28706,"name":"Boxiang Zhu","email":"bxzhu_5355@163.com","username":"ZhuBoxiang"},"change_message_id":"6a207400a67ff3b8cb8d1f11ec6da25f93e6cdac","unresolved":false,"context_lines":[{"line_number":7,"context_line":"fix: Upgrade the version of databases"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For this commit https://review.opendev.org/c/openstack/requirements/+/900435"},{"line_number":10,"context_line":"The version of SQLAlchemy has been as 1.4.50"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"From the install_requires of databases, we need to upgrade it."},{"line_number":13,"context_line":"For 0.8.0 version of databases, sqlalchemy\u003e\u003d1.4.42,\u003c1.5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a03cf63_815e7069","line":10,"in_reply_to":"f95843b7_3f34b2ab","updated":"2024-01-31 09:02:24.000000000","message":"\u003e Note there is another update already pending https://review.opendev.org/c/openstack/requirements/+/907285\n\u003e \n\u003e It would also be good if you could prepare for using SQLAlchemy 2.x by adding a dedicated test job for that. See https://review.opendev.org/c/openstack/placement/+/886229/ for an example of how to do this.\n\nGot it, if openstack will use SQLAlchemy 2.x, maybe skyline needs other packages to fix this. Thanks ：）","commit_id":"73e4855b869207138233201d7d2ccac5a14fbf88"}],"requirements.txt":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"5d82b7820df001721eeba10596be0f2cdeefca70","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"pbr\u003e\u003d5.8.0 # Apache-2.0"},{"line_number":6,"context_line":"fastapi\u003c\u003d0.58.1 # MIT"}],"source_content_type":"text/plain","patch_set":1,"id":"dd6af88c_33db1162","line":3,"updated":"2024-01-31 06:42:24.000000000","message":"Side note: this comment is no longer true with latest pip, you could drop it","commit_id":"73e4855b869207138233201d7d2ccac5a14fbf88"},{"author":{"_account_id":28706,"name":"Boxiang Zhu","email":"bxzhu_5355@163.com","username":"ZhuBoxiang"},"change_message_id":"6a207400a67ff3b8cb8d1f11ec6da25f93e6cdac","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"pbr\u003e\u003d5.8.0 # Apache-2.0"},{"line_number":6,"context_line":"fastapi\u003c\u003d0.58.1 # MIT"}],"source_content_type":"text/plain","patch_set":1,"id":"8bc02386_a593e5e7","line":3,"in_reply_to":"dd6af88c_33db1162","updated":"2024-01-31 09:02:24.000000000","message":"Done","commit_id":"73e4855b869207138233201d7d2ccac5a14fbf88"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"5d82b7820df001721eeba10596be0f2cdeefca70","unresolved":true,"context_lines":[{"line_number":8,"context_line":"uvicorn\u003c\u003d0.17.6 # BSD License (3 clause)"},{"line_number":9,"context_line":"gunicorn\u003e\u003d20.1.0 # MIT"},{"line_number":10,"context_line":"python-jose\u003c\u003d3.3.0 # MIT"},{"line_number":11,"context_line":"databases\u003c\u003d0.8.0 # BSD License (3 clause)"},{"line_number":12,"context_line":"aiomysql\u003c\u003d0.1.1 # MIT"},{"line_number":13,"context_line":"aiosqlite\u003c\u003d0.17.0 # MIT"},{"line_number":14,"context_line":"loguru\u003c\u003d0.5.3 # MIT"}],"source_content_type":"text/plain","patch_set":1,"id":"41c0c723_9eaa3c67","line":11,"updated":"2024-01-31 06:42:24.000000000","message":"seems this library and all the others where you are having \"\u003c\u003d\" type constraints are not listed in global-requirements.txt. I think in order to better integrate with the whole OpenStack ecosystem, it would be better to add them there and then replace these upper constraints in your project with what you are getting from using requirements/upper-constraints.txt just like it is happening for the other dependencies already","commit_id":"73e4855b869207138233201d7d2ccac5a14fbf88"},{"author":{"_account_id":28706,"name":"Boxiang Zhu","email":"bxzhu_5355@163.com","username":"ZhuBoxiang"},"change_message_id":"6a207400a67ff3b8cb8d1f11ec6da25f93e6cdac","unresolved":false,"context_lines":[{"line_number":8,"context_line":"uvicorn\u003c\u003d0.17.6 # BSD License (3 clause)"},{"line_number":9,"context_line":"gunicorn\u003e\u003d20.1.0 # MIT"},{"line_number":10,"context_line":"python-jose\u003c\u003d3.3.0 # MIT"},{"line_number":11,"context_line":"databases\u003c\u003d0.8.0 # BSD License (3 clause)"},{"line_number":12,"context_line":"aiomysql\u003c\u003d0.1.1 # MIT"},{"line_number":13,"context_line":"aiosqlite\u003c\u003d0.17.0 # MIT"},{"line_number":14,"context_line":"loguru\u003c\u003d0.5.3 # MIT"}],"source_content_type":"text/plain","patch_set":1,"id":"a786e4b1_d1329585","line":11,"in_reply_to":"41c0c723_9eaa3c67","updated":"2024-01-31 09:02:24.000000000","message":"Acknowledged\n\nWill try to add these into requirements/upper-constraints.txt","commit_id":"73e4855b869207138233201d7d2ccac5a14fbf88"}]}
