)]}'
{"tempest/lib/services/volume/v2/volumes_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9ebea6d17ef9f49055df39fdbd289aa0b5c898ea","unresolved":false,"context_lines":[{"line_number":284,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"    def show_volume_metadata_item(self, volume_id, id):"},{"line_number":287,"context_line":"        \"\"\"Get volume metadata for a specific key."},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        TODO(jeremyZ): Current api-site doesn\u0027t contain this API description."},{"line_number":290,"context_line":"        After fixing the api-site, we need to fix here also for putting"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_ea5b7a2b","line":287,"range":{"start_line":287,"start_character":8,"end_line":287,"end_character":50},"updated":"2017-06-30 15:15:49.000000000","message":"I think it\u0027s better to reword this as:\n\nShow metadata item for the volume.\n\nTo be consistent with ``update_volume_metadata_item`` and others below.","commit_id":"63f117d38038c863751b8eaaeab949a22d1922a8"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"334432ca6355cf9f2c5b6b09e43659c5d94dad23","unresolved":false,"context_lines":[{"line_number":284,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"    def show_volume_metadata_item(self, volume_id, id):"},{"line_number":287,"context_line":"        \"\"\"Get volume metadata for a specific key."},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        TODO(jeremyZ): Current api-site doesn\u0027t contain this API description."},{"line_number":290,"context_line":"        After fixing the api-site, we need to fix here also for putting"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_51e7b613","line":287,"range":{"start_line":287,"start_character":8,"end_line":287,"end_character":50},"in_reply_to":"3f1d235d_ea5b7a2b","updated":"2017-07-01 02:40:09.000000000","message":"Ok, it will be updated in next patch.","commit_id":"63f117d38038c863751b8eaaeab949a22d1922a8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9ebea6d17ef9f49055df39fdbd289aa0b5c898ea","unresolved":false,"context_lines":[{"line_number":286,"context_line":"    def show_volume_metadata_item(self, volume_id, id):"},{"line_number":287,"context_line":"        \"\"\"Get volume metadata for a specific key."},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        TODO(jeremyZ): Current api-site doesn\u0027t contain this API description."},{"line_number":290,"context_line":"        After fixing the api-site, we need to fix here also for putting"},{"line_number":291,"context_line":"        the link to api-site."},{"line_number":292,"context_line":"        \"\"\""},{"line_number":293,"context_line":"        url \u003d \"volumes/%s/metadata/%s\" % (volume_id, id)"},{"line_number":294,"context_line":"        resp, body \u003d self.get(url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_aa38e2d2","line":291,"range":{"start_line":289,"start_character":8,"end_line":291,"end_character":29},"updated":"2017-06-30 15:15:49.000000000","message":"This is only true if this endpoint takes **kwargs or **params.","commit_id":"63f117d38038c863751b8eaaeab949a22d1922a8"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"334432ca6355cf9f2c5b6b09e43659c5d94dad23","unresolved":false,"context_lines":[{"line_number":286,"context_line":"    def show_volume_metadata_item(self, volume_id, id):"},{"line_number":287,"context_line":"        \"\"\"Get volume metadata for a specific key."},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        TODO(jeremyZ): Current api-site doesn\u0027t contain this API description."},{"line_number":290,"context_line":"        After fixing the api-site, we need to fix here also for putting"},{"line_number":291,"context_line":"        the link to api-site."},{"line_number":292,"context_line":"        \"\"\""},{"line_number":293,"context_line":"        url \u003d \"volumes/%s/metadata/%s\" % (volume_id, id)"},{"line_number":294,"context_line":"        resp, body \u003d self.get(url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_71e072fb","line":291,"range":{"start_line":289,"start_character":8,"end_line":291,"end_character":29},"in_reply_to":"3f1d235d_aa38e2d2","updated":"2017-07-01 02:40:09.000000000","message":"Yeah, it can be removed.","commit_id":"63f117d38038c863751b8eaaeab949a22d1922a8"}],"tempest/tests/lib/services/volume/v2/test_volumes_client.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"418e5fc7f5fdbacb8c61b9e94f090cb70703d280","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def _test_show_volume_metadata_item(self, bytes_body\u003dFalse):"},{"line_number":55,"context_line":"        self.check_service_client_function("},{"line_number":56,"context_line":"            self.client.show_volume_metadata_item,"},{"line_number":57,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":58,"context_line":"            self.FAKE_VOLUME_METADATA_ITEM,"},{"line_number":59,"context_line":"            bytes_body,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_03db26ed","line":56,"updated":"2017-07-03 05:51:02.000000000","message":"I think the bytes_body parameter isn\u0027t necessary because this method doesn\u0027t have a request body.","commit_id":"84a2dbc268eb4e149839ecfb1e688cfd0ce692ed"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"3e5b5c044108163a4c04e9b3f0c47edb64b50219","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def _test_show_volume_metadata_item(self, bytes_body\u003dFalse):"},{"line_number":55,"context_line":"        self.check_service_client_function("},{"line_number":56,"context_line":"            self.client.show_volume_metadata_item,"},{"line_number":57,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":58,"context_line":"            self.FAKE_VOLUME_METADATA_ITEM,"},{"line_number":59,"context_line":"            bytes_body,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_ee1c8e6b","line":56,"in_reply_to":"3f1d235d_03db26ed","updated":"2017-07-03 08:10:21.000000000","message":"Done, thanks","commit_id":"84a2dbc268eb4e149839ecfb1e688cfd0ce692ed"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8447fa31e6c154bf9090e1cbe3d4254e415665b8","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        self._test_force_detach_volume(bytes_body\u003dTrue)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def test_show_volume_metadata_item(self):"},{"line_number":61,"context_line":"        self.check_service_client_function("},{"line_number":62,"context_line":"            self.client.show_volume_metadata_item,"},{"line_number":63,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":64,"context_line":"            self.FAKE_VOLUME_METADATA_ITEM,"},{"line_number":65,"context_line":"            volume_id\u003d\"a3be971b-8de5-4bdf-bdb8-3d8eb0fb69f8\","},{"line_number":66,"context_line":"            id\u003d\"key1\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_6548625b","line":66,"range":{"start_line":61,"start_character":0,"end_line":66,"end_character":22},"updated":"2017-07-05 02:07:06.000000000","message":"we should tests this with byte_body as it has response body.","commit_id":"91913cfd275db1b9d290a1062f729be3637aa383"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"49ad2d57330b9673eb088fa66141f709b28376a4","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        self._test_force_detach_volume(bytes_body\u003dTrue)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def test_show_volume_metadata_item(self):"},{"line_number":61,"context_line":"        self.check_service_client_function("},{"line_number":62,"context_line":"            self.client.show_volume_metadata_item,"},{"line_number":63,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":64,"context_line":"            self.FAKE_VOLUME_METADATA_ITEM,"},{"line_number":65,"context_line":"            volume_id\u003d\"a3be971b-8de5-4bdf-bdb8-3d8eb0fb69f8\","},{"line_number":66,"context_line":"            id\u003d\"key1\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_ebda0349","line":66,"range":{"start_line":61,"start_character":0,"end_line":66,"end_character":22},"in_reply_to":"3f1d235d_6548625b","updated":"2017-07-05 04:14:05.000000000","message":"Done","commit_id":"91913cfd275db1b9d290a1062f729be3637aa383"}]}