Omer Frenkel has submitted this change and it was merged. Change subject: gluster: Set additional fields in ServerServices ...................................................................... gluster: Set additional fields in ServerServices Modified the GlusterServicesReturnForXmlRpc to accept server id in constructor, and populate it on the received GlusterServerService object. Also modified it to keep a static map of service name to id, and use it to popuate service id and type in the received object. Change-Id: I8e322abb58ffc5a4c9dccadd6345b68805921423 Signed-off-by: Shireesh Anjal <anjalshireesh@gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterServicesListVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterServicesReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 4 files changed, 42 insertions(+), 13 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14643 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e322abb58ffc5a4c9dccadd6345b68805921423 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Sahina Bose <sabose@redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal@redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat@redhat.com>