Shubhendu Tripathi has posted comments on this change.

Change subject: restapi: Run activate if remove bricks task is FINISHED
......................................................................


Patch Set 3:

(1 comment)

....................................................
File 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResource.java
Line 289:                         VdcQueryType.GetGlusterVolumeById,
Line 290:                         new IdQueryParameters(asGuid(getVolumeId())),
Line 291:                         "");
Line 292: 
Line 293:         if (volume.getAsyncTask() != null && 
volume.getAsyncTask().getType() == GlusterTaskType.REMOVE_BRICK
At a time only one action can be executed on a volume. So if activate is ran on 
a for a brick say B1 on volume V1, there is no way user can fire delete on 
another brick B2 on the same volume V1.
Line 294:                 && volume.getAsyncTask().getStatus() == 
JobExecutionStatus.FINISHED) {
Line 295:             return stopMigrate(action);
Line 296:         } else {
Line 297:             Fault fault = new Fault();


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifd5731b43132943a808bccae68fbcf0efaeeef89
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@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