)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1c3e1817564bd3dba07ac18c16f47bfbc62ae08d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     yenai \u003cyenai@cmss.chinamobile.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-11-12 14:44:40 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extends RBD volume if required"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_01ec80b9","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":30},"updated":"2019-11-12 13:38:16.000000000","message":"nit: Fix create RBD volume from encrypted snapshot","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"55e5d267d44e3f746ed8be3fdb6ffb801291883e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     yenai \u003cyenai@cmss.chinamobile.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-11-12 14:44:40 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extends RBD volume if required"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_12b3620c","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":30},"in_reply_to":"3fa7e38b_01ec80b9","updated":"2019-11-13 09:32:46.000000000","message":"Done","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1c3e1817564bd3dba07ac18c16f47bfbc62ae08d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extends RBD volume if required"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_c1b8a8c8","line":9,"range":{"start_line":9,"start_character":57,"end_line":9,"end_character":68},"updated":"2019-11-12 13:38:16.000000000","message":"nit: volume\u0027s size","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"55e5d267d44e3f746ed8be3fdb6ffb801291883e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extends RBD volume if required"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_32b69efb","line":9,"range":{"start_line":9,"start_character":57,"end_line":9,"end_character":68},"in_reply_to":"3fa7e38b_c1b8a8c8","updated":"2019-11-13 09:32:46.000000000","message":"Done","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1c3e1817564bd3dba07ac18c16f47bfbc62ae08d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Extends RBD volume if required"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."},{"line_number":13,"context_line":"This patch fix it just by checking the size from cinder DB, which is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_81b2b0e4","line":10,"range":{"start_line":10,"start_character":51,"end_line":10,"end_character":55},"updated":"2019-11-12 13:38:16.000000000","message":"nit: from","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"55e5d267d44e3f746ed8be3fdb6ffb801291883e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Extends RBD volume if required"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."},{"line_number":13,"context_line":"This patch fix it just by checking the size from cinder DB, which is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_d2ccea8b","line":10,"range":{"start_line":10,"start_character":51,"end_line":10,"end_character":55},"in_reply_to":"3fa7e38b_81b2b0e4","updated":"2019-11-13 09:32:46.000000000","message":"Done","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1c3e1817564bd3dba07ac18c16f47bfbc62ae08d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."},{"line_number":13,"context_line":"This patch fix it just by checking the size from cinder DB, which is"},{"line_number":14,"context_line":"the same with cloning volume."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_c1dd0814","line":12,"range":{"start_line":11,"start_character":34,"end_line":12,"end_character":12},"updated":"2019-11-12 13:38:16.000000000","message":"nit: Maybe replace with this part with:\n\nmaking the volume unusable by VMs.\n\nThe message we see in dmesg is \"bad geometry: block count xxx exceeds size of device.\"","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"55e5d267d44e3f746ed8be3fdb6ffb801291883e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027size"},{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."},{"line_number":13,"context_line":"This patch fix it just by checking the size from cinder DB, which is"},{"line_number":14,"context_line":"the same with cloning volume."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_f2c7a664","line":12,"range":{"start_line":11,"start_character":34,"end_line":12,"end_character":12},"in_reply_to":"3fa7e38b_c1dd0814","updated":"2019-11-13 09:32:46.000000000","message":"Done","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1c3e1817564bd3dba07ac18c16f47bfbc62ae08d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."},{"line_number":13,"context_line":"This patch fix it just by checking the size from cinder DB, which is"},{"line_number":14,"context_line":"the same with cloning volume."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If2b189f12d10b0b108430308f54e98b11ffef113"},{"line_number":17,"context_line":"Closes-Bug: #1852168"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_e1d7e4ed","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":28},"updated":"2019-11-12 13:38:16.000000000","message":"nit: skipping resizing when it\u0027s the same size.","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"55e5d267d44e3f746ed8be3fdb6ffb801291883e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"in cinder DB. So doing resize when create a volume form snapshot will"},{"line_number":11,"context_line":"reduce the capacity of the volume which will cause the volume can\u0027t be"},{"line_number":12,"context_line":"used in VMs."},{"line_number":13,"context_line":"This patch fix it just by checking the size from cinder DB, which is"},{"line_number":14,"context_line":"the same with cloning volume."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If2b189f12d10b0b108430308f54e98b11ffef113"},{"line_number":17,"context_line":"Closes-Bug: #1852168"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_b2c12e51","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":28},"in_reply_to":"3fa7e38b_e1d7e4ed","updated":"2019-11-13 09:32:46.000000000","message":"Done","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"f6d2c182959ae54bc90840dc1019f5e1a9939ea1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix create RBD volume from encrypted snapshot"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027s"},{"line_number":10,"context_line":"size in cinder DB. So doing resize when create a volume from"},{"line_number":11,"context_line":"snapshot will reduce the capacity of the volume and make the volume"},{"line_number":12,"context_line":"unusable by VMs. The message we see in dmesg is \"bad geometry:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3fa7e38b_e92c6db3","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":12},"updated":"2020-01-21 14:26:27.000000000","message":"Encrypted","commit_id":"fe542d096c2e59a66dd77365231ffee2ee774b27"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"c41731476d5482af8fb7b3314fb63f58bc48d887","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix create RBD volume from encrypted snapshot"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"En encrypted Ceph RBD volume is a little larger than the volume\u0027s"},{"line_number":10,"context_line":"size in cinder DB. So doing resize when create a volume from"},{"line_number":11,"context_line":"snapshot will reduce the capacity of the volume and make the volume"},{"line_number":12,"context_line":"unusable by VMs. The message we see in dmesg is \"bad geometry:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"df33271e_2674fcce","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":12},"in_reply_to":"3fa7e38b_e92c6db3","updated":"2020-04-09 03:17:00.000000000","message":"Done","commit_id":"fe542d096c2e59a66dd77365231ffee2ee774b27"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"1dd3eb3c22b6a108ed68e412083998bd512807ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b60bb5f7_33ae0553","updated":"2021-11-11 16:29:18.000000000","message":"fixed : https://review.opendev.org/c/openstack/cinder/+/784623/","commit_id":"8ac818ccb932033bece6df843974f9d25c26d3d7"}],"cinder/volume/drivers/rbd.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1c3e1817564bd3dba07ac18c16f47bfbc62ae08d","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_8184f00c","line":999,"updated":"2019-11-12 13:38:16.000000000","message":"This patch should only fix this specific issue, but after reading your bug report I think we should also fix (in a different patch) ALL similar issues related to it, since at least the following operations on encrypted resources:\n\n- Create bigger volume from snapshot\n- Clone volume to bigger volume\n- Extend volume\n\nResult in volumes with less available space than requested, since the encryption overhead will take up part of the available volume space instead of having a slightly larger volume in RBD so the user can have the exact size that was requested.\n\nDo you know if we have a bug for these cases?","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"f0430b1dc3566b7b702af5559a826017bd4bf3d0","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_dd9e1668","line":999,"in_reply_to":"3fa7e38b_0d04927d","updated":"2019-11-15 10:06:05.000000000","message":"Yes, this must be done!\nThis patch is just avoiding a known problem(not fix the root).","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"55e5d267d44e3f746ed8be3fdb6ffb801291883e","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_eda6eb0d","line":999,"in_reply_to":"3fa7e38b_8184f00c","updated":"2019-11-13 09:32:46.000000000","message":"Create bigger volume from snapshot -\u003e No error.\nClone volume to bigger volume      -\u003e No error.\nExtend volume                      -\u003e No error.\nCreate a bigger volume and extend volume works fine in my test env. I think no problem is reasonable, because they only increase the space (provided that luks does not take up more than 1024M space).","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"a880b8f52564e4034cd032a8d9b5934a6e7d92f7","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_0d04927d","line":999,"in_reply_to":"3fa7e38b_eda6eb0d","updated":"2019-11-14 10:42:12.000000000","message":"When I mentioned issues I didn\u0027t mean that they would fail or return an error, but they just have less space than they should, because there\u0027s a part used by the encryption metadata.  We need to find a way to make this consistent across all backends so that we can migrate without problem.","commit_id":"30692ee939073ac857ad7bd6996063279994a48d"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4252f05ab7be8363f829c0473c4b04b41374f920","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_fb38ded4","line":999,"updated":"2019-11-13 14:25:24.000000000","message":"Does this really only affect encrypted volumes?  What happened before this change with non-encrypted volumes?","commit_id":"fe542d096c2e59a66dd77365231ffee2ee774b27"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"f0430b1dc3566b7b702af5559a826017bd4bf3d0","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_7ddfe237","line":999,"in_reply_to":"3fa7e38b_0f76d440","updated":"2019-11-15 10:06:05.000000000","message":"I just used _extend_if_required in patch 1, but size for snapshot is snapshot.volume_size, not snapshot.size.\n o(╥﹏╥)o","commit_id":"fe542d096c2e59a66dd77365231ffee2ee774b27"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4907a38142a0f12b07e34320949d5cb512d887c1","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_0f76d440","line":999,"in_reply_to":"3fa7e38b_db1962d1","updated":"2019-11-13 17:35:32.000000000","message":"I think the problem only happens with encrypted volumes, because it actually shrinks the volumes.\n\nFor non-encrypted volumes all that probably happens is that me make an unnecessary call.\n\nBy the way, we have the equivalent check in _extend_if_required which is used by create_cloned_volume","commit_id":"fe542d096c2e59a66dd77365231ffee2ee774b27"},{"author":{"_account_id":25837,"name":"yenai","email":"yenai_yewu@cmss.chinamobile.com","username":"yenai2008"},"change_message_id":"ed78bd083b76af8920f5e2bd659a06aeb6a9055f","unresolved":false,"context_lines":[{"line_number":996,"context_line":"        if self.configuration.rbd_flatten_volume_from_snapshot:"},{"line_number":997,"context_line":"            self._flatten(self.configuration.rbd_pool, volume.name)"},{"line_number":998,"context_line":""},{"line_number":999,"context_line":"        if volume.size !\u003d snapshot.volume_size:"},{"line_number":1000,"context_line":"            LOG.debug(\"resize volume \u0027%(dst_vol)s\u0027 from %(src_size)d to \""},{"line_number":1001,"context_line":"                      \"%(dst_size)d.\","},{"line_number":1002,"context_line":"                      {\u0027dst_vol\u0027: volume.name,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_db1962d1","line":999,"in_reply_to":"3fa7e38b_fb38ded4","updated":"2019-11-13 15:02:35.000000000","message":"No, it will affect all volumes that created from snapshot.\nBefore this change, all volume will do resize even the size is just the same.","commit_id":"fe542d096c2e59a66dd77365231ffee2ee774b27"}]}