)]}'
{"tempest/api/volume/test_volumes_negative.py":[{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"a7cc6e424927b8ef3513eb350e0f44b609dda215","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import uuid"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from tempest.api.volume import base"},{"line_number":21,"context_line":"from tempest.common.utils.data_utils import rand_name"},{"line_number":22,"context_line":"from tempest import exceptions"},{"line_number":23,"context_line":"from tempest.test import attr"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2Fhbc%3D","line":20,"updated":"2013-10-15 01:36:26.000000000","message":"After the review of https://review.openstack.org/#/c/46886/9/tempest/api/compute/floating_ips/test_floating_ips_actions.py , we are trying to change\n  from tempest.common.utils.data_utils import rand_name\nto\n  from tempest.common.utils import data_utils\nCould you also apply it in this patch?","commit_id":"c54b44d5e3873c2c106e6dc0021ec811785e3aa5"},{"author":{"_account_id":8286,"name":"zhangyanzi","email":"yanzi.zhang@samsung.com","username":"zhangyanzi"},"change_message_id":"7673c2448c19609e911d5a31857a86fade5f23fb","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import uuid"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from tempest.api.volume import base"},{"line_number":21,"context_line":"from tempest.common.utils.data_utils import rand_name"},{"line_number":22,"context_line":"from tempest import exceptions"},{"line_number":23,"context_line":"from tempest.test import attr"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FhZ4%3D","line":20,"in_reply_to":"AAAATH%2F%2Fhbc%3D","updated":"2013-10-15 01:45:56.000000000","message":"Should I change all of this in this file?","commit_id":"c54b44d5e3873c2c106e6dc0021ec811785e3aa5"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"a7cc6e424927b8ef3513eb350e0f44b609dda215","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":88,"context_line":"    def test_create_volume_with_nonexistant_volume_type(self):"},{"line_number":89,"context_line":"        # Should not be able to create volume with non-existant source volume"},{"line_number":90,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":91,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":92,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2Fhbo%3D","line":89,"updated":"2013-10-15 01:36:26.000000000","message":"the annotation isn\u0027t correct.","commit_id":"c54b44d5e3873c2c106e6dc0021ec811785e3aa5"},{"author":{"_account_id":8286,"name":"zhangyanzi","email":"yanzi.zhang@samsung.com","username":"zhangyanzi"},"change_message_id":"7673c2448c19609e911d5a31857a86fade5f23fb","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":88,"context_line":"    def test_create_volume_with_nonexistant_volume_type(self):"},{"line_number":89,"context_line":"        # Should not be able to create volume with non-existant source volume"},{"line_number":90,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":91,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":92,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FhaY%3D","line":89,"in_reply_to":"AAAATH%2F%2Fhbo%3D","updated":"2013-10-15 01:45:56.000000000","message":"OK","commit_id":"c54b44d5e3873c2c106e6dc0021ec811785e3aa5"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"a7cc6e424927b8ef3513eb350e0f44b609dda215","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":97,"context_line":"    def test_create_volume_with_nonexistant_snapshot_id(self):"},{"line_number":98,"context_line":"        # Should not be able to create volume with non-existant source volume"},{"line_number":99,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":100,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":101,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2Fhbw%3D","line":98,"updated":"2013-10-15 01:36:26.000000000","message":"the annotation isn\u0027t correct.","commit_id":"c54b44d5e3873c2c106e6dc0021ec811785e3aa5"},{"author":{"_account_id":8286,"name":"zhangyanzi","email":"yanzi.zhang@samsung.com","username":"zhangyanzi"},"change_message_id":"7673c2448c19609e911d5a31857a86fade5f23fb","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":97,"context_line":"    def test_create_volume_with_nonexistant_snapshot_id(self):"},{"line_number":98,"context_line":"        # Should not be able to create volume with non-existant source volume"},{"line_number":99,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":100,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":101,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FhaU%3D","line":98,"in_reply_to":"AAAATH%2F%2Fhbw%3D","updated":"2013-10-15 01:45:56.000000000","message":"OK","commit_id":"c54b44d5e3873c2c106e6dc0021ec811785e3aa5"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"8d1801637a620ed028eb08efd82f9ecdf4e90c82","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    def test_create_volume_with_nonexistant_snapshot_id(self):"},{"line_number":98,"context_line":"        # Should not be able to create volume with non-existant snapshot"},{"line_number":99,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":100,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":101,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"},{"line_number":102,"context_line":"                          size\u003d\u00271\u0027, snapshot_id\u003dstr(uuid.uuid4()),"},{"line_number":103,"context_line":"                          display_name\u003dv_name, metadata\u003dmetadata)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FXqg%3D","line":100,"updated":"2013-10-17 03:55:44.000000000","message":"There is question: if use both nonexistant volume type and nonexistant snapshot id, we don\u0027t know which is NotFound exactly.","commit_id":"b866f05064e44cf0ef7cff1b15d8bb84a9b16b2a"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"39b8e1115afbefa972a35c7791aca65e725affbc","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    def test_create_volume_with_nonexistant_snapshot_id(self):"},{"line_number":98,"context_line":"        # Should not be able to create volume with non-existant snapshot"},{"line_number":99,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":100,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":101,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"},{"line_number":102,"context_line":"                          size\u003d\u00271\u0027, snapshot_id\u003dstr(uuid.uuid4()),"},{"line_number":103,"context_line":"                          display_name\u003dv_name, metadata\u003dmetadata)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FXRc%3D","line":100,"in_reply_to":"AAAATH%2F%2FXo8%3D","updated":"2013-10-17 07:25:04.000000000","message":"sorry, I have wrong understanding about this test. It\u0027s my fault. This test is no problem.","commit_id":"b866f05064e44cf0ef7cff1b15d8bb84a9b16b2a"},{"author":{"_account_id":8286,"name":"zhangyanzi","email":"yanzi.zhang@samsung.com","username":"zhangyanzi"},"change_message_id":"88dcfbf503cbca9bc7a05ed5283cfd329ca35ccb","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    def test_create_volume_with_nonexistant_snapshot_id(self):"},{"line_number":98,"context_line":"        # Should not be able to create volume with non-existant snapshot"},{"line_number":99,"context_line":"        v_name \u003d rand_name(\u0027Volume-\u0027)"},{"line_number":100,"context_line":"        metadata \u003d {\u0027Type\u0027: \u0027work\u0027}"},{"line_number":101,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.create_volume,"},{"line_number":102,"context_line":"                          size\u003d\u00271\u0027, snapshot_id\u003dstr(uuid.uuid4()),"},{"line_number":103,"context_line":"                          display_name\u003dv_name, metadata\u003dmetadata)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FXo8%3D","line":100,"in_reply_to":"AAAATH%2F%2FXqg%3D","updated":"2013-10-17 04:15:14.000000000","message":"I did not pass volume_type,so no problem.","commit_id":"b866f05064e44cf0ef7cff1b15d8bb84a9b16b2a"}]}