From Yaniv Bronhaim <[email protected]>: Yaniv Bronhaim has submitted this change and it was merged. ( https://gerrit.ovirt.org/80428 )
Change subject: vm: Log changes in drive extension info ...................................................................... vm: Log changes in drive extension info Each time drives' capacity, allocation, or physical change, log a debug log with the new values. This log will allow easy debugging when lower level component misbehave. Here is an example log when starting a VM: 2017-08-02 02:54:24,468+0300 DEBUG (periodic/0) [virt.vm] (vmId='f33a7fbc-3dab-448c-bb41-204eccee70ba') Extension info for drive sda volume 13b40805-a43b-4c0e-9a0b-0c03a2e4ecbb: BlockInfo(capacity=8589934592L, allocation=0L, physical=1073741824L) (vm:1057) The allocation value is 0, since qemu did not write anything yet to this drive. Once qemu write to the drive, we get the correct allocation: 2017-08-02 02:54:38,473+0300 DEBUG (periodic/1) [virt.vm] (vmId='f33a7fbc-3dab-448c-bb41-204eccee70ba') Extension info for drive sda volume 13b40805-a43b-4c0e-9a0b-0c03a2e4ecbb: BlockInfo(capacity=8589934592L, allocation=2490368L, physical=1073741824L) (vm:1057) Change-Id: I68701bcb1b83295f346139b2aed64b9221a4ee44 Bug-Url: https://bugzilla.redhat.com/1461536 Signed-off-by: Nir Soffer <[email protected]> --- M vdsm/virt/vm.py M vdsm/virt/vmdevices/storage.py 2 files changed, 20 insertions(+), 2 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/80428 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68701bcb1b83295f346139b2aed64b9221a4ee44 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]
