Arik Hadas has uploaded a new change for review.

Change subject: do not use OOP for padding snapshot's memory volume
......................................................................

do not use OOP for padding snapshot's memory volume

Change-Id: I2a94354e188019f3afd209633979ec5a5b35293b
Signed-off-by: Arik Hadas <[email protected]>
---
M vdsm/virt/vm.py
1 file changed, 3 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/38/26538/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 6711bc6..b8ce533 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -3543,15 +3543,9 @@
                     '_srcDomXML': self._dom.XMLDesc(0),
                     'elapsedTimeOffset': time.time() - self._startTime}
 
-        def _padMemoryVolume(memoryVolPath, sdUUID):
-            sdType = sd.name2type(
-                self.cif.irs.getStorageDomainInfo(sdUUID)['info']['type'])
-            if sdType in sd.FILE_DOMAIN_TYPES:
-                if sdType == sd.NFS_DOMAIN:
-                    oop.getProcessPool(sdUUID).fileUtils. \
-                        padToBlockSize(memoryVolPath)
-                else:
-                    fileUtils.padToBlockSize(memoryVolPath)
+        def _padMemoryVolume(memoryVolPath):
+            if not utils.isBlockDevice(memoryVolPath):
+                fileUtils.padToBlockSize(memoryVolPath)
 
         snap = xml.dom.minidom.Element('domainsnapshot')
         disks = xml.dom.minidom.Element('disks')


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2a94354e188019f3afd209633979ec5a5b35293b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to