)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"c42bb413406c2970f1016f498c89c8e3cd52edda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4271fac4_18716bb5","updated":"2023-11-06 11:07:59.000000000","message":"Still missing testing coverage, but tested it and it is already working fine","commit_id":"8199b5909fd9f3de74b0c39c1dddaa5b0e4c4ab5"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f46a828c8034971f9e18ea06cba57136f4e8709d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3371b0b0_60d8cbcb","updated":"2023-11-07 10:01:40.000000000","message":"Thanks Luis!","commit_id":"278beb7a4cba87c0ad62adf7375247f715f771f2"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"026075f11507d01ccd2d5ae0a1ea66ae390fadb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d5414d6d_ef8ea69e","updated":"2023-11-08 13:23:20.000000000","message":"Thanks Luis for addressing the comments and adding tests LGTM!","commit_id":"e37b9860681e123060d30ec0673f629d455cab51"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2ca8ee413333f9130ff1b2e0a50544f1dfd813a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"06446d7c_7ecc3cb5","updated":"2023-11-15 10:06:31.000000000","message":"I guess we can go ahead with this ?","commit_id":"0c6a71314b97e15d0aeb34c6e147a835e2c5c9e2"}],"ovn_bgp_agent/drivers/openstack/nb_ovn_bgp_driver.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"978c45b9eafac2b4f6f7725286c7f542c77bfc97","unresolved":true,"context_lines":[{"line_number":370,"context_line":"        VRF), and removes the IP of:"},{"line_number":371,"context_line":"        - VM IP on the provider network"},{"line_number":372,"context_line":"        \u0027\u0027\u0027"},{"line_number":373,"context_line":"        logical_switch \u003d ips_info.get(\u0027logical_switch\u0027)"},{"line_number":374,"context_line":"        if not logical_switch:"},{"line_number":375,"context_line":"            return"},{"line_number":376,"context_line":"        _, bridge_device, bridge_vlan \u003d self._get_ls_localnet_info("}],"source_content_type":"text/x-python","patch_set":6,"id":"aa07fd94_66d855e1","line":373,"updated":"2023-11-07 10:29:29.000000000","message":"Apparently tests are failing because the fake/mock object (FakeObject) does not have the get() method.\n\n\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c16/898198/6/check/openstack-tox-py38/c16e0ef/testr_results.html","commit_id":"278beb7a4cba87c0ad62adf7375247f715f771f2"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"21bfd10624d8dd4d0ead3b813f00ed4d6dbb9254","unresolved":false,"context_lines":[{"line_number":370,"context_line":"        VRF), and removes the IP of:"},{"line_number":371,"context_line":"        - VM IP on the provider network"},{"line_number":372,"context_line":"        \u0027\u0027\u0027"},{"line_number":373,"context_line":"        logical_switch \u003d ips_info.get(\u0027logical_switch\u0027)"},{"line_number":374,"context_line":"        if not logical_switch:"},{"line_number":375,"context_line":"            return"},{"line_number":376,"context_line":"        _, bridge_device, bridge_vlan \u003d self._get_ls_localnet_info("}],"source_content_type":"text/x-python","patch_set":6,"id":"99ce08a2_12a0a248","line":373,"in_reply_to":"aa07fd94_66d855e1","updated":"2023-11-08 12:01:07.000000000","message":"Done","commit_id":"278beb7a4cba87c0ad62adf7375247f715f771f2"}],"ovn_bgp_agent/drivers/openstack/utils/ovn.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f46a828c8034971f9e18ea06cba57136f4e8709d","unresolved":true,"context_lines":[{"line_number":430,"context_line":""},{"line_number":431,"context_line":"        for row in rows:"},{"line_number":432,"context_line":"            if not hasattr(row, \u0027status\u0027):"},{"line_number":433,"context_line":"                LOG.debug(\"OVN version does not include support for status \""},{"line_number":434,"context_line":"                          \"information. Therefore router ports and tenant IPs \""},{"line_number":435,"context_line":"                          \"cannot be exposed.\")"},{"line_number":436,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":6,"id":"41f745a9_99f7a7bc","line":433,"range":{"start_line":433,"start_character":20,"end_line":433,"end_character":25},"updated":"2023-11-07 10:01:40.000000000","message":"nit:\n\nThis probably should be a warning rather than a debug","commit_id":"278beb7a4cba87c0ad62adf7375247f715f771f2"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"21bfd10624d8dd4d0ead3b813f00ed4d6dbb9254","unresolved":false,"context_lines":[{"line_number":430,"context_line":""},{"line_number":431,"context_line":"        for row in rows:"},{"line_number":432,"context_line":"            if not hasattr(row, \u0027status\u0027):"},{"line_number":433,"context_line":"                LOG.debug(\"OVN version does not include support for status \""},{"line_number":434,"context_line":"                          \"information. Therefore router ports and tenant IPs \""},{"line_number":435,"context_line":"                          \"cannot be exposed.\")"},{"line_number":436,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":6,"id":"f182660e_72f4096f","line":433,"range":{"start_line":433,"start_character":20,"end_line":433,"end_character":25},"in_reply_to":"41f745a9_99f7a7bc","updated":"2023-11-08 12:01:07.000000000","message":"Done","commit_id":"278beb7a4cba87c0ad62adf7375247f715f771f2"}]}