)]}'
{"cinder/volume/drivers/vmware/vmdk.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c5a150d56cac33bb3adf58724ba91f8f0aa929ec","unresolved":false,"context_lines":[{"line_number":2182,"context_line":"            self._clone_attached_volume(src_vref, volume)"},{"line_number":2183,"context_line":"        else:"},{"line_number":2184,"context_line":"            try:"},{"line_number":2185,"context_line":"                self._clone_backing(volume, backing, snapshot, clone_type,"},{"line_number":2186,"context_line":"                                    src_vref[\u0027size\u0027])"},{"line_number":2187,"context_line":"            finally:"},{"line_number":2188,"context_line":"                if snapshot:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_37084f90","line":2185,"range":{"start_line":2185,"start_character":21,"end_line":2185,"end_character":35},"updated":"2018-04-13 01:25:52.000000000","message":"There are two cases where a new volume from an existing source may be larger than the original: creating from snapshot and cloning a volume. This doesn\u0027t look like it takes that into consideration, but I\u0027m not sure if I am tracing everything right. Can you confirm both cases are covered?","commit_id":"cce7307ec1281745c718ddbed2d15ba1b07acde5"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"b529056ada6d0610ddfff32e1b5eacd0bfea892e","unresolved":false,"context_lines":[{"line_number":2182,"context_line":"            self._clone_attached_volume(src_vref, volume)"},{"line_number":2183,"context_line":"        else:"},{"line_number":2184,"context_line":"            try:"},{"line_number":2185,"context_line":"                self._clone_backing(volume, backing, snapshot, clone_type,"},{"line_number":2186,"context_line":"                                    src_vref[\u0027size\u0027])"},{"line_number":2187,"context_line":"            finally:"},{"line_number":2188,"context_line":"                if snapshot:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_7ba8d78a","line":2185,"range":{"start_line":2185,"start_character":21,"end_line":2185,"end_character":35},"in_reply_to":"bf659307_37084f90","updated":"2018-04-13 22:49:32.000000000","message":"Yes, both cases are now covered.","commit_id":"cce7307ec1281745c718ddbed2d15ba1b07acde5"}]}