)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"2b45efc294251e54281994375d72856425d9a533","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a1410e43_1320ce58","updated":"2023-12-13 06:49:40.000000000","message":"just a small nit","commit_id":"c1d16bc75737ab6e24e7d338da4faa790faa83be"}],"ovn_bgp_agent/tests/unit/utils/test_linux_net.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"2b45efc294251e54281994375d72856425d9a533","unresolved":true,"context_lines":[{"line_number":461,"context_line":""},{"line_number":462,"context_line":"        linux_net.delete_ip_routes(routes)"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"        route0.pop(\u0027proto\u0027)"},{"line_number":465,"context_line":"        route1.pop(\u0027proto\u0027)"},{"line_number":466,"context_line":"        mock_route_delete.assert_has_calls("},{"line_number":467,"context_line":"            [mock.call(route0), mock.call(route1)])"},{"line_number":468,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9439d496_8f0bd441","line":465,"range":{"start_line":464,"start_character":1,"end_line":465,"end_character":27},"updated":"2023-12-13 06:49:40.000000000","message":"why not removing the proto from the dict instead of adding it and removing it?","commit_id":"c1d16bc75737ab6e24e7d338da4faa790faa83be"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"39a69bcb536402a00a86e35e6f1ba7659b7dd292","unresolved":true,"context_lines":[{"line_number":461,"context_line":""},{"line_number":462,"context_line":"        linux_net.delete_ip_routes(routes)"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"        route0.pop(\u0027proto\u0027)"},{"line_number":465,"context_line":"        route1.pop(\u0027proto\u0027)"},{"line_number":466,"context_line":"        mock_route_delete.assert_has_calls("},{"line_number":467,"context_line":"            [mock.call(route0), mock.call(route1)])"},{"line_number":468,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e141b9aa_585ce60b","line":465,"range":{"start_line":464,"start_character":1,"end_line":465,"end_character":27},"in_reply_to":"9439d496_8f0bd441","updated":"2023-12-13 07:07:02.000000000","message":"Because this is how this method is usually called, with the \"proto\" in the route dictionary. Then the \"delete_ip_routes\" will create its own dictionary to be passed to \"linux_net.route_delete\" that doesn\u0027t contain \"proto\".","commit_id":"c1d16bc75737ab6e24e7d338da4faa790faa83be"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"648f6121365752d516a9840ec8ed690dc2142fa0","unresolved":true,"context_lines":[{"line_number":461,"context_line":""},{"line_number":462,"context_line":"        linux_net.delete_ip_routes(routes)"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"        route0.pop(\u0027proto\u0027)"},{"line_number":465,"context_line":"        route1.pop(\u0027proto\u0027)"},{"line_number":466,"context_line":"        mock_route_delete.assert_has_calls("},{"line_number":467,"context_line":"            [mock.call(route0), mock.call(route1)])"},{"line_number":468,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9724c1ed_12e10e29","line":465,"range":{"start_line":464,"start_character":1,"end_line":465,"end_character":27},"in_reply_to":"e141b9aa_585ce60b","updated":"2023-12-13 07:44:13.000000000","message":"fair enough","commit_id":"c1d16bc75737ab6e24e7d338da4faa790faa83be"}]}