Change in ovirt-engine[master]: gluster: SP to update gluster server service
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(a)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(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server