From Sandro Bonazzola <sbonazzo(a)redhat.com>:
Sandro Bonazzola has submitted this change and it was merged.
Change subject: metrics: Prevent errors on failed collectd/fluentd installation
......................................................................
metrics: Prevent errors on failed collectd/fluentd installation
host-deploy already emitted only a warning, but otopi emits an error,
unless we ask it to ignore errors. These errors cause the engine to
emit:
Failed to install Host ${VdsName}. ${Message}
which is confusing.
Change-Id: Iec26b55ec8a04ce01a7ca6d27d20a75a248f50eb
Bug-Url:
https://bugzilla.redhat.com/1443508
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M src/plugins/ovirt-host-deploy/collectd/packages.py
M src/plugins/ovirt-host-deploy/fluentd/packages.py
2 files changed, 16 insertions(+), 20 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Jenkins CI: Passed CI tests
Yedidyah Bar David: Verified
--
To view, visit
https://gerrit.ovirt.org/76570
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iec26b55ec8a04ce01a7ca6d27d20a75a248f50eb
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Ido Rosenzwig <irosenzw(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Rafael Martins <rafael(a)rafaelmartins.eng.br>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>