)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"ecd711271739ee3ce28661ca6b89707ae502892b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4f0fdf2a_81f8256d","updated":"2022-03-18 14:20:01.000000000","message":"LGTM. As its testing timeout exception, we dont need to wait for the full timeout default value.","commit_id":"7a575388ff59410019ab196bcc5a160315d25138"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c6a39cfa5a8d1261ef71d2a6b518cc1f1e9e2ee3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a4749bdc_017d02cd","updated":"2022-03-18 15:17:52.000000000","message":"Looks fine.","commit_id":"7a575388ff59410019ab196bcc5a160315d25138"}],"cinder/tests/unit/volume/drivers/hitachi/test_hitachi_hbsd_rest_fc.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c6a39cfa5a8d1261ef71d2a6b518cc1f1e9e2ee3","unresolved":true,"context_lines":[{"line_number":661,"context_line":"    @reduce_retrying_time"},{"line_number":662,"context_line":"    @mock.patch.object(requests.Session, \"request\")"},{"line_number":663,"context_line":"    def test_create_volume_timeout(self, request):"},{"line_number":664,"context_line":"        self.configuration.hitachi_rest_timeout \u003d 2"},{"line_number":665,"context_line":"        request.return_value \u003d FakeResponse("},{"line_number":666,"context_line":"            500, ERROR_RESULT,"},{"line_number":667,"context_line":"            headers\u003d{\u0027Content-Type\u0027: \u0027json\u0027})"}],"source_content_type":"text/x-python","patch_set":1,"id":"4ed01f42_d6c6dec0","line":664,"updated":"2022-03-18 15:17:52.000000000","message":"I was going to suggest using \"self.flags(\u0027hitachi_rest_timeout\u0027, 2)\" here to keep the change local, but it looks like the configuration object is regenerated for each test (!).","commit_id":"7a575388ff59410019ab196bcc5a160315d25138"}]}