)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"b790cadf75a79252874b1210bc33d42a47a78ca3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"64578d24_c96f3660","updated":"2023-05-17 05:51:05.000000000","message":"changes looks good to me, but seems the requirements cannot be met:\nERROR: No matching distribution found for neutron\u003e\u003d23.0.0.0b2","commit_id":"15909921bbe75248ea9b5469c468216297b6cb96"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4ae2def5a02cfb128f48222e0155239a0cce0f02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ce3edb03_931e99dc","updated":"2023-05-17 08:44:08.000000000","message":"recheck neutron 23.0.0.0b2 released","commit_id":"15909921bbe75248ea9b5469c468216297b6cb96"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e5b012fbe7dda9c3fc72a2b716b64e45dc8843f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6a353a78_c04a1d17","in_reply_to":"64578d24_c96f3660","updated":"2023-05-17 07:16:06.000000000","message":"It will merged probably today: https://review.opendev.org/c/openstack/releases/+/883242","commit_id":"15909921bbe75248ea9b5469c468216297b6cb96"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e5d90ac43c68cbb4866ee78b3a4008b85a7c9ad8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e25a4711_7e0bebc0","updated":"2023-05-18 10:38:23.000000000","message":"recheck tempest","commit_id":"58d53676255594d0ab334876b525832e7c3116c8"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0d3cb489e00e26749e6d1ebd5136dd9a25c39918","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cc665c5c_474583ea","updated":"2023-05-31 18:52:13.000000000","message":"The neutron-tempest-plugin-bgpvpn-bagpipe job hasn\u0027t passed for a while on this patch. Do we have to disable it like in neutron-tempest-plugin jobs until we get all the patches in? Or is there just an order things need to merge?","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b5993e085abdfaf1ba957f9a47b53ecc7d3c73d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fb2bbb4a_16268d76","updated":"2023-06-02 13:40:40.000000000","message":"recheck\ntempest patch is merged, let\u0027s have fresh results, though timeout and failures looks weird","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"69734d30d214df13c6e9d25dbc49fc3c0bd7f444","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4a0ee314_9c929006","updated":"2023-06-01 10:22:15.000000000","message":"recheck depends-on patch approved","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1a4a9da9165a3f389f3efc03eaace1336864e2da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"01473c86_8f5598b2","updated":"2023-05-31 15:11:40.000000000","message":"recheck neutron-tempest-plugin-bgpvpn-bagpipe","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bf7f22066be8908554d00820ed46aafc3c77533a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c1dcb6ea_177a9d5a","updated":"2023-06-01 13:23:48.000000000","message":"recheck tempest","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"516feab777de1cdbeb9809ccd092a68815fe98db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f17fbbdf_6e1c2ef2","in_reply_to":"cc665c5c_474583ea","updated":"2023-06-01 08:16:18.000000000","message":"The whole chain seems to be ok now (see the results of https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884515)\n\nThe chain should be:\nhttps://review.opendev.org/c/openstack/networking-bagpipe/+/883064 -\u003e https://review.opendev.org/c/openstack/networking-bgpvpn/+/883057  -\u003e https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"80d160c8415aa1c880e03f0bf33d1683e6b74874","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"253cb290_a1379f95","in_reply_to":"f17fbbdf_6e1c2ef2","updated":"2023-06-01 16:16:32.000000000","message":"Well that last patch never ran the failing job","commit_id":"3d9d7aa758a07e11bb825f5e70539d29256a953a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d5df886b5c7dbd4af5feb7c5474e92dec0fc34b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9a3468b1_8b3966fd","updated":"2023-06-07 23:39:10.000000000","message":"You need to make the neutron-tempest-plugin-bgpvpn-bagpipe non-voting, then after https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 you can revert, but this has the same symptoms of the neutron-tempest-plugin repo failure.","commit_id":"491c1c76739875c0a7bc5afd5af1052fb64c5b94"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c0fd73e83c2b8c0721fa417bab5c049c0829df91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"65a1ad21_02fb8221","in_reply_to":"9a3468b1_8b3966fd","updated":"2023-06-08 08:06:06.000000000","message":"true, I hoped I can magically order the patches to have all green....","commit_id":"491c1c76739875c0a7bc5afd5af1052fb64c5b94"}],"networking_bgpvpn/neutron/services/service_drivers/bagpipe/bagpipe.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d603b5ea04e294e22682f76369bedc667900d6c9","unresolved":true,"context_lines":[{"line_number":40,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"DEVICE_OWNER_NETWORK_PROBE \u003d const.DEVICE_OWNER_NETWORK_PREFIX + \u0027probe\u0027"},{"line_number":43,"context_line":"DEVICE_OWNER_COMPUTE_PROBE \u003d const.DEVICE_OWNER_COMPUTE_PREFIX + \u0027probe\u0027"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"@log_helpers.log_method_call"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f767314_6081eadb","line":43,"updated":"2023-05-24 18:12:19.000000000","message":"Copied from my comment in the other review...\n\nI\u0027m not sure you need to worry about these anymore, right? The debug code wasn\u0027t really used for a long time, I\u0027d just remove the check below.","commit_id":"a5193cee89fbbb1580f7156919551cdb9bdc0cba"}],"networking_bgpvpn/tests/unit/services/test_plugin.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"b790cadf75a79252874b1210bc33d42a47a78ca3","unresolved":true,"context_lines":[{"line_number":884,"context_line":"                             res.status_int)"},{"line_number":885,"context_line":"            # Assert that existing bgpvpn remains"},{"line_number":886,"context_line":"            list \u003d self._list(\u0027bgpvpn/bgpvpns\u0027, fmt\u003d\u0027json\u0027)"},{"line_number":887,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027import_targets\u0027)"},{"line_number":888,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027project_id\u0027)"},{"line_number":889,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027tenant_id\u0027)"},{"line_number":890,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027route_distinguishers\u0027)"},{"line_number":891,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027route_targets\u0027)"},{"line_number":892,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027export_targets\u0027)"},{"line_number":893,"context_line":"            self.assertEqual([bgpvpn[\u0027bgpvpn\u0027]], list[\u0027bgpvpns\u0027])"},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"    @mock.patch.object(driver_api.BGPVPNDriver,"}],"source_content_type":"text/x-python","patch_set":3,"id":"618a4c0a_6ed26270","line":892,"range":{"start_line":887,"start_character":0,"end_line":892,"end_character":50},"updated":"2023-05-17 05:51:05.000000000","message":"perhaps worth to just compare the key (routers? networks?) that we are interested in?","commit_id":"15909921bbe75248ea9b5469c468216297b6cb96"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3c7676dba4b3f237266b49b489aa404e87768eae","unresolved":true,"context_lines":[{"line_number":884,"context_line":"                             res.status_int)"},{"line_number":885,"context_line":"            # Assert that existing bgpvpn remains"},{"line_number":886,"context_line":"            list \u003d self._list(\u0027bgpvpn/bgpvpns\u0027, fmt\u003d\u0027json\u0027)"},{"line_number":887,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027import_targets\u0027)"},{"line_number":888,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027project_id\u0027)"},{"line_number":889,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027tenant_id\u0027)"},{"line_number":890,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027route_distinguishers\u0027)"},{"line_number":891,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027route_targets\u0027)"},{"line_number":892,"context_line":"            bgpvpn[\u0027bgpvpn\u0027].pop(\u0027export_targets\u0027)"},{"line_number":893,"context_line":"            self.assertEqual([bgpvpn[\u0027bgpvpn\u0027]], list[\u0027bgpvpns\u0027])"},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"    @mock.patch.object(driver_api.BGPVPNDriver,"}],"source_content_type":"text/x-python","patch_set":3,"id":"000ba39d_533c8f81","line":892,"range":{"start_line":887,"start_character":0,"end_line":892,"end_character":50},"in_reply_to":"618a4c0a_6ed26270","updated":"2023-05-17 13:30:04.000000000","message":"let me check it","commit_id":"15909921bbe75248ea9b5469c468216297b6cb96"}],"requirements.txt":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e93904324b1192b4f19864fc1c59d4463ad4de7c","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# OpenStack CI will install the following projects from git"},{"line_number":19,"context_line":"# if they are in the required-projects list for a job:"},{"line_number":20,"context_line":"neutron\u003e\u003d23.0.0.0b1 # Apache-2.0"},{"line_number":21,"context_line":"networking-bagpipe\u003e\u003d12.0.0 # Apache-2.0"},{"line_number":22,"context_line":"horizon\u003e\u003d17.1.0 # Apache-2.0"},{"line_number":23,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"a17ab2bc_f299cdc6","line":20,"range":{"start_line":20,"start_character":16,"end_line":20,"end_character":19},"updated":"2023-05-16 10:17:14.000000000","message":"Please use 0b2","commit_id":"c2d1db1fb0c0cd4f58e9b6663fca00591df247fa"}]}