)]}'
{"tempest/api/compute/base.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        cls.volumes_client \u003d cls.os_primary.volumes_v2_client"},{"line_number":101,"context_line":"        cls.assisted_volume_snapshots_client \u003d \\"},{"line_number":102,"context_line":"            cls.os_primary.assisted_volume_snapshots_client"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    @classmethod"},{"line_number":105,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_0c098a94","line":102,"range":{"start_line":102,"start_character":16,"end_line":102,"end_character":26},"updated":"2017-05-24 18:34:44.000000000","message":"Should be os_admin and should be in admin class. \n\nhttps://github.com/openstack/nova/blob/master/nova/policies/assisted_volume_snapshots.py#L25\n\nIt\u0027s an admin API.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"}],"tempest/api/compute/volumes/test_volume_snapshots.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"ff5b6806f2a764b8800a64011324527da8244449","unresolved":false,"context_lines":[{"line_number":25,"context_line":"CONF \u003d config.CONF"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class VolumesSnapshotsTestJSON(base.BaseV2ComputeAdminTest):"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def skip_checks(cls):"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff0f0b1f_1f269669","line":28,"range":{"start_line":28,"start_character":32,"end_line":28,"end_character":58},"updated":"2017-05-22 23:34:16.000000000","message":"Please don\u0027t change non-admin tests to admin without changing the path of tests.\n\nIt would be good to add a new test class with admin credential separately.","commit_id":"bea7f9d14b0b05e7d8291810b3f64790c0d0dd33"}],"tempest/lib/services/compute/__init__.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":76,"context_line":"           \u0027SecurityGroupRulesClient\u0027, \u0027SecurityGroupsClient\u0027,"},{"line_number":77,"context_line":"           \u0027ServerGroupsClient\u0027, \u0027ServersClient\u0027, \u0027ServicesClient\u0027,"},{"line_number":78,"context_line":"           \u0027SnapshotsClient\u0027, \u0027TenantNetworksClient\u0027, \u0027TenantUsagesClient\u0027,"},{"line_number":79,"context_line":"           \u0027VersionsClient\u0027, \u0027VolumesClient\u0027, \u0027AssistedVolumeSnapshotsClient\u0027]"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_cce5d2dd","line":79,"range":{"start_line":79,"start_character":47,"end_line":79,"end_character":76},"updated":"2017-05-24 18:34:44.000000000","message":"Better to put in alphabetical order.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"}],"tempest/lib/services/compute/assisted_volume_snapshots_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class AssistedVolumeSnapshotsClient(base_compute_client.BaseComputeClient):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def delete_assisted_volume_snapshot(self, volume_snapshots_id, volume_id):"},{"line_number":26,"context_line":"        query_param \u003d {\u0027delete_info\u0027: json.dumps({\u0027volume_id\u0027: volume_id})}"},{"line_number":27,"context_line":"        resp, body \u003d self.delete(\"os-assisted-volume-snapshots/%s?%s\""},{"line_number":28,"context_line":"                                 % (volume_snapshots_id,"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_ecbd36b7","line":25,"range":{"start_line":25,"start_character":46,"end_line":25,"end_character":76},"updated":"2017-05-24 18:34:44.000000000","message":"volume_id, snapshot_id\n\nto be consistent with order and naming of params in create_assisted_volume_snapshot.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    def create_assisted_volume_snapshot(self, volume_id, snapshot_id):"},{"line_number":33,"context_line":"        url \u003d \"os-assisted-volume-snapshots\""},{"line_number":34,"context_line":"        body \u003d {\"snapshot\": {\"create_info\": {\"snapshot_id\": snapshot_id,"},{"line_number":35,"context_line":"                                             \"type\": \"qcow2\","},{"line_number":36,"context_line":"                                             \"new_file\": \"fake\"},"},{"line_number":37,"context_line":"                             \"volume_id\": volume_id}}"},{"line_number":38,"context_line":"        post_body \u003d json.dumps(body)"},{"line_number":39,"context_line":"        resp, body \u003d self.post(url, post_body)"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_4cc42240","line":36,"range":{"start_line":35,"start_character":45,"end_line":36,"end_character":63},"updated":"2017-05-24 18:34:44.000000000","message":"These should be optional params.\n\nPass in **kwargs to create_assisted_volume_snapshot.\n\nThen do:\n\nif kwargs:\n    body.update(kwargs)","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9dd4cd782f837564f477eb36471be854a69856e1","unresolved":false,"context_lines":[{"line_number":30,"context_line":"                                    urllib.urlencode(query_param)))"},{"line_number":31,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def create_assisted_volume_snapshot(self, volume_id, snapshot_id):"},{"line_number":34,"context_line":"        \"\"\"Create a new assisted volume snapshot."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        For a full list of available parameters, please refer to the official"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_37f7f68d","line":33,"range":{"start_line":33,"start_character":46,"end_line":33,"end_character":68},"updated":"2017-05-24 20:49:28.000000000","message":"volume_id, snapshot_id, **kwargs","commit_id":"e332742f1c126f795703bb95e61b3bbe605c4a36"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9dd4cd782f837564f477eb36471be854a69856e1","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        \"\"\""},{"line_number":40,"context_line":"        url \u003d \"os-assisted-volume-snapshots\""},{"line_number":41,"context_line":"        body \u003d {\"snapshot\": {\"create_info\": {\"snapshot_id\": snapshot_id,"},{"line_number":42,"context_line":"                                             \"type\": \"qcow2\","},{"line_number":43,"context_line":"                                             \"new_file\": \"fake\"},"},{"line_number":44,"context_line":"                             \"volume_id\": volume_id}}"},{"line_number":45,"context_line":"        post_body \u003d json.dumps(body)"},{"line_number":46,"context_line":"        resp, body \u003d self.post(url, post_body)"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_57fcaa6e","line":43,"range":{"start_line":42,"start_character":45,"end_line":43,"end_character":65},"updated":"2017-05-24 20:49:28.000000000","message":"This is still incorrect. The type and new_file should be passed in by the user to this function.","commit_id":"e332742f1c126f795703bb95e61b3bbe605c4a36"},{"author":{"_account_id":20809,"name":"Adnan Khan","email":"rk221s@att.com","username":"rao.adnan.khan"},"change_message_id":"0dfc86b7e4888ce8882887eb5cd63714df81f14d","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        \"\"\""},{"line_number":40,"context_line":"        url \u003d \"os-assisted-volume-snapshots\""},{"line_number":41,"context_line":"        body \u003d {\"snapshot\": {\"create_info\": {\"snapshot_id\": snapshot_id,"},{"line_number":42,"context_line":"                                             \"type\": \"qcow2\","},{"line_number":43,"context_line":"                                             \"new_file\": \"fake\"},"},{"line_number":44,"context_line":"                             \"volume_id\": volume_id}}"},{"line_number":45,"context_line":"        post_body \u003d json.dumps(body)"},{"line_number":46,"context_line":"        resp, body \u003d self.post(url, post_body)"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_b2e2db9e","line":43,"range":{"start_line":42,"start_character":45,"end_line":43,"end_character":65},"in_reply_to":"ff0f0b1f_57fcaa6e","updated":"2017-05-25 13:57:52.000000000","message":"yup. missed that, coming in the next patch!","commit_id":"e332742f1c126f795703bb95e61b3bbe605c4a36"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"1851990a9e0b591e38cbc8478745a941f0270541","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        API reference:"},{"line_number":39,"context_line":"        https://developer.openstack.org/api-ref/compute/#create-assisted-volume-snapshots"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param type: Type of snapshot, such as qcow2"},{"line_number":42,"context_line":"        :param new_file: The name of image file that will be created"},{"line_number":43,"context_line":"        \"\"\""},{"line_number":44,"context_line":"        url \u003d \"os-assisted-volume-snapshots\""},{"line_number":45,"context_line":"        info \u003d {\"snapshot_id\": snapshot_id}"}],"source_content_type":"text/x-python","patch_set":10,"id":"ff0f0b1f_04cc975f","line":42,"range":{"start_line":41,"start_character":8,"end_line":42,"end_character":68},"updated":"2017-05-25 20:01:13.000000000","message":"I believe this is just reserved for the args not the kwargs, i.e. for volume_id and snapshot_id. Not sure if there\u0027s a Tempest policy against including kwargs as well.","commit_id":"046f8f01a176561a641f5e4aabeb7976890b4016"},{"author":{"_account_id":20809,"name":"Adnan Khan","email":"rk221s@att.com","username":"rao.adnan.khan"},"change_message_id":"ff1dded7eab5bccc3b619c014743949c9d97d4d6","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        API reference:"},{"line_number":39,"context_line":"        https://developer.openstack.org/api-ref/compute/#create-assisted-volume-snapshots"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param type: Type of snapshot, such as qcow2"},{"line_number":42,"context_line":"        :param new_file: The name of image file that will be created"},{"line_number":43,"context_line":"        \"\"\""},{"line_number":44,"context_line":"        url \u003d \"os-assisted-volume-snapshots\""},{"line_number":45,"context_line":"        info \u003d {\"snapshot_id\": snapshot_id}"}],"source_content_type":"text/x-python","patch_set":10,"id":"ff0f0b1f_f7fef736","line":42,"range":{"start_line":41,"start_character":8,"end_line":42,"end_character":68},"in_reply_to":"ff0f0b1f_04cc975f","updated":"2017-05-25 20:49:45.000000000","message":"Actually, it is used elsewhere too. For example, see create_server of servers_client.","commit_id":"046f8f01a176561a641f5e4aabeb7976890b4016"}],"tempest/tests/lib/services/compute/test_assisted_volume_snapshots_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestVolumesClient(base.BaseServiceTest):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    FAKE_SNAPSHOT \u003d {"},{"line_number":23,"context_line":"        \"id\": \"bf7b810c-70df-4c64-88a7-8588f7a6739c\","},{"line_number":24,"context_line":"        \"volumeId\": \"59f17c4f-66d4-4271-be40-f200523423a9\""},{"line_number":25,"context_line":"    }"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(TestVolumesClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_8ce91ae7","line":25,"range":{"start_line":22,"start_character":20,"end_line":25,"end_character":5},"updated":"2017-05-24 18:34:44.000000000","message":"This is what it looks like according to docs:\n\n    {\n        \"volume_id\": \"521752a6-acf6-4b2d-bc7a-119f9148cd8c\",\n        \"create_info\": {\n            \"snapshot_id\": \"421752a6-acf6-4b2d-bc7a-119f9148cd8c\",\n            \"type\": \"qcow2\",\n            \"new_file\": \"new_file_name\",\n            \"id\": \"421752a6-acf6-4b2d-bc7a-119f9148cd8c\"\n        }\n    }\n\nhttps://developer.openstack.org/api-ref/compute/?expanded\u003dcreate-assisted-volume-snapshots-detail#","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"53dfbfbb9660c5f85924b6667ccf83a445687e93","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestVolumesClient(base.BaseServiceTest):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    FAKE_SNAPSHOT \u003d {"},{"line_number":23,"context_line":"        \"id\": \"bf7b810c-70df-4c64-88a7-8588f7a6739c\","},{"line_number":24,"context_line":"        \"volumeId\": \"59f17c4f-66d4-4271-be40-f200523423a9\""},{"line_number":25,"context_line":"    }"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(TestVolumesClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_d7c9da25","line":25,"range":{"start_line":22,"start_character":20,"end_line":25,"end_character":5},"in_reply_to":"ff0f0b1f_51d39a04","updated":"2017-05-24 20:51:36.000000000","message":"Ah, ok. But the response body on L37 should be what I have above.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"8010a8f624e8d8779f12b2f9a7645d47401f766b","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestVolumesClient(base.BaseServiceTest):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    FAKE_SNAPSHOT \u003d {"},{"line_number":23,"context_line":"        \"id\": \"bf7b810c-70df-4c64-88a7-8588f7a6739c\","},{"line_number":24,"context_line":"        \"volumeId\": \"59f17c4f-66d4-4271-be40-f200523423a9\""},{"line_number":25,"context_line":"    }"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(TestVolumesClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_94d1fc32","line":25,"range":{"start_line":22,"start_character":20,"end_line":25,"end_character":5},"in_reply_to":"ff0f0b1f_52c37fed","updated":"2017-05-25 18:18:21.000000000","message":"Yes, you\u0027re right: L37 is correct. As long as you pass in arguments for \"type\" and \"new_file\" around L39 so the request is correct.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":20809,"name":"Adnan Khan","email":"rk221s@att.com","username":"rao.adnan.khan"},"change_message_id":"167c4b2a4982a8c80bb20183a865217a7c3ca8f5","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestVolumesClient(base.BaseServiceTest):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    FAKE_SNAPSHOT \u003d {"},{"line_number":23,"context_line":"        \"id\": \"bf7b810c-70df-4c64-88a7-8588f7a6739c\","},{"line_number":24,"context_line":"        \"volumeId\": \"59f17c4f-66d4-4271-be40-f200523423a9\""},{"line_number":25,"context_line":"    }"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(TestVolumesClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_51d39a04","line":25,"range":{"start_line":22,"start_character":20,"end_line":25,"end_character":5},"in_reply_to":"ff0f0b1f_8ce91ae7","updated":"2017-05-24 20:15:23.000000000","message":"What\u0027s given is the response of POST call. This mocked resp is passed to the service client function, for verifying the post response.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":20809,"name":"Adnan Khan","email":"rk221s@att.com","username":"rao.adnan.khan"},"change_message_id":"429c1b21e74f035889dac495243cd6180ff65a0a","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestVolumesClient(base.BaseServiceTest):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    FAKE_SNAPSHOT \u003d {"},{"line_number":23,"context_line":"        \"id\": \"bf7b810c-70df-4c64-88a7-8588f7a6739c\","},{"line_number":24,"context_line":"        \"volumeId\": \"59f17c4f-66d4-4271-be40-f200523423a9\""},{"line_number":25,"context_line":"    }"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def setUp(self):"},{"line_number":28,"context_line":"        super(TestVolumesClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_52c37fed","line":25,"range":{"start_line":22,"start_character":20,"end_line":25,"end_character":5},"in_reply_to":"ff0f0b1f_d7c9da25","updated":"2017-05-25 13:58:32.000000000","message":"the body provided in your comment is the request, what I am talking about is the response, give on L22.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":36,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.post\u0027,"},{"line_number":37,"context_line":"            {\"snapshot\": self.FAKE_SNAPSHOT},"},{"line_number":38,"context_line":"            bytes_body, status\u003d200, volume_id\u003dself.FAKE_SNAPSHOT[\u0027volumeId\u0027],"},{"line_number":39,"context_line":"            snapshot_id\u003dself.FAKE_SNAPSHOT[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_create_assisted_volume_snapshot_with_str_body(self):"},{"line_number":42,"context_line":"        self._test_create_assisted_volume_snapshot()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_acacde9e","line":39,"range":{"start_line":39,"start_character":12,"end_line":39,"end_character":48},"updated":"2017-05-24 18:34:44.000000000","message":"Should pass in create_info dict instead, i.e.:\n\ncreate_info \u003d {\n    \"create_info\": {\n        \"snapshot_id\": \"421752a6-acf6-4b2d-bc7a-119f9148cd8c\",\n        \"type\": \"qcow2\",\n        \"new_file\": \"new_file_name\"\n     }\n}\n\nand then pass **create_info to check_service_client_function. Note that create_info.id is optional.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":20809,"name":"Adnan Khan","email":"rk221s@att.com","username":"rao.adnan.khan"},"change_message_id":"167c4b2a4982a8c80bb20183a865217a7c3ca8f5","unresolved":false,"context_lines":[{"line_number":36,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.post\u0027,"},{"line_number":37,"context_line":"            {\"snapshot\": self.FAKE_SNAPSHOT},"},{"line_number":38,"context_line":"            bytes_body, status\u003d200, volume_id\u003dself.FAKE_SNAPSHOT[\u0027volumeId\u0027],"},{"line_number":39,"context_line":"            snapshot_id\u003dself.FAKE_SNAPSHOT[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_create_assisted_volume_snapshot_with_str_body(self):"},{"line_number":42,"context_line":"        self._test_create_assisted_volume_snapshot()"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_b130b657","line":39,"range":{"start_line":39,"start_character":12,"end_line":39,"end_character":48},"in_reply_to":"ff0f0b1f_acacde9e","updated":"2017-05-24 20:15:23.000000000","message":"Client prepares the body based on the snap_id passed in.","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        self.check_service_client_function("},{"line_number":49,"context_line":"            self.client.delete_assisted_volume_snapshot,"},{"line_number":50,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.delete\u0027,"},{"line_number":51,"context_line":"            {}, status\u003d202, volume_snapshots_id\u003dself.FAKE_SNAPSHOT[\u0027id\u0027],"},{"line_number":52,"context_line":"            volume_id\u003dself.FAKE_SNAPSHOT[\u0027volumeId\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_8c97da48","line":51,"range":{"start_line":51,"start_character":23,"end_line":51,"end_character":26},"updated":"2017-05-24 18:34:44.000000000","message":"204","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cfc7fbde68cf7844b13d9f49e1d0238b8ac8a670","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        self.check_service_client_function("},{"line_number":49,"context_line":"            self.client.delete_assisted_volume_snapshot,"},{"line_number":50,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.delete\u0027,"},{"line_number":51,"context_line":"            {}, status\u003d202, volume_snapshots_id\u003dself.FAKE_SNAPSHOT[\u0027id\u0027],"},{"line_number":52,"context_line":"            volume_id\u003dself.FAKE_SNAPSHOT[\u0027volumeId\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff0f0b1f_4c9d4264","line":51,"range":{"start_line":51,"start_character":28,"end_line":51,"end_character":47},"updated":"2017-05-24 18:34:44.000000000","message":"snapshot_id","commit_id":"ff205ca4b31c8a68568ae7ed05186f1c7441c0db"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9dd4cd782f837564f477eb36471be854a69856e1","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        self.check_service_client_function("},{"line_number":35,"context_line":"            self.client.create_assisted_volume_snapshot,"},{"line_number":36,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.post\u0027,"},{"line_number":37,"context_line":"            {\"snapshot\": self.FAKE_SNAPSHOT},"},{"line_number":38,"context_line":"            bytes_body, status\u003d200, volume_id\u003dself.FAKE_SNAPSHOT[\u0027volumeId\u0027],"},{"line_number":39,"context_line":"            snapshot_id\u003dself.FAKE_SNAPSHOT[\u0027id\u0027])"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_f7417ea7","line":37,"range":{"start_line":37,"start_character":12,"end_line":37,"end_character":45},"updated":"2017-05-24 20:49:28.000000000","message":"This response body is incorrect.\n\nShould be:\n\n{\n    \"snapshot\": {\n        \"volume_id\": \"521752a6-acf6-4b2d-bc7a-119f9148cd8c\",\n        \"create_info\": {\n            \"snapshot_id\": \"421752a6-acf6-4b2d-bc7a-119f9148cd8c\",\n            \"type\": \"qcow2\",\n            \"new_file\": \"new_file_name\",\n            \"id\": \"421752a6-acf6-4b2d-bc7a-119f9148cd8c\"\n        }\n    }\n}","commit_id":"e332742f1c126f795703bb95e61b3bbe605c4a36"}]}