Kanagaraj M has posted comments on this change.

Change subject: engine: Add job subject entity when adding job from cli
......................................................................


Patch Set 1:

(1 comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterTasksSyncJob.java
Line 192:         GlusterVolumeEntity vol = 
getVolumeDao().getByName(cluster.getId(), volumeName);
Line 193: 
Line 194:         VdcReturnValueBase result = 
getBackend().runInternalAction(VdcActionType.AddInternalJob,
Line 195:                 new 
AddInternalJobParameters(ExecutionMessageDirector.resolveJobMessage(actionType, 
getGlusterTaskUtils().getMessageMap(cluster, task)),
Line 196:                         actionType, true, 
VdcObjectType.GlusterVolume, vol.getId()) );
vol could be null, if its not yet imported to engine
Line 197:         if (!result.getSucceeded()) {
Line 198:             //log and return
Line 199:             throw new VdcBLLException(result.getFault().getError());
Line 200:         }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7996c5b2c52f276f40777fa485cf1a70ba76f381
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to