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

Reply via email to