)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"3c3e19e23bf7165f4e630e6f45a3b02a178b44fb","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-03-22 14:31:28 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add test case for update volume encryption type"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Including:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[1] Add update encryption type api to v2 encryption types client"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ffe62b97_15f0b0fd","line":8,"updated":"2017-03-28 08:55:33.000000000","message":"I actually do want to know the reason why we need this test in the commit message. It\u0027s really useful to understand the patch.","commit_id":"316d56462c23cffba3ad99bc1e306a81781c39f8"}],"tempest/api/volume/admin/test_volume_types.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"6af90eb31948066986a527696f3ed546a324bc1c","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            fetched_volume_type[\u0027os-volume-type-access:is_public\u0027])"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    @decorators.idempotent_id(\u00277830abd0-ff99-4793-a265-405684a54d46\u0027)"},{"line_number":126,"context_line":"    def test_volume_type_encryption_crud(self):"},{"line_number":127,"context_line":"        # Create/get/update/delete encryption type."},{"line_number":128,"context_line":"        create_kwargs \u003d {\u0027provider\u0027: \u0027LuksEncryptor\u0027,"},{"line_number":129,"context_line":"                         \u0027control_location\u0027: \u0027front-end\u0027}"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_99789e2d","line":126,"range":{"start_line":126,"start_character":36,"end_line":126,"end_character":40},"updated":"2017-03-20 07:10:32.000000000","message":"why rename to such a testcase name?","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"4da37b61ced254a02852beae7439ad0ce7182fb2","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            fetched_volume_type[\u0027os-volume-type-access:is_public\u0027])"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    @decorators.idempotent_id(\u00277830abd0-ff99-4793-a265-405684a54d46\u0027)"},{"line_number":126,"context_line":"    def test_volume_type_encryption_crud(self):"},{"line_number":127,"context_line":"        # Create/get/update/delete encryption type."},{"line_number":128,"context_line":"        create_kwargs \u003d {\u0027provider\u0027: \u0027LuksEncryptor\u0027,"},{"line_number":129,"context_line":"                         \u0027control_location\u0027: \u0027front-end\u0027}"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_e3b000e0","line":126,"range":{"start_line":126,"start_character":36,"end_line":126,"end_character":40},"in_reply_to":"1a1ced50_99789e2d","updated":"2017-03-20 09:07:53.000000000","message":"Here crud is short for create_get_update_delete","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"7dd390c7e0a9ac321bb4f5bca53976769a1bfae0","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            fetched_volume_type[\u0027os-volume-type-access:is_public\u0027])"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    @decorators.idempotent_id(\u00277830abd0-ff99-4793-a265-405684a54d46\u0027)"},{"line_number":126,"context_line":"    def test_volume_type_encryption_crud(self):"},{"line_number":127,"context_line":"        # Create/get/update/delete encryption type."},{"line_number":128,"context_line":"        create_kwargs \u003d {\u0027provider\u0027: \u0027LuksEncryptor\u0027,"},{"line_number":129,"context_line":"                         \u0027control_location\u0027: \u0027front-end\u0027}"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_e5ad0985","line":126,"range":{"start_line":126,"start_character":36,"end_line":126,"end_character":40},"in_reply_to":"1a1ced50_e35617d1","updated":"2017-03-22 02:41:09.000000000","message":"you\u0027re right, I will change the name to test_volume_type_encryption_create_get_update_delete","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"c8181e1279e38407e1137ed2eea3dc3276568e77","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            fetched_volume_type[\u0027os-volume-type-access:is_public\u0027])"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    @decorators.idempotent_id(\u00277830abd0-ff99-4793-a265-405684a54d46\u0027)"},{"line_number":126,"context_line":"    def test_volume_type_encryption_crud(self):"},{"line_number":127,"context_line":"        # Create/get/update/delete encryption type."},{"line_number":128,"context_line":"        create_kwargs \u003d {\u0027provider\u0027: \u0027LuksEncryptor\u0027,"},{"line_number":129,"context_line":"                         \u0027control_location\u0027: \u0027front-end\u0027}"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_e35617d1","line":126,"range":{"start_line":126,"start_character":36,"end_line":126,"end_character":40},"in_reply_to":"1a1ced50_e3b000e0","updated":"2017-03-21 03:04:54.000000000","message":"this seems a bit hard to understand:)","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"}],"tempest/lib/services/volume/v1/encryption_types_client.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"6af90eb31948066986a527696f3ed546a324bc1c","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def update_encryption_type(self, volume_type_id, **kwargs):"},{"line_number":71,"context_line":"        \"\"\"Update an encryption type for an existing volume type.\"\"\""},{"line_number":72,"context_line":"        url \u003d \"/types/%s/encryption/provider\" % volume_type_id"},{"line_number":73,"context_line":"        put_body \u003d json.dumps({\u0027encryption\u0027: kwargs})"},{"line_number":74,"context_line":"        resp, body \u003d self.put(url, put_body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_d91696f0","line":71,"range":{"start_line":71,"start_character":8,"end_line":71,"end_character":68},"updated":"2017-03-20 07:10:32.000000000","message":"api ref link should be added if any","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"c8181e1279e38407e1137ed2eea3dc3276568e77","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def update_encryption_type(self, volume_type_id, **kwargs):"},{"line_number":71,"context_line":"        \"\"\"Update an encryption type for an existing volume type.\"\"\""},{"line_number":72,"context_line":"        url \u003d \"/types/%s/encryption/provider\" % volume_type_id"},{"line_number":73,"context_line":"        put_body \u003d json.dumps({\u0027encryption\u0027: kwargs})"},{"line_number":74,"context_line":"        resp, body \u003d self.put(url, put_body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_e36d3799","line":71,"range":{"start_line":71,"start_character":8,"end_line":71,"end_character":68},"in_reply_to":"1a1ced50_438554ab","updated":"2017-03-21 03:04:54.000000000","message":"maybe you add a TODO here, besides, there is api link in v3","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"4da37b61ced254a02852beae7439ad0ce7182fb2","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def update_encryption_type(self, volume_type_id, **kwargs):"},{"line_number":71,"context_line":"        \"\"\"Update an encryption type for an existing volume type.\"\"\""},{"line_number":72,"context_line":"        url \u003d \"/types/%s/encryption/provider\" % volume_type_id"},{"line_number":73,"context_line":"        put_body \u003d json.dumps({\u0027encryption\u0027: kwargs})"},{"line_number":74,"context_line":"        resp, body \u003d self.put(url, put_body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_438554ab","line":71,"range":{"start_line":71,"start_character":8,"end_line":71,"end_character":68},"in_reply_to":"1a1ced50_d91696f0","updated":"2017-03-20 09:07:53.000000000","message":"It seems that no suitable api ref links were found for encryption type cliens.","commit_id":"fce7e84995d7139a40698cbdcdf53c5fb290bf1c"}]}