Timothy Asir has posted comments on this change. Change subject: Add remoteServer parameter into gluster volumes list ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/36783/2/vdsm/rpc/vdsmapi-gluster-schema.json File vdsm/rpc/vdsmapi-gluster-schema.json: Line 640: # Get the list of Gluster volumes Line 641: # Line 642: # @volumeName: Gluster volume name Line 643: # Line 644: # @remoteServer: Remote server name > I can see in the code that it is optional. Please make it optional in schem Done Line 645: # Line 646: # Returns: Line 647: # List of Gluster volumes Line 648: # -- To view, visit http://gerrit.ovirt.org/36783 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0039cc0de9fbef6a7125df7b2c1dac14aa12de1 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Timothy Asir <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Bala.FA <[email protected]> Gerrit-Reviewer: Darshan N <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: Timothy Asir <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
