Tomas Golembiovsky has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info ......................................................................
Patch Set 1: Verified+1 (1 comment) https://gerrit.ovirt.org/#/c/57418/1/lib/api/vdsm-api.yml File lib/api/vdsm-api.yml: Line 7095: type: string Line 7096: Line 7097: - defaultvalue: False Line 7098: description: Request only names, not complete info about hosts Line 7099: name: names_only This is a misnomer actually, because we are returning also VM state and not just the name. Should I rename it somehow, or is this OK? Any suggestions? Line 7100: added: '4.0' Line 7101: type: boolean Line 7102: Line 7103: - defaultvalue: null -- To view, visit https://gerrit.ovirt.org/57418 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3dc6fc4573b2c0b1c03ed87025452e14af2fc566 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Tomas Golembiovsky <tgole...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com> Gerrit-Reviewer: Tomas Golembiovsky <tgole...@redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@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