Change in ovirt-log-collector[ovirt-log-collector-3.2]: Use max=1000 option return results when calling the oVirt API.

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Nov 4 11:15:03 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: Use max=1000 option return results when calling the oVirt API.
......................................................................


Use max=1000 option return results when calling the oVirt API.

When calling the API the LC does not provide a max value, limiting
the results to 100 by default. For environments with 100+ DCs,
Clusters or hosts this leads to incomplete LCs being collected.

This change introduces the max=1000 option to each .list() call to allow
a greater number of results to be returned.
(cherry picked from b1b4b2a79ba91239dfa7e1ba1fb49f5e83321f5a)

Change-Id: I0c9f61938df1626b1d81e4c0320b5bca10614f34
Bug-Url: https://bugzilla.redhat.com/1025409
Signed-off-by: Lee Yarwood <lyarwood at redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M .gitignore
M src/helper/hypervisors.py
2 files changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Kiril Nesenko: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/20779
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0c9f61938df1626b1d81e4c0320b5bca10614f34
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: ovirt-log-collector-3.2
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Humble Devassy Chirammal <hchiramm at redhat.com>
Gerrit-Reviewer: Keith Robertson <kroberts at redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Lee Yarwood <lyarwood at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list