Nir Soffer has posted comments on this change. Change subject: lvm: Fix error handling when resizing lvs ......................................................................
Patch Set 5: Verified+1 Reproduced both positive and negative reduce flows. To reproduce negative reduce flow, call lvm.reduceLV() twice in BlockVolume.reduce - we get this log: 73cc748f-3d1f-4a8b-bc48-d3ec723bfdfc::DEBUG::2016-08-16 17:45:52,600::lvm::1190::Storage.LVM::(_resizeLV) LV 5f35b5c0-17d7-4475-9125-e97f1cdb06f9/67fd6a68-a063-4680-b7f9-282 6523340f9 already reduced (extents=9, requested=9) We don't expect this in real code, just checking that error handling works as expected. -- To view, visit https://gerrit.ovirt.org/62338 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id557df0b809f7ee141f2e9ea0e1c83084c71bb49 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org