)]}'
{"cinder/volume/drivers/huawei/huawei_driver.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"9ac56d084849d8c4519435f0398f984751c175bd","unresolved":false,"context_lines":[{"line_number":655,"context_line":"        pool_name \u003d host[\u0027capabilities\u0027][\u0027pool_name\u0027]"},{"line_number":656,"context_line":"        pools \u003d self.client.get_all_pools()"},{"line_number":657,"context_line":"        pool_info \u003d self.client.get_pool_info(pool_name, pools)"},{"line_number":658,"context_line":"        dst_volume_name \u003d str(uuid.uuid4())"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"        src_id, lun_wwn \u003d huawei_utils.get_volume_lun_id(self.client, volume)"},{"line_number":661,"context_line":"        opts \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf3e73b7_11d97482","line":658,"range":{"start_line":658,"start_character":26,"end_line":658,"end_character":29},"updated":"2017-08-07 03:34:54.000000000","message":"six.text_type?","commit_id":"49f40ed9edfff6d45fd8aa37ef011b9d2106a800"},{"author":{"_account_id":16203,"name":"zengyingzhe","email":"zengyingzhe@huawei.com","username":"zengyingzhe"},"change_message_id":"e22dd8a056171b46e0d377bf1d7f086d49eb4b76","unresolved":false,"context_lines":[{"line_number":655,"context_line":"        pool_name \u003d host[\u0027capabilities\u0027][\u0027pool_name\u0027]"},{"line_number":656,"context_line":"        pools \u003d self.client.get_all_pools()"},{"line_number":657,"context_line":"        pool_info \u003d self.client.get_pool_info(pool_name, pools)"},{"line_number":658,"context_line":"        dst_volume_name \u003d str(uuid.uuid4())"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"        src_id, lun_wwn \u003d huawei_utils.get_volume_lun_id(self.client, volume)"},{"line_number":661,"context_line":"        opts \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f436f4f_15af3495","line":658,"range":{"start_line":658,"start_character":26,"end_line":658,"end_character":29},"in_reply_to":"bf3e73b7_11d97482","updated":"2017-08-08 02:35:30.000000000","message":"Done","commit_id":"49f40ed9edfff6d45fd8aa37ef011b9d2106a800"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"9ac56d084849d8c4519435f0398f984751c175bd","unresolved":false,"context_lines":[{"line_number":1671,"context_line":"            for snapshot in snapshots:"},{"line_number":1672,"context_line":"                volume \u003d snapshot.volume"},{"line_number":1673,"context_line":"                if not volume:"},{"line_number":1674,"context_line":"                    msg \u003d _(\"Can\u0027t get volume from snapshot %s\") % snapshot.id"},{"line_number":1675,"context_line":"                    LOG.error(msg)"},{"line_number":1676,"context_line":"                    raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1677,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf3e73b7_1194d48a","line":1674,"range":{"start_line":1674,"start_character":29,"end_line":1674,"end_character":62},"updated":"2017-08-07 03:34:54.000000000","message":"Missing period.","commit_id":"49f40ed9edfff6d45fd8aa37ef011b9d2106a800"},{"author":{"_account_id":16203,"name":"zengyingzhe","email":"zengyingzhe@huawei.com","username":"zengyingzhe"},"change_message_id":"e22dd8a056171b46e0d377bf1d7f086d49eb4b76","unresolved":false,"context_lines":[{"line_number":1671,"context_line":"            for snapshot in snapshots:"},{"line_number":1672,"context_line":"                volume \u003d snapshot.volume"},{"line_number":1673,"context_line":"                if not volume:"},{"line_number":1674,"context_line":"                    msg \u003d _(\"Can\u0027t get volume from snapshot %s\") % snapshot.id"},{"line_number":1675,"context_line":"                    LOG.error(msg)"},{"line_number":1676,"context_line":"                    raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1677,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f436f4f_f593905b","line":1674,"range":{"start_line":1674,"start_character":29,"end_line":1674,"end_character":62},"in_reply_to":"bf3e73b7_1194d48a","updated":"2017-08-08 02:35:30.000000000","message":"Done","commit_id":"49f40ed9edfff6d45fd8aa37ef011b9d2106a800"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e21711b5c00f2437c2d357a6608a75b3638d99d0","unresolved":false,"context_lines":[{"line_number":740,"context_line":""},{"line_number":741,"context_line":"        snapshot_id \u003d huawei_utils.get_snapshot_id(self.client, snapshot)"},{"line_number":742,"context_line":"        if snapshot_id is None:"},{"line_number":743,"context_line":"            msg \u003d _(\u0027create_volume_from_snapshot: Snapshot %s not exist.\u0027"},{"line_number":744,"context_line":"                    ) % snapshot.id"},{"line_number":745,"context_line":"            LOG.error(msg)"},{"line_number":746,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_7b7d4ae6","line":743,"updated":"2017-08-08 09:11:18.000000000","message":"OK for string freeze since this string already exists, just changing the format in the code.","commit_id":"5222d6def13aaad5be04400b2cb04f5f5275bca5"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e21711b5c00f2437c2d357a6608a75b3638d99d0","unresolved":false,"context_lines":[{"line_number":821,"context_line":"        \"\"\""},{"line_number":822,"context_line":"        lun_id, lun_wwn \u003d huawei_utils.get_volume_lun_id(self.client, volume)"},{"line_number":823,"context_line":"        if not lun_id:"},{"line_number":824,"context_line":"            msg \u003d _(\"Cannot get lun id of volume %s.\") % volume.id"},{"line_number":825,"context_line":"            if action \u003d\u003d constants.VOLUME_NOT_EXISTS_WARN:"},{"line_number":826,"context_line":"                LOG.warning(msg)"},{"line_number":827,"context_line":"            if action \u003d\u003d constants.VOLUME_NOT_EXISTS_RAISE:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_bb4bb2ae","line":824,"updated":"2017-08-08 09:11:18.000000000","message":"Hmm, this one doesn\u0027t exist, so this would be a string change during string freeze. Can you change it back to the old string for now? It could then be updated later, after string freeze is over, as part of another patch.","commit_id":"5222d6def13aaad5be04400b2cb04f5f5275bca5"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e21711b5c00f2437c2d357a6608a75b3638d99d0","unresolved":false,"context_lines":[{"line_number":886,"context_line":"    def create_snapshot(self, snapshot):"},{"line_number":887,"context_line":"        volume \u003d snapshot.volume"},{"line_number":888,"context_line":"        if not volume:"},{"line_number":889,"context_line":"            msg \u003d _(\"Can\u0027t get volume from snapshot %s.\") % snapshot.id"},{"line_number":890,"context_line":"            LOG.error(msg)"},{"line_number":891,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":892,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_1bdabee8","line":889,"updated":"2017-08-08 09:11:18.000000000","message":"Same","commit_id":"5222d6def13aaad5be04400b2cb04f5f5275bca5"}]}