)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f2415c0eb2f8ef400418a7964befccc1c3be5461","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a634361_a4de3d31","updated":"2024-06-21 06:20:50.000000000","message":"As I mentioned earlier, the u-c bump has already been merged, so in its current form, this patch is a no-op.","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"cca13439e296f911349a8655785b520211065208","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b879d6c2_afb818f8","updated":"2024-06-20 10:30:33.000000000","message":"I think as this addresses a CVE we should land this.  We can for sure remove it from global-requirements after we\u0027ve also removed the reference in glance.  And ideally Kolla also.","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"c999c4045a2a24956b26f56ea0b48695934ccb44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"25f0ff2b_354717d0","updated":"2024-06-24 07:45:36.000000000","message":"I\u0027m in favor removing sqlparse altogether once we can get ride of these references:\ntony@thor:~/src/opendev.org/openstack$ beagle --server https://codesearch.openstack.org search -f grep sqlparse | grep ^openstack | grep -v \u0027/requirements\u0027\nopenstack/glance:requirements.txt:10:sqlparse\u003e\u003d0.2.2 # BSD\nopenstack/kolla:docker/openstack-base/Dockerfile.j2:175:\u0027sqlparse\u0027,\nopenstack/rpm-packaging:openstack/glance/glance.spec.j2:54:BuildRequires:  {{ py3(\u0027sqlparse\u0027) }}\nopenstack/rpm-packaging:openstack/glance/glance.spec.j2:134:Requires:       {{ py3(\u0027sqlparse\u0027) }}\nopenstack/rpm-packaging:openstack/keystone/keystone.spec.j2:47:BuildRequires:  {{ py3(\u0027sqlparse\u0027) }}","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":25277,"name":"Mariusz Karpiarz","email":"mariusz.karpiarz@codethink.co.uk","username":"mkarpiarz"},"change_message_id":"6f4a5660b1af2c422bcd21b1c413d49134b811de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8907123d_745efebd","updated":"2024-06-25 01:07:07.000000000","message":"This change has been abandoned in favour of https://review.opendev.org/c/openstack/requirements/+/922587.","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"ddbc7d6448f441478d9bc45c34f39e88751fb5f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a43d4631_565569a7","updated":"2024-05-28 06:44:45.000000000","message":"While it is o.k. to bump this version, I\u0027m not sure if it is still needed. The only reference I can see where it has been used was in sqlalchemy-migrate, which is no longer being used. So maybe we can drop it from g-r instead?","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"217de97cad5338baea41451473d43e59a7fbb436","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1be2129d_d0463fe8","updated":"2024-06-10 10:44:59.000000000","message":"seems the bump was already done as part of the automated update I8b1654e58c42acb0e4ade0c5ef3ba518087560b7\n\nif you want to amend this to drop sqlparse from reqs, that would be nice, otherwise this can be abandoned","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":25277,"name":"Mariusz Karpiarz","email":"mariusz.karpiarz@codethink.co.uk","username":"mkarpiarz"},"change_message_id":"96b07f450f468c77b5b9d80e5cccf6f028d112fc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba65f208_37d491a5","in_reply_to":"1be2129d_d0463fe8","updated":"2024-06-13 13:31:53.000000000","message":"Thanks, Jens. I can submit a patch which removes ``sqlparse`` altogether but should we first post a deprecation notice?","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":25277,"name":"Mariusz Karpiarz","email":"mariusz.karpiarz@codethink.co.uk","username":"mkarpiarz"},"change_message_id":"ac83ebfe8b39802b8319ae25c0d3b69fb2732b33","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9f2101d4_51fb473e","in_reply_to":"576369b8_1e9e9b0b","updated":"2024-05-31 11:34:03.000000000","message":"Thanks for the link!\n\nIt looks to me StarlingX are using this library in one of their repo but they also maitain their own upper-constraints there...","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"fe9a9677eeb7c4ce1d39e5b3f2066e73c73bb68b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"576369b8_1e9e9b0b","in_reply_to":"85891b71_23207a1a","updated":"2024-05-28 11:48:28.000000000","message":"https://codesearch.openstack.org/ allows you to search in all openstack repos at once","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"c999c4045a2a24956b26f56ea0b48695934ccb44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c81ba1b2_b37abca2","in_reply_to":"9057301b_c7a45b8b","updated":"2024-06-24 07:45:36.000000000","message":"Okay I messed that comment.","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":25277,"name":"Mariusz Karpiarz","email":"mariusz.karpiarz@codethink.co.uk","username":"mkarpiarz"},"change_message_id":"c4052b44e15848beec00dede184f3a4ef93cb455","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"85891b71_23207a1a","in_reply_to":"a43d4631_565569a7","updated":"2024-05-28 10:24:49.000000000","message":"Thank you, Jens.\nI\u0027d be all for removing this dependency altogether if others agree or we can confirm it\u0027s no longer used.\nDo we have a good method to find out which projects rely on specific requirements from this repo without grepping through all of them manually?","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"03285ac247d98f628ed2d6a65ae3373638d004da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9057301b_c7a45b8b","in_reply_to":"b879d6c2_afb818f8","updated":"2024-06-21 06:24:50.000000000","message":"Seems I missed these references when doing codesearch earlier, I\u0027ll make patches for these two projects to drop it.","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"ae1d1916f5d516eb34da2c7a3e13fbc024368837","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3b15bb7b_ecc92818","in_reply_to":"ba65f208_37d491a5","updated":"2024-06-14 04:57:37.000000000","message":"I don\u0027t think we do deprecation notices for requirements, and since the library isn\u0027t actually in use anywhere, it should be fine to just drop it directly.","commit_id":"52166543ed4b392e4d235a940874ce075df4ee04"}]}
