Piotr Kliczewski has posted comments on this change.

Change subject: gluster: fix schema
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/54065/1/lib/api/vdsmapi-gluster-schema.json
File lib/api/vdsmapi-gluster-schema.json:

Line 1455: ##
Line 1456: # @GlusterGeoRepSessionList:
Line 1457: #
Line 1458: # Gluster geo-replication session list.
Line 1459: #
> @Darshan, is the volumeName an optional parameter here?
It was not there in type data section but only in description. Please let me 
know whether it should be here or not.
Line 1460: # @sessions:     The session details of all sessions
Line 1461: #
Line 1462: # Since: 4.17.0
Line 1463: ##


-- 
To view, visit https://gerrit.ovirt.org/54065
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I070c9cd7509918a58cc70d4e66dae68b7d8cae0f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Darshan N <dnara...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to