From Douglas Schilling Landgraf <dougsland@redhat.com>:

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: CollectorBase: fix list index out of range ...................................................................... CollectorBase: fix list index out of range Re-ordered tests on an list avoiding an index out of range exception if the sosreport filename is not in the output of sosreport call. The IndexError was used to detect an error in sosreport call instead of properly handling a parsing error. Added exception handling for this cases and made it a critical error since this means the underlying sosreport is having troubles. Change-Id: I40235242bf4eabd4b1cd9599eb45cb124e182df3 Bug-Url: https://bugzilla.redhat.com/1460734 Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com> --- M src/__main__.py 1 file changed, 18 insertions(+), 12 deletions(-) Approvals: Douglas Schilling Landgraf: Looks good to me, approved Sandro Bonazzola: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/78534 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40235242bf4eabd4b1cd9599eb45cb124e182df3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland@redhat.com> Gerrit-Reviewer: Ido Rosenzwig <irosenzw@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde@redhat.com> Gerrit-Reviewer: Rafael Martins <rafael@rafaelmartins.eng.br> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi@redhat.com> Gerrit-Reviewer: Yuval Turgeman <yturgema@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>