)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"b5558d44ef68a72657d8d3aac04e46ca8faed2a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"06fad123_055a876e","updated":"2022-08-22 06:54:35.000000000","message":"Thank you for working on this. ","commit_id":"badf490a5fdf276ed9cae869348dfeedc2985caf"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"49396195fe2d9ca88c715758927d3369aaae8c35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a2f16fc9_16e394f4","updated":"2022-08-29 08:37:34.000000000","message":"ah, I see the dependent patch is -2\u0027d","commit_id":"badf490a5fdf276ed9cae869348dfeedc2985caf"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"36fc2939acdd3fdd61833579e7b47a3b425cd6b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"96c35f5a_ad52ec44","updated":"2022-08-29 08:36:35.000000000","message":"recheck zuul error","commit_id":"badf490a5fdf276ed9cae869348dfeedc2985caf"}],"requirements.txt":[{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"d42f216e674c9f98051d12bc65512ef6800eded0","unresolved":true,"context_lines":[{"line_number":39,"context_line":"python-octaviaclient\u003e\u003d1.8.0"},{"line_number":40,"context_line":"python-dateutil  # BSD"},{"line_number":41,"context_line":"python-heatclient\u003e\u003d1.17.0"},{"line_number":42,"context_line":"setuptools\u003d\u003d\u003d60.9.3;python_version\u003d\u003d\u00273.9\u0027"}],"source_content_type":"text/plain","patch_set":2,"id":"27ff0500_bc414833","side":"PARENT","line":42,"updated":"2022-08-30 17:22:57.000000000","message":"You can\u0027t add an upper constraint like this (as noted in the commit message), but it might be permissable to add setuptools\u003c\u003d60.9.3;python_version\u003d\u003d\u00273.9\u0027.\nYou will need to check with someone from the requirements team though.","commit_id":"0564e94c50f327a36ab686c6a96dd653fe4eceb4"},{"author":{"_account_id":14288,"name":"Matthew Thode","display_name":"prometheanfire","email":"mthode@mthode.org","username":"prometheanfire"},"change_message_id":"aaca0e65a99163c8ec8ed1faff2f65964f58c731","unresolved":true,"context_lines":[{"line_number":39,"context_line":"python-octaviaclient\u003e\u003d1.8.0"},{"line_number":40,"context_line":"python-dateutil  # BSD"},{"line_number":41,"context_line":"python-heatclient\u003e\u003d1.17.0"},{"line_number":42,"context_line":"setuptools\u003d\u003d\u003d60.9.3;python_version\u003d\u003d\u00273.9\u0027"}],"source_content_type":"text/plain","patch_set":2,"id":"7781ea45_338c2da8","side":"PARENT","line":42,"in_reply_to":"27ff0500_bc414833","updated":"2022-08-31 15:06:21.000000000","message":"yes, a cap can be set, but I\u0027d suggest that capping setuptools not be done as it often interferes and conflicts with the pre-installation environment (what an OS or venv provides before installation).\n\nAlso, aodh should have a requirements-check job like other projects to catch things like this.\n\nhttps://docs.openstack.org/project-team-guide/dependency-management.html#enforcement-in-projects","commit_id":"0564e94c50f327a36ab686c6a96dd653fe4eceb4"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"1269fb06102ab03ed8317bb36ec4d2a1d02d2491","unresolved":true,"context_lines":[{"line_number":39,"context_line":"python-octaviaclient\u003e\u003d1.8.0"},{"line_number":40,"context_line":"python-dateutil  # BSD"},{"line_number":41,"context_line":"python-heatclient\u003e\u003d1.17.0"},{"line_number":42,"context_line":"setuptools\u003d\u003d\u003d60.9.3;python_version\u003d\u003d\u00273.9\u0027"}],"source_content_type":"text/plain","patch_set":2,"id":"8fd4136c_37a52a62","side":"PARENT","line":42,"in_reply_to":"7781ea45_338c2da8","updated":"2022-08-31 15:33:04.000000000","message":"At very least patch should be updated to drop Depends-On...\n\nI will also test out approach suggested by Emma.","commit_id":"0564e94c50f327a36ab686c6a96dd653fe4eceb4"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"b202ac1129b50c399ec100d73d41a481aec1359a","unresolved":true,"context_lines":[{"line_number":39,"context_line":"python-octaviaclient\u003e\u003d1.8.0"},{"line_number":40,"context_line":"python-dateutil  # BSD"},{"line_number":41,"context_line":"python-heatclient\u003e\u003d1.17.0"},{"line_number":42,"context_line":"setuptools\u003d\u003d\u003d60.9.3;python_version\u003d\u003d\u00273.9\u0027"}],"source_content_type":"text/plain","patch_set":2,"id":"1a9c0368_9aec95f1","side":"PARENT","line":42,"in_reply_to":"8fd4136c_37a52a62","updated":"2022-09-01 10:47:57.000000000","message":"Yes, I can confirm that `setuptools\u003c\u003d60.9.3;python_version\u003d\u003d\u00273.9\u0027` works as expected. Though, I\u0027m not if it\u0027s needed indeed, since the reason why it was implemented at first place is not valid anymore (tox-py39 job is passing). So I\u0027d say it might be good just to revert the patch indeed.","commit_id":"0564e94c50f327a36ab686c6a96dd653fe4eceb4"}]}