Liron Aravot has uploaded a new change for review.

Change subject: core: correct ProcessOvfUpdateForStorageDomainCommand lock 
properties
......................................................................

core: correct ProcessOvfUpdateForStorageDomainCommand lock properties

ProcessOvfUpdateForStorageDomainCommand lock is released always on the
execute phase prior to change I54cc929937d3b14dc49a2edab9e763cbe603a9c3
- either by the child command or because the created tasks aren't added
to the command task
list.

This patch changes the applyLockProperties method to state that
explictly.

Change-Id: I938e8a0ce74bb931635948210f26a1f451fbe815
Signed-off-by: Liron Aravot <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/21/35821/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
index fe92a66..44b15ab 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
@@ -78,7 +78,7 @@
 
     @Override
     protected LockProperties applyLockProperties(LockProperties 
lockProperties) {
-        return lockProperties.withScope(Scope.Command);
+        return lockProperties.withScope(Scope.Execution);
     }
 
     @Override


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I938e8a0ce74bb931635948210f26a1f451fbe815
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Liron Aravot <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to