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