)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"536b2bef1e74f6851d726b428de658d1aa65d924","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c3970340_b98da148","updated":"2021-12-10 23:01:04.000000000","message":"lgtm, but please rebase it","commit_id":"e425db5502e028036f6e26483f829073e05e011e"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"31d5ba18504d3526a2a6c31547dc01096ddba697","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a277e9b5_25878851","updated":"2021-12-13 12:30:26.000000000","message":"thanks, let\u0027s go","commit_id":"bb1d5d04d226da68b8277d343ce89accaea3e7fd"}],"tempest/tests/lib/services/network/test_routers_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"d9a2e0f1a93fb8d2badce266167f5061c353c20e","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            {\"subnet_id\": \"a2f1f29d-571b-4533-907f-5803ab96ead1\"},"},{"line_number":147,"context_line":"            bytes_body,"},{"line_number":148,"context_line":"            router_id\u003d\"8604a0de-7f6b-409a-a47c-a1cc7bc77b2e\","},{"line_number":149,"context_line":"            subnet_id\u003d\"a2f1f29d-571b-4533-907f-5803ab96ead1\")"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"    def _test_remove_router_interface(self, bytes_body\u003dFalse):"},{"line_number":152,"context_line":"        self.check_service_client_function("}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_81d33438","line":149,"range":{"start_line":149,"start_character":12,"end_line":149,"end_character":60},"updated":"2019-10-10 21:53:27.000000000","message":"Redundant with L146 and inconsistent with https://docs.openstack.org/api-ref/network/v2/index.html?expanded\u003dadd-interface-to-router-detail#id217","commit_id":"27c4e474d020a501232a82afddd54ff0dbcb84a8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"ffc76e3f9a0cbebf2d003134b96797a27fd9b185","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        self.check_service_client_function("},{"line_number":144,"context_line":"            self.client.add_router_interface,"},{"line_number":145,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.put\u0027,"},{"line_number":146,"context_line":"            {\"subnet_id\": \"a2f1f29d-571b-4533-907f-5803ab96ead1\"},"},{"line_number":147,"context_line":"            bytes_body,"},{"line_number":148,"context_line":"            200,"},{"line_number":149,"context_line":"            router_id\u003d\"8604a0de-7f6b-409a-a47c-a1cc7bc77b2e\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_47422c6a","line":146,"range":{"start_line":146,"start_character":12,"end_line":146,"end_character":66},"updated":"2019-10-11 02:58:00.000000000","message":"This should be the response body, not the request body, so:\n\n{\n    \"id\": \"915a14a6-867b-4af7-83d1-70efceb146f9\",\n    \"network_id\": \"91c013e2-d65a-474e-9177-c3e1799ca726\",\n    \"port_id\": \"2dc46bcc-d1f2-4077-b99e-91ee28afaff0\",\n    \"subnet_id\": \"a2f1f29d-571b-4533-907f-5803ab96ead1\",\n    \"subnet_ids\": [\n        \"a2f1f29d-571b-4533-907f-5803ab96ead1\"\n    ],\n    \"project_id\": \"0bd18306d801447bb457a46252d82d13\",\n    \"tenant_id\": \"0bd18306d801447bb457a46252d82d13\",\n    \"tags\": [\"tag1,tag2\"]\n}","commit_id":"890c984aeb3166d3ef938e713609ddf736eefc11"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"ffc76e3f9a0cbebf2d003134b96797a27fd9b185","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            {\"subnet_id\": \"a2f1f29d-571b-4533-907f-5803ab96ead1\"},"},{"line_number":147,"context_line":"            bytes_body,"},{"line_number":148,"context_line":"            200,"},{"line_number":149,"context_line":"            router_id\u003d\"8604a0de-7f6b-409a-a47c-a1cc7bc77b2e\")"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"    def _test_remove_router_interface(self, bytes_body\u003dFalse):"},{"line_number":152,"context_line":"        self.check_service_client_function("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_e779d8c4","line":149,"updated":"2019-10-11 02:58:00.000000000","message":"Sorry, I looked at this incorrectly before. Here, we need either port_id or subnet_id.\n\nAlso, here we can do router_id\u003dself.FAKE_ROUTER_ID.","commit_id":"890c984aeb3166d3ef938e713609ddf736eefc11"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"ffc76e3f9a0cbebf2d003134b96797a27fd9b185","unresolved":false,"context_lines":[{"line_number":155,"context_line":"            {\"subnet_id\": \"a2f1f29d-571b-4533-907f-5803ab96ead1\"},"},{"line_number":156,"context_line":"            bytes_body,"},{"line_number":157,"context_line":"            200,"},{"line_number":158,"context_line":"            router_id\u003d\"8604a0de-7f6b-409a-a47c-a1cc7bc77b2e\")"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    def test_list_routers_with_str_body(self):"},{"line_number":161,"context_line":"        self._test_list_routers()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_275db046","line":158,"updated":"2019-10-11 02:58:00.000000000","message":"Ditto, see above.","commit_id":"890c984aeb3166d3ef938e713609ddf736eefc11"}]}
