Francesco Romani has posted comments on this change. Change subject: api: Move after_get_all_vm_stats hook from API to clientIF ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/59393/1/vdsm/API.py File vdsm/API.py: Line 1339 Line 1340 Line 1341 Line 1342 Line 1343 > cif is not the place to have getAllVmStats. Do we have any virt module whic Agreed. The best long term fix IMO is https://gerrit.ovirt.org/#/c/53101/ or something like that. Unfortunately, 53101 still needs some work. -- To view, visit https://gerrit.ovirt.org/59393 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibf9cdf57e7fc4e72733b195acdd84a10602df7ca Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org