Change in ovirt-engine[master]: restapi: correct status code for add brick when volume is down.

kmayilsa at redhat.com kmayilsa at redhat.com
Tue Dec 10 05:38:46 UTC 2013


Kanagaraj M has submitted this change and it was merged.

Change subject: restapi: correct status code for add brick when volume is down.
......................................................................


restapi: correct status code for add brick when volume is down.

In Method resolveCreatedList for populating entity,
populate method is used instead of doPopulate.
As doPopulate doesnot return proper status code
when the volume is down.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1018904
Change-Id: Ic6c680383ea75d898a6496c86daabe38630f9d18
Signed-off-by: ndarshan <dnarayan at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResourceTest.java
2 files changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Kanagaraj M: Looks good to me, approved
  Darshan N: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic6c680383ea75d898a6496c86daabe38630f9d18
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list