Change in vdsm[master]: v2v: Try to be nicer to other processes

nsoffer at redhat.com nsoffer at redhat.com
Sun Jul 19 18:37:49 UTC 2015


Nir Soffer has uploaded a new change for review.

Change subject: v2v: Try to be nicer to other processes
......................................................................

v2v: Try to be nicer to other processes

Under the hood, virt-v2v is running qemu-img convert. We use high nice
and ionice values when running qemu-img convert, but we did not specify
such options when running virt-v2v.

According to bug 1243935, sanlock is loosing access to the storage while
v2v is importing a vm. Hopefully running virt-v2v in a nicer manner
would solve this issue.

Change-Id: Id163844a5976384ca9e9a75d58a581645f08a89f
Bug-Url: https://bugzilla.redhat.com/1243935
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M vdsm/v2v.py
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/17/43817/1

diff --git a/vdsm/v2v.py b/vdsm/v2v.py
index 524cbdf..cac2a0a 100644
--- a/vdsm/v2v.py
+++ b/vdsm/v2v.py
@@ -39,7 +39,7 @@
 from vdsm.define import errCode, doneCode
 from vdsm import libvirtconnection
 from vdsm.infra import zombiereaper
-from vdsm.utils import traceback, CommandPath, execCmd
+from vdsm.utils import traceback, CommandPath, execCmd, NICENESS, IOCLASS
 
 import caps
 
@@ -368,7 +368,10 @@
         cmd = self._create_command()
         logging.info('Job %r starting import', self._id)
 
+        # This is the way we run qemu-img convert jobs. virt-v2v is invoking
+        # qemu-img convert to perform the migration.
         self._proc = execCmd(cmd, sync=False, deathSignal=15,
+                             nice=NICENESS.HIGH, ionice=IOCLASS.IDLE,
                              env={'LIBGUESTFS_BACKEND': 'direct'})
 
         self._proc.blocking = True


-- 
To view, visit https://gerrit.ovirt.org/43817
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id163844a5976384ca9e9a75d58a581645f08a89f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list