From Yaniv Bronhaim <[email protected]>: Yaniv Bronhaim has submitted this change and it was merged. ( https://gerrit.ovirt.org/80427 )
Change subject: vm: Use time.Clock for timing extend flow ...................................................................... vm: Use time.Clock for timing extend flow Use the new clock facility to record detailed timing of the extension process. We record: - total: the time from detecting that a drive needs extension until the guest can see the new size. When extending during live storage migration, this time includes the time to extend the replica and the volume. - extend-replica: The time to extend the replica volume during live storage migration. - refresh-replica: The time to refresh the replica volume after it was extended during live storage migration. - extend-volume: The time to extend the drive volume. - refresh-volume: The time to refresh the drive volume after it was extended. Here is an example log during live storage migration from block storage to block storage - both the volume and the replica are extended: 2017-07-31 02:39:33,390+0300 INFO (mailbox-hsm/0) [virt.vm] (vmId='338f0c16-286a-4250-937a-e65f1fdaeedd') Extend volume c951edf1-16aa-4a8e-9197-cdec7c893dca completed <Clock(total=6.30, extend-replica=2.08, refresh-replica=0.13, extend-volume=3.98, refresh-volume=0.11)> (vm:1259) Here is an example log during live storage migration from block storage to file storage - only the volume is extended: 2017-07-31 02:43:46,356+0300 INFO (mailbox-hsm/1) [virt.vm] (vmId='338f0c16-286a-4250-937a-e65f1fdaeedd') Extend volume cad101f2-dedd-4b5e-a3db-49370ff3813a completed <Clock(total=4.36, extend-volume=4.18, refresh-volume=0.18)> (vm:1259) Here is an example log during live storage migration from file storage to block storage - only the replica is extended: 2017-07-31 02:59:22,390+0300 INFO (mailbox-hsm/1) [virt.vm] (vmId='338f0c16-286a-4250-937a-e65f1fdaeedd') Extend replica 37b8493e-f9d3-4cbe-bc2a-783303dd1a2e completed <Clock(total=2.31, extend-replica=2.12, refresh-replica=0.19)> (vm:1198) Change-Id: I23a5cdb8197fd081e28a39de23f38931c0f79e41 Bug-Url: https://bugzilla.redhat.com/1461536 Signed-off-by: Nir Soffer <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 42 insertions(+), 24 deletions(-) Approvals: Adam Litke: Looks good to me, but someone else must approve Nir Soffer: Verified Yaniv Bronhaim: Looks good to me, approved; Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Francesco Romani: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/80427 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23a5cdb8197fd081e28a39de23f38931c0f79e41 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
