)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"2eec9dacd9c2cc643e33868ee28335fada438b96","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_38327511","line":23,"updated":"2015-05-13 11:03:55.000000000","message":"How and where this is ensured?","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0d399d73193a028cadb8cd5f175aec01cd92ac46","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_3edeea6d","line":23,"updated":"2015-05-13 04:07:52.000000000","message":"This note implies that this option can be on or off controlled externally.  Is there a way to control when to use this option?  It looks that if lvm_type \u003d\u003d \u0027thin\u0027, sparse will be set to True and this option will be used automatically.","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"change_message_id":"0ddc97ffadab1ea8ef23baf9f699830af417e4d0","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_673de0ba","line":23,"in_reply_to":"9a0c5dc1_38327511","updated":"2015-05-13 15:53:00.000000000","message":"If lvm_type \u003d\u003d \u0027thin\u0027, destination volume(thin-lv) will be created from the thin-pool. In my understanding thin-lv returns zero for unallocated block, this means destination volume is zero cleared.","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"change_message_id":"0ddc97ffadab1ea8ef23baf9f699830af417e4d0","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_a7bf5dfa","line":23,"in_reply_to":"9a0c5dc1_3edeea6d","updated":"2015-05-13 15:53:00.000000000","message":"Hi, Xyang,\n\nMy bad. This just explains the condition when we can apply conv\u003dsparse option for dd command. But user might mis-understand whether they can on/off of this parameter at Cinder. I will rephrase this.","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"40a997c64a7903e429a6565254c7899549a7d33c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"- Without conv\u003dsparse option"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_01bbb4e6","line":24,"updated":"2015-05-13 15:29:13.000000000","message":"Agree that this needs to be better explained.  It doesn\u0027t appear to be configurable.  Do we want to make it configurable or do we need to make sure that volumes are being zero cleared when using this option?","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"e6be078a0cee36ff1087e4f69ccdc7bc193d0a4c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"- Without conv\u003dsparse option"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_c43b6aea","line":24,"in_reply_to":"9a0c5dc1_01bbb4e6","updated":"2015-05-13 15:49:42.000000000","message":"This shouldn\u0027t be an option.  It\u0027s not something an admin would have a reason to change.  We just need to ensure that we don\u0027t do a sparse write when we are wiping a volume to erase the data (thick LVM).\n\nThis is done with this method:\nhttp://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/utils.py?id\u003d88f4c0f05#n338\n\nThis looks ok to me as-is.","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"17da9b5b100d2818e8a30def3802f041afc03973","unresolved":false,"context_lines":[{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"- Without conv\u003dsparse option"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_a712b807","line":24,"in_reply_to":"9a0c5dc1_c43b6aea","updated":"2015-05-13 15:57:51.000000000","message":"Agree, it is looks like ok for now, but I propose to add note about  security issue to code, it allows to exclude possible issues in future.","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"},{"author":{"_account_id":10115,"name":"Mitsuhiro Tanino","email":"mitsuhiro.tanino@lycorp.co.jp","username":"mtanino"},"change_message_id":"c88e7db4896dca5019cd8afb6cf5e76aca19b730","unresolved":false,"context_lines":[{"line_number":21,"context_line":"[NOTE]"},{"line_number":22,"context_line":"This option can be applied if the destination volume is ensured"},{"line_number":23,"context_line":"to be zero cleared beforehand to avoid security issue."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Here are some results for this option."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"- Without conv\u003dsparse option"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a0c5dc1_07d4e41d","line":24,"in_reply_to":"9a0c5dc1_c43b6aea","updated":"2015-05-13 15:57:38.000000000","message":"I agree on it. We DONOT use conv\u003dsparse for the purpose of volume wiping because the volume is not cleared at all.","commit_id":"36df4a3bc308ce521abc1a042ddffe725c1d06e1"}]}