)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"e817b0805a0325d51487334283eeedb70599bee8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b81ccc65_caf96889","updated":"2022-06-24 07:56:14.000000000","message":"lgtm, could you test the verify feature? One short test for the case when the keys are matching and one when they are not - https://opendev.org/openstack/tempest/src/commit/7559bb649e11377c8cba4e7a6129025a7d3d5d06/tempest/tests/lib/common/test_rest_client.py#L251-L281","commit_id":"97926755b73b20c359b79124c15df8c5eefc37c3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"bf09f1a717b74d34c327897dbc2d1f1bb3773497","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f82231ab_340659b5","in_reply_to":"b81ccc65_caf96889","updated":"2022-06-26 09:07:34.000000000","message":"Thanks Martin for the suggestion.\nDone!","commit_id":"97926755b73b20c359b79124c15df8c5eefc37c3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6d02e96d442a25fac62b86419054d7ebf8e56541","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6238a20c_a6280666","updated":"2022-06-27 19:58:56.000000000","message":"looks good but let\u0027s name the param in different way to avoid confusion while using it.","commit_id":"936d3b775262206904854f68b6fce4715edd7ebb"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"c8713dc682615118348c9a90632128d15b756a39","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"10898761_20f52200","updated":"2022-06-27 09:36:20.000000000","message":"thanks","commit_id":"936d3b775262206904854f68b6fce4715edd7ebb"}],"tempest/lib/common/rest_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6d02e96d442a25fac62b86419054d7ebf8e56541","unresolved":true,"context_lines":[{"line_number":492,"context_line":"            self._log_request_full(resp, req_headers, req_body,"},{"line_number":493,"context_line":"                                   resp_body, extra)"},{"line_number":494,"context_line":""},{"line_number":495,"context_line":"    def _parse_resp(self, body, verify_top_key\u003dNone):"},{"line_number":496,"context_line":"        try:"},{"line_number":497,"context_line":"            body \u003d json.loads(body)"},{"line_number":498,"context_line":"        except ValueError:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7b96f84b_0f890654","line":495,"range":{"start_line":495,"start_character":32,"end_line":495,"end_character":46},"updated":"2022-06-27 19:58:56.000000000","message":"verify_top_key sounds like a boolean to tell if we need to verify the top level key or not. As this needs to be a actual key we can name it as \u0027top_key\u0027 or \u0027top_key_to_verify\u0027","commit_id":"936d3b775262206904854f68b6fce4715edd7ebb"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"f4333d5286a29a30df41373f653cbaa6a9db0f30","unresolved":false,"context_lines":[{"line_number":492,"context_line":"            self._log_request_full(resp, req_headers, req_body,"},{"line_number":493,"context_line":"                                   resp_body, extra)"},{"line_number":494,"context_line":""},{"line_number":495,"context_line":"    def _parse_resp(self, body, verify_top_key\u003dNone):"},{"line_number":496,"context_line":"        try:"},{"line_number":497,"context_line":"            body \u003d json.loads(body)"},{"line_number":498,"context_line":"        except ValueError:"}],"source_content_type":"text/x-python","patch_set":4,"id":"e7f567e9_7a5e11f1","line":495,"range":{"start_line":495,"start_character":32,"end_line":495,"end_character":46},"in_reply_to":"7b96f84b_0f890654","updated":"2022-06-28 05:22:45.000000000","message":"Done","commit_id":"936d3b775262206904854f68b6fce4715edd7ebb"}]}
