Ravi Nori has posted comments on this change.

Change subject: core : Persist AsyncTask before submitting to vdsm
......................................................................


Patch Set 20: (1 inline comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateCloneOfTemplateCommand.java
Line 51:                 .getStoragePoolId().getValue() : Guid.Empty;
Line 52: 
Line 53:         VDSReturnValue vdsReturnValue = null;
Line 54:         try {
Line 55:             Guid taskId = 
persistAsyncTaskPlaceHolder(VdcActionType.AddVmFromTemplate);
I think entity ids can be updated when the row is updated with vdsm id with out 
changing the functionality. Correct me if I am wrong
Line 56: 
Line 57:             vdsReturnValue = Backend
Line 58:                     .getInstance()
Line 59:                     .getResourceManager()


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc5a548cb53172e6399f4334f24163dd855291ab
Gerrit-PatchSet: 20
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to