Yaniv Bronhaim has submitted this change and it was merged. Change subject: qemu-img: Handle image end offset on different lines of output ......................................................................
qemu-img: Handle image end offset on different lines of output Current code assumes that qemu-img check output will place the image end offset in the second line of output but it can appear later: $ sudo qemu-img check /rhev/data-center/.../41b2b944-77f0-40d5-a681-c63e0dbb7482 No errors were found on the image. 70088/98304 = 71.30% allocated, 0.17% fragmented, 0.00% compressed clusters Image end offset: 4594466816 Update the scanner so it can find it on any line from the second until the end of output. Change-Id: Id6610baf1702c9075d8512bdb1da0e05527a0060 Bug-Url: https://bugzilla.redhat.com/1199815 Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/38438 Reviewed-by: Allon Mureinik <amure...@redhat.com> Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> --- M lib/vdsm/qemuimg.py M tests/qemuimgTests.py 2 files changed, 51 insertions(+), 11 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Adam Litke: Verified Yaniv Bronhaim: Looks good to me, approved Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38438 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6610baf1702c9075d8512bdb1da0e05527a0060 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches