Dan Kenigsberg has submitted this change and it was merged.

Change subject: vm: Update LUN size when starting a vm
......................................................................


vm: Update LUN size when starting a vm

In order to handle resize of DirectLUN disks
(i.e. when a LUN volume is extended by user),
relevant data should be retrieved when invoking getVmStats.

Hence, added the following information to disks stats for
DirectLUN disks: truesize, apparentsize and lunGUID
(the sizes values already exist but are zeroed).
Using these values, the engine could keep the DB updated.

The following changes are introduced:
* hsm:
  appropriateDevice method now fetches LUN's device size
  using 'multipath -> getDeviceSize'.
* clientIF:
  For LUN devices, updated drive's truesize/apparentsize
  using hsm's getVolumeSize method.
* vm:
  _getDiskStats -> in order to identify each LUN
  (e.g. on engine side), added 'lunGUID' to dStats
  when vmDrive.GUID exists (applicable for DirectLUN disks).

Change-Id: Ia4988212f7f96078e774d2a4e7b5cd1681383cb0
Bug-Url: https://bugzilla.redhat.com/1026868
Signed-off-by: Daniel Erez <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/24235
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Federico Simoncelli <[email protected]>
---
M vdsm/clientIF.py
M vdsm/storage/hsm.py
M vdsm/vm.py
3 files changed, 13 insertions(+), 0 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Federico Simoncelli: Looks good to me, approved
  Daniel Erez: Verified



-- 
To view, visit http://gerrit.ovirt.org/24235
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia4988212f7f96078e774d2a4e7b5cd1681383cb0
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to