Federico Simoncelli has uploaded a new change for review.

Change subject: task: support scheduleJob without persistency
......................................................................

task: support scheduleJob without persistency

Change-Id: I95b3063048b910dc5994d0a264bf4791e98fef51
Signed-off-by: Federico Simoncelli <[email protected]>
---
M vdsm/storage/hsm.py
M vdsm/storage/taskManager.py
2 files changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/71/26771/1

diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index 9040d79..397bbb6 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -412,8 +412,7 @@
         self.domainStateChangeCallbacks.add(callbackFunc)
 
     def _hsmSchedule(self, name, func, *args):
-        self.taskMng.scheduleJob("hsm", self.tasksDir, vars.task,
-                                 name, func, *args)
+        self.taskMng.scheduleJob("hsm", None, vars.task, name, func, *args)
 
     def __validateLvmLockingType(self):
         """
diff --git a/vdsm/storage/taskManager.py b/vdsm/storage/taskManager.py
index e247cf8..4a7a04f 100644
--- a/vdsm/storage/taskManager.py
+++ b/vdsm/storage/taskManager.py
@@ -61,7 +61,8 @@
 
     def scheduleJob(self, type, store, task, jobName, func, *args):
         task.setTag(type)
-        task.setPersistence(store, cleanPolicy=TaskCleanType.manual)
+        if store is not None:
+            task.setPersistence(store, cleanPolicy=TaskCleanType.manual)
         task.setManager(self)
         task.setRecoveryPolicy("auto")
         task.addJob(Job(jobName, func, *args))


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

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

Reply via email to