)]}'
{".zuul.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8d3ccbfdd931dc5f61c410bdc302de1bd81b2d8d","unresolved":true,"context_lines":[{"line_number":153,"context_line":"    templates:"},{"line_number":154,"context_line":"      - check-requirements"},{"line_number":155,"context_line":"      - lib-forward-testing-python3"},{"line_number":156,"context_line":"      - openstack-python3-yoga-jobs"},{"line_number":157,"context_line":"      - publish-openstack-docs-pti"},{"line_number":158,"context_line":"      - release-notes-jobs-python3"},{"line_number":159,"context_line":"    check:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1f6da778_dbb4a2cf","line":156,"range":{"start_line":156,"start_character":26,"end_line":156,"end_character":30},"updated":"2022-07-07 22:55:03.000000000","message":"Don\u0027t know if this is what\u0027s causing the problem, but we should be running the zed template ... I think this patch needs a rebase.","commit_id":"3968fee396644f6b4f8ca6ad6f38e2519c2404ba"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8ae24d8762beba2729bb631cac4e08aeae64a210","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4dfc3bb4_abd777fa","updated":"2022-07-07 23:00:06.000000000","message":"Also, I thought you had a patch up with a groovy way to keep this backward compatible for people still using UPPER_CONSTRAINTS_FILE ?  Or did the team decide not to merge that (we seem to be using only TOX_CONSTRAINTS_FILE in glance/tox.ini)","commit_id":"3968fee396644f6b4f8ca6ad6f38e2519c2404ba"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"81e185dd261bd7ec4ee0cf45416c94b4540e90f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4a80db36_fc9730d2","updated":"2022-07-07 22:56:08.000000000","message":"My other concern is that instead of using u-c in the deps, we should do it in the install_command like in https://review.opendev.org/c/openstack/glance/+/839786","commit_id":"3968fee396644f6b4f8ca6ad6f38e2519c2404ba"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8d3ccbfdd931dc5f61c410bdc302de1bd81b2d8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1598b5f2_60d5bffd","updated":"2022-07-07 22:55:03.000000000","message":"Not sure what\u0027s happening with all the tox tests:\n\nthe playbook verifies that the u-c file is there:\nhttps://zuul.opendev.org/t/openstack/build/3f8733d186fc4f3b9cb4b98f17663f40/console#4/0/3/ubuntu-focal\n\nbut when it runs tox without tests, tox can\u0027t find the file:\nhttps://zuul.opendev.org/t/openstack/build/3f8733d186fc4f3b9cb4b98f17663f40/console#4/0/7/ubuntu-focal\n\nThis needs a rebase, though (see comment inline).","commit_id":"3968fee396644f6b4f8ca6ad6f38e2519c2404ba"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5f1598db9f0959ef5257f88e6fba98d06240242b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cec3cd22_c8a19e6f","updated":"2022-07-07 21:04:42.000000000","message":"recheck failures on all tox jobs, couldn\u0027t find the requirements file, looks spurious","commit_id":"3968fee396644f6b4f8ca6ad6f38e2519c2404ba"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"1c6ac67da9380241f3fa2647c0786f06c4ffce75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"31cef78d_0f1385f9","updated":"2022-08-05 15:03:22.000000000","message":"\u003e Also, I thought you had a patch up with a groovy way to keep this backward compatible for people still using UPPER_CONSTRAINTS_FILE ?  Or did the team decide not to merge that (we seem to be using only TOX_CONSTRAINTS_FILE in glance/tox.ini)\n\nSo we only use TOX_ in glance/tox.ini, but we use both TOX_ and UPPER_ in python-glanceclient/tox.ini. So I guess we only half-care about compatibility with older scripts? :) It\u0027s not a hill on which I\u0027m willing to die, but we could support both in glance_store if you\u0027d like.","commit_id":"ad5ba82a5a43ae57b6d0cb0c1909d82665c6f058"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"78dd690f927b236cf2e90301ee8456a539c3a38d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"93047ac1_a5c666c7","updated":"2023-02-09 14:25:05.000000000","message":"In favor, thank you!","commit_id":"6b49171e024ca3a69a77706ae7d2fe124d5aad7d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0dea8c3223cf90b62daf97d1d8218898ea28296a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e0714df8_aff052ac","updated":"2023-02-09 14:28:39.000000000","message":"This aligns us with other projects; LGTM.","commit_id":"6b49171e024ca3a69a77706ae7d2fe124d5aad7d"}]}