Change in vdsm[ovirt-3.3]: vm: refresh raw disk before live extension

fsimonce at redhat.com fsimonce at redhat.com
Thu Nov 14 14:10:00 UTC 2013


Hello Ayal Baron,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/21274

to review the following change.

Change subject: vm: refresh raw disk before live extension
......................................................................

vm: refresh raw disk before live extension

As part of the online disk extension (raw format on block domains) flow
the leaf logical volume is extended by the SPM to the new size.
On the HSM host (where the VM is running) we need to refresh the volume
in order to update the volume segments and make the new size visible to
the qemu process.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1029501
Change-Id: I4fc3b578a865e25586a719f7c5120df12842d046
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21177
Reviewed-by: Ayal Baron <abaron at redhat.com>
---
M vdsm/vm.py
1 file changed, 6 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/74/21274/1

diff --git a/vdsm/vm.py b/vdsm/vm.py
index 1a16d53..475f6ce 100644
--- a/vdsm/vm.py
+++ b/vdsm/vm.py
@@ -4054,6 +4054,12 @@
         return {'status': doneCode, 'size': str(sizeRoundedBytes)}
 
     def _diskSizeExtendRaw(self, drive, newSizeBytes):
+        # Picking up the volume size extension
+        self.__refreshDriveVolume({
+            'domainID': drive.domainID, 'poolID': drive.poolID,
+            'imageID': drive.imageID, 'volumeID': drive.volumeID,
+        })
+
         volumeInfo = self.cif.irs.getVolumeSize(
             drive.domainID, drive.poolID, drive.imageID, drive.volumeID)
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4fc3b578a865e25586a719f7c5120df12842d046
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>


More information about the vdsm-patches mailing list