)]}'
{"tempest/api/volume/test_snapshot_metadata.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"56482bbd94c1667e66ed3b2372912826a1095647","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        self.assertNotIn(\"key3\", body)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @decorators.idempotent_id(\u0027e8ff85c5-8f97-477f-806a-3ac364a949ed\u0027)"},{"line_number":82,"context_line":"    def test_update_show_snapshot_metadata_item(self):"},{"line_number":83,"context_line":"        # Update metadata item for the snapshot"},{"line_number":84,"context_line":"        metadata \u003d {\"key1\": \"value1\","},{"line_number":85,"context_line":"                    \"key2\": \"value2\","}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_3541b53a","line":82,"updated":"2017-07-03 06:09:12.000000000","message":"This is one of defcore tests[1]. So I think we need to sync defcore.\n\n[1] http://codesearch.openstack.org/?q\u003dtest_update_snapshot_metadata_item\u0026i\u003dnope\u0026files\u003d\u0026repos\u003d","commit_id":"05f490492c9a972972072492c84421ef40d568bd"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"a55f08545c3fdcee4167e506b43cd3da2a3bc339","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        self.assertNotIn(\"key3\", body)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @decorators.idempotent_id(\u0027e8ff85c5-8f97-477f-806a-3ac364a949ed\u0027)"},{"line_number":82,"context_line":"    def test_update_show_snapshot_metadata_item(self):"},{"line_number":83,"context_line":"        # Update metadata item for the snapshot"},{"line_number":84,"context_line":"        metadata \u003d {\"key1\": \"value1\","},{"line_number":85,"context_line":"                    \"key2\": \"value2\","}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_0bd69cc4","line":82,"in_reply_to":"3f1d235d_3541b53a","updated":"2017-07-03 07:52:58.000000000","message":"Done.\nAccording defcore patch is I672bbcb9983e7a3ace7769722d6ff97f7bc0ccc8","commit_id":"05f490492c9a972972072492c84421ef40d568bd"}],"tempest/lib/services/volume/v2/snapshots_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"12da1b6d6c8a7f1f7639be7940bf1519bd31c3f5","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    def show_snapshot_metadata_item(self, snapshot_id, id):"},{"line_number":168,"context_line":"        \"\"\"Get snapshot metadata for a specific key."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"        TODO(jeremyZ): Current api-site doesn\u0027t contain this API description."},{"line_number":171,"context_line":"        After fixing the api-site, we need to fix here also for putting"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_4ae94e18","line":168,"updated":"2017-06-30 15:17:38.000000000","message":"Same comments from this patch apply here: https://review.openstack.org/#/c/479152/","commit_id":"e82f2c183ed0f9951bd5a3ec206ec23526b85161"}],"tempest/tests/lib/services/volume/v2/test_snapshots_client.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"56482bbd94c1667e66ed3b2372912826a1095647","unresolved":false,"context_lines":[{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def _test_show_snapshot_metadata_item(self, bytes_body\u003dFalse):"},{"line_number":145,"context_line":"        self.check_service_client_function("},{"line_number":146,"context_line":"            self.client.show_snapshot_metadata_item,"},{"line_number":147,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":148,"context_line":"            self.FAKE_SNAPSHOT_METADATA_ITEM,"},{"line_number":149,"context_line":"            bytes_body,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_555af129","line":146,"updated":"2017-07-03 06:09:12.000000000","message":"I think we don\u0027t need bytes_body parameter becase show_snapshot_metadata_item doesn\u0027t send a request body.","commit_id":"05f490492c9a972972072492c84421ef40d568bd"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"a55f08545c3fdcee4167e506b43cd3da2a3bc339","unresolved":false,"context_lines":[{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def _test_show_snapshot_metadata_item(self, bytes_body\u003dFalse):"},{"line_number":145,"context_line":"        self.check_service_client_function("},{"line_number":146,"context_line":"            self.client.show_snapshot_metadata_item,"},{"line_number":147,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":148,"context_line":"            self.FAKE_SNAPSHOT_METADATA_ITEM,"},{"line_number":149,"context_line":"            bytes_body,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_cbdf84d4","line":146,"in_reply_to":"3f1d235d_555af129","updated":"2017-07-03 07:52:58.000000000","message":"Done, thanks","commit_id":"05f490492c9a972972072492c84421ef40d568bd"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"cdaef720e557be38005042efb9b188de2e7047ae","unresolved":false,"context_lines":[{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def _test_show_snapshot_metadata_item(self, bytes_body\u003dFalse):"},{"line_number":145,"context_line":"        self.check_service_client_function("},{"line_number":146,"context_line":"            self.client.show_snapshot_metadata_item,"},{"line_number":147,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":148,"context_line":"            self.FAKE_SNAPSHOT_METADATA_ITEM,"},{"line_number":149,"context_line":"            bytes_body,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_257b8af5","line":146,"in_reply_to":"3f1d235d_cbdf84d4","updated":"2017-07-05 02:41:07.000000000","message":"@Masayuki\nHere I check the source codes of functions \u0027check_service_client_function\u0027 and \u0027create_response\u0027 in tempest/tests/lib/services/base.py.\nThe \u0027bytes_body\u0027 parameter are used for response body.","commit_id":"05f490492c9a972972072492c84421ef40d568bd"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"54c9fafdf8471eeaf62ee3b084ae2cd2c25c2ce2","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        self._test_update_snapshot_metadata(bytes_body\u003dTrue)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"    def test_show_snapshot_metadata_item(self):"},{"line_number":194,"context_line":"        self.check_service_client_function("},{"line_number":195,"context_line":"            self.client.show_snapshot_metadata_item,"},{"line_number":196,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":197,"context_line":"            self.FAKE_SNAPSHOT_METADATA_ITEM,"},{"line_number":198,"context_line":"            snapshot_id\u003d\"3fbbcccf-d058-4502-8844-6feeffdf4cb5\","},{"line_number":199,"context_line":"            id\u003d\"key1\")"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    def test_force_delete_snapshot(self):"},{"line_number":202,"context_line":"        self.check_service_client_function("}],"source_content_type":"text/x-python","patch_set":6,"id":"3f1d235d_2a335916","line":199,"range":{"start_line":194,"start_character":0,"end_line":199,"end_character":22},"updated":"2017-07-05 02:06:02.000000000","message":"we should do this with and without bytes_body as it has response body.","commit_id":"b34fb2b978d5f77ed5d48c145b96b9ec4a773753"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"7d4c9c7d4fcebbaa4a66336bc961ae0af2c0c055","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        self._test_update_snapshot_metadata(bytes_body\u003dTrue)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"    def test_show_snapshot_metadata_item(self):"},{"line_number":194,"context_line":"        self.check_service_client_function("},{"line_number":195,"context_line":"            self.client.show_snapshot_metadata_item,"},{"line_number":196,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":197,"context_line":"            self.FAKE_SNAPSHOT_METADATA_ITEM,"},{"line_number":198,"context_line":"            snapshot_id\u003d\"3fbbcccf-d058-4502-8844-6feeffdf4cb5\","},{"line_number":199,"context_line":"            id\u003d\"key1\")"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    def test_force_delete_snapshot(self):"},{"line_number":202,"context_line":"        self.check_service_client_function("}],"source_content_type":"text/x-python","patch_set":6,"id":"3f1d235d_4beb37db","line":199,"range":{"start_line":194,"start_character":0,"end_line":199,"end_character":22},"in_reply_to":"3f1d235d_2a335916","updated":"2017-07-05 04:14:48.000000000","message":"Done","commit_id":"b34fb2b978d5f77ed5d48c145b96b9ec4a773753"}]}