Change in ovirt-engine[master]: gluster: SP to update gluster server service

[kmayilsa at redhat.com]

Kanagaraj M has submitted this change and it was merged.

Change subject: gluster: SP to update gluster server service
......................................................................


gluster: SP to update gluster server service

Introduced a stored procedure to update the gluster server service by
server id and service id. The same is used in
ManageGlusterServiceCommand to update the details of the server services
post start/stop/restart.

Already existing stored procedure for update needs id field to be
available for update to work and same is not available during BLL
command execution. So the simple update stored procedure cannot be used
as is for the purpose of updating the server service status.

Change-Id: I2518516c93a264e4a7107ded6f24c3e338f03e4e
Signed-off-by: Shubhendu Tripathi <shtripat at redhat.com>
---
M backend/manager/dbscripts/gluster_services_sp.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ManageGlusterServiceCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/ManageGlusterServiceCommandTest.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterServerServiceDao.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterServerServiceDaoDbFacadeImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterServerServiceDaoTest.java
6 files changed, 62 insertions(+), 13 deletions(-)

Approvals:
  Shubhendu Tripathi: Verified
  Kanagaraj M: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2518516c93a264e4a7107ded6f24c3e338f03e4e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server