)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f32c5decf25997e87b8072a43869af0ccc45c28d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"810099d4_e2bf5c6f","updated":"2023-06-22 10:41:07.000000000","message":"Great addition, waiting for the CI","commit_id":"51e671abd3aa220e7ad9d0bd889e0a294565d761"}],"ovn_bgp_agent/privileged/linux_net.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"0e2c85f9396372153ccc83f6bfd193effd858b04","unresolved":true,"context_lines":[{"line_number":501,"context_line":"    except agent_exc.NetworkInterfaceNotFound:"},{"line_number":502,"context_line":"        LOG.debug(\"No need to %s nei for dev %s as it does not exists\","},{"line_number":503,"context_line":"                  command, device)"},{"line_number":504,"context_line":"    except netlink_exceptions.NetlinkError:"},{"line_number":505,"context_line":"        raise"},{"line_number":506,"context_line":""},{"line_number":507,"context_line":""},{"line_number":508,"context_line":"@ovn_bgp_agent.privileged.default.entrypoint"}],"source_content_type":"text/x-python","patch_set":4,"id":"99ce8022_b2f21094","line":505,"range":{"start_line":504,"start_character":0,"end_line":505,"end_character":13},"updated":"2023-06-22 12:56:13.000000000","message":"not needed, it would be raised already. Unless you wanted to log something here ?","commit_id":"51e671abd3aa220e7ad9d0bd889e0a294565d761"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"d3a424f93f27792b6a9e6547e2cb19d52e6947cd","unresolved":false,"context_lines":[{"line_number":501,"context_line":"    except agent_exc.NetworkInterfaceNotFound:"},{"line_number":502,"context_line":"        LOG.debug(\"No need to %s nei for dev %s as it does not exists\","},{"line_number":503,"context_line":"                  command, device)"},{"line_number":504,"context_line":"    except netlink_exceptions.NetlinkError:"},{"line_number":505,"context_line":"        raise"},{"line_number":506,"context_line":""},{"line_number":507,"context_line":""},{"line_number":508,"context_line":"@ovn_bgp_agent.privileged.default.entrypoint"}],"source_content_type":"text/x-python","patch_set":4,"id":"acbbd14d_04e6f5ef","line":505,"range":{"start_line":504,"start_character":0,"end_line":505,"end_character":13},"in_reply_to":"99ce8022_b2f21094","updated":"2023-06-22 13:00:51.000000000","message":"Done","commit_id":"51e671abd3aa220e7ad9d0bd889e0a294565d761"}],"ovn_bgp_agent/tests/unit/privileged/test_linux_net.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"0e2c85f9396372153ccc83f6bfd193effd858b04","unresolved":true,"context_lines":[{"line_number":96,"context_line":"        exp \u003d FakeException()"},{"line_number":97,"context_line":"        exp.stderr \u003d \u0027No such file or directory\u0027"},{"line_number":98,"context_line":"        self.mock_exc.side_effect \u003d exp"},{"line_number":99,"context_line":"        self.assertIsNone(priv_linux_net.del_ndp_proxy(self.ipv6, self.dev))"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    @mock.patch(\u0027builtins.open\u0027, new_callable\u003dmock.mock_open())"},{"line_number":102,"context_line":"    def test_create_routing_table_for_bridge(self, mock_o):"}],"source_content_type":"text/x-python","patch_set":4,"id":"0b903e44_9716644a","line":99,"updated":"2023-06-22 12:56:13.000000000","message":"We need to find a way to get functions into the cover test. We are currently in 85% because functional tests are not part of the cover test.","commit_id":"51e671abd3aa220e7ad9d0bd889e0a294565d761"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"d3a424f93f27792b6a9e6547e2cb19d52e6947cd","unresolved":true,"context_lines":[{"line_number":96,"context_line":"        exp \u003d FakeException()"},{"line_number":97,"context_line":"        exp.stderr \u003d \u0027No such file or directory\u0027"},{"line_number":98,"context_line":"        self.mock_exc.side_effect \u003d exp"},{"line_number":99,"context_line":"        self.assertIsNone(priv_linux_net.del_ndp_proxy(self.ipv6, self.dev))"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    @mock.patch(\u0027builtins.open\u0027, new_callable\u003dmock.mock_open())"},{"line_number":102,"context_line":"    def test_create_routing_table_for_bridge(self, mock_o):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ccbc3bf_ed99a1e0","line":99,"in_reply_to":"0b903e44_9716644a","updated":"2023-06-22 13:00:51.000000000","message":"I agree, either that or skip this file from the coverage counting","commit_id":"51e671abd3aa220e7ad9d0bd889e0a294565d761"}]}