Francesco Romani has posted comments on this change. Change subject: hostdev: report totalvfs ......................................................................
Patch Set 4: Code-Review-1 (1 comment) sorry for late noticing, spotted a possible schema/code inconsistency. Looks good otherwise. http://gerrit.ovirt.org/#/c/35891/4/vdsm/rpc/vdsmapi-schema.json File vdsm/rpc/vdsmapi-schema.json: Line 3760: # Line 3761: # @vendor: String representation of the vendor Line 3762: # Line 3763: # @totalvfs: Number of available virtual functions (if sriov Line 3764: # capable) but there is a code path that would make this key absent, so should this be made #optional? Line 3765: # Line 3766: # Since: 4.17.0 Line 3767: ## Line 3768: {'type': 'HostDeviceParams', -- To view, visit http://gerrit.ovirt.org/35891 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic54228deb64db200615ef8e2ba8f51e449e68022 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Martin Betak <mbe...@redhat.com> Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches