)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c3f61ac8ebaec41891418681e04337f3a8e00f38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e1a75741_03f1c7ea","updated":"2022-01-17 18:40:44.000000000","message":"Jakub is right, could you please check if unit tests can be added to cover this case (https://opendev.org/openstack/neutron/src/branch/master/neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py#L1991 can be a good place to check)","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"2e2038dbecdeff4da6d25d117fa880805b3b438e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5137d081_c44c3c6b","updated":"2022-01-17 15:30:10.000000000","message":"The error should be uncovered by unittests - perhaps they are missing?","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00f6b3d41c6b6d35141b3f380c2bff8e2a3ac6da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"752a2ba6_fdfbb5d8","updated":"2022-01-17 12:23:43.000000000","message":"recheck","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c3f61ac8ebaec41891418681e04337f3a8e00f38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7a184a49_7a2a955d","in_reply_to":"5137d081_c44c3c6b","updated":"2022-01-17 18:40:44.000000000","message":"+1","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"11653b6238f210acb2743e019b6fae6699a40a22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ab8ae8fb_2193b24f","in_reply_to":"7a184a49_7a2a955d","updated":"2022-01-18 09:42:04.000000000","message":"+1","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"},{"author":{"_account_id":11928,"name":"jpic","email":"jamespic@gmail.com","username":"jpic"},"change_message_id":"363ec7c053066fdde0b70b34c35f978ade692797","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff686644_812f9e02","in_reply_to":"ab8ae8fb_2193b24f","updated":"2022-01-18 15:10:00.000000000","message":"Thanks for your review!\n\nTest added","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"},{"author":{"_account_id":31357,"name":"LEDUC Florian","display_name":"Florian LEDUC","email":"florian.leduc@socgen.com","username":"leducflorian","status":"Société Générale"},"change_message_id":"f4c0c6c7782f0f5061d8f362ecacf34035ef5ac5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cf53ea2c_fa529b70","updated":"2022-01-18 15:11:25.000000000","message":"LGTM","commit_id":"d033c9f93327b5845280bbc21e5a229555b905f4"},{"author":{"_account_id":11928,"name":"jpic","email":"jamespic@gmail.com","username":"jpic"},"change_message_id":"363ec7c053066fdde0b70b34c35f978ade692797","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"93ae51d4_2dc96050","updated":"2022-01-18 15:10:00.000000000","message":"Thanks for your review!\n\nTest added","commit_id":"d033c9f93327b5845280bbc21e5a229555b905f4"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"b671af667c7c99ce97d0fd27a4b04827877b63f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1e7574f1_e98af84b","updated":"2022-01-19 11:46:41.000000000","message":"recheck Bug 1911925","commit_id":"d033c9f93327b5845280bbc21e5a229555b905f4"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00f6b3d41c6b6d35141b3f380c2bff8e2a3ac6da","unresolved":true,"context_lines":[{"line_number":1062,"context_line":"        \u0027\u0027\u0027"},{"line_number":1063,"context_line":"        try:"},{"line_number":1064,"context_line":"            lvm \u003d vlanmanager.LocalVlanManager().pop(net_uuid)"},{"line_number":1065,"context_line":"        except vlanmanager.MappingNotFound:"},{"line_number":1066,"context_line":"            LOG.debug(\"Network %s not used on agent.\", net_uuid)"},{"line_number":1067,"context_line":"            return"},{"line_number":1068,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5ff0fa3e_ccdbca7a","line":1065,"range":{"start_line":1065,"start_character":15,"end_line":1065,"end_character":43},"updated":"2022-01-17 12:23:43.000000000","message":"Why didn\u0027t we find/see this error before? This is obvious.","commit_id":"9e1503f3a348d221ba1107b6530f80cf4095c1f6"}],"neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"51b89cf8d6bce5ed98b63e35bbc389978adca9d2","unresolved":true,"context_lines":[{"line_number":1984,"context_line":"        with mock.patch.object(self.mod_agent.LOG, \u0027debug\u0027) as log_debug_fn:"},{"line_number":1985,"context_line":"            self.agent.reclaim_local_vlan(\u0027net999\u0027)"},{"line_number":1986,"context_line":"            log_debug_fn.assert_called_once_with("},{"line_number":1987,"context_line":"                \u0027Network %s not used on agent.\u0027,"},{"line_number":1988,"context_line":"                \u0027net999\u0027,"},{"line_number":1989,"context_line":"            )"},{"line_number":1990,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"0ed77b52_0769f4b8","line":1987,"range":{"start_line":1987,"start_character":25,"end_line":1987,"end_character":27},"updated":"2022-01-18 15:46:26.000000000","message":"nit: unneeded","commit_id":"d033c9f93327b5845280bbc21e5a229555b905f4"}]}