Roy Golan has uploaded a new change for review.

Change subject: core: update isInitialized in the same tx
......................................................................

core: update isInitialized in the same tx

a part of a CreateVmVDSCommand is to upadate the isInitialized db
column.
Trying to set isInitialized with a new transaction will always
fail since the current (parent)  TX is locking vm_static already by
other pending commits that have been made.

Change-Id: I65557ca425eebd91c78ce4a4001e4ba4532b56d6
Bug-Url: https://bugzilla.redhat.com/890953
Signed-off-by: Roy Golan <[email protected]>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/34/12134/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
index 16e2c0f..e5e788f 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
@@ -165,7 +165,7 @@
     }
 
     private void saveSetInitializedToDb(final Guid vmId) {
-        TransactionSupport.executeInScope(TransactionScopeOption.RequiresNew,
+        TransactionSupport.executeInScope(TransactionScopeOption.Required,
                 new TransactionMethod<Void>() {
                     @Override
                     public Void runInTransaction() {


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

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

Reply via email to