Dan Kenigsberg has submitted this change and it was merged. Change subject: image: Make progress report interval configurable ......................................................................
image: Make progress report interval configurable Previously we logged a progress report every 60 seconds during qemu-img operations. This seems too long, in particular when watching the logs during testing. Decrease the interval to 30 seconds, and make it configurable (irs:progress_interval). Here is an example, using 10 seconds progress interval: 2015-12-14 00:15:39,042 waiting for qemu-img operation to complete 2015-12-14 00:15:49,045 qemu-img operation progress: 9.11% 2015-12-14 00:15:59,041 qemu-img operation progress: 20.25% 2015-12-14 00:16:09,042 qemu-img operation progress: 29.36% 2015-12-14 00:16:19,043 qemu-img operation progress: 39.48% 2015-12-14 00:16:29,043 qemu-img operation progress: 50.62% 2015-12-14 00:16:39,039 qemu-img operation progress: 60.74% 2015-12-14 00:16:49,038 qemu-img operation progress: 69.85% 2015-12-14 00:16:59,042 qemu-img operation progress: 79.97% 2015-12-14 00:17:09,044 qemu-img operation progress: 91.11% 2015-12-14 00:17:17,828 qemu-img operation progress: 100.0% 2015-12-14 00:17:17,829 qemu-img operation has completed Change-Id: Ice27c49cc4902c24325e0d96eadec50532dfe40f Signed-off-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/50405 Continuous-Integration: Jenkins CI Reviewed-by: Adam Litke <ali...@redhat.com> --- M lib/vdsm/config.py.in M vdsm/storage/image.py 2 files changed, 8 insertions(+), 2 deletions(-) Approvals: Adam Litke: Looks good to me, approved Nir Soffer: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50405 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ice27c49cc4902c24325e0d96eadec50532dfe40f Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches