diff options
| author | Mehdi Abaakouk <sileht@redhat.com> | 2017-10-12 16:58:26 +0200 |
|---|---|---|
| committer | Mehdi Abaakouk <sileht@redhat.com> | 2017-11-30 08:34:27 +0100 |
| commit | 12b3cc63b5b4422926535c7e0d5abfb19f31da53 (patch) | |
| tree | abe399fdfed8572801722c02e12855b343e00513 | |
| parent | fa2cb95cf92fb6b0e0339f248a2285c28357b80a (diff) | |
| download | ceilometer-12b3cc63b5b4422926535c7e0d5abfb19f31da53.tar.gz | |
snmp: warn if snmp call timeout
If the snmp call timeout because the host is unreachable
just warn the operator.
We can't really known what happen with the snmp error.
That's why the change just filter on "timeout" word.
Change-Id: I564b290c1e07882a7591a003c4da3ac6034ac794
(cherry picked from commit 988a2f3f2d6141711c63772024974b0da3cb646e)
| -rw-r--r-- | ceilometer/hardware/pollsters/generic.py | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/ceilometer/hardware/pollsters/generic.py b/ceilometer/hardware/pollsters/generic.py index 647cfca1..1781185e 100644 --- a/ceilometer/hardware/pollsters/generic.py +++ b/ceilometer/hardware/pollsters/generic.py @@ -161,11 +161,15 @@ class GenericHardwareDeclarativePollster(plugin_base.PollsterBase): parsed_url, i_cache[identifier])) except Exception as err: - LOG.exception(_LE('inspector call failed for %(ident)s ' - 'host %(host)s: %(err)s'), - dict(ident=identifier, - host=parsed_url.hostname, - err=err)) + msg = ('inspector call failed for %(ident)s ' + 'host %(host)s: %(err)s' % + dict(ident=identifier, + host=parsed_url.hostname, + err=err)) + if "timeout" in str(err): + LOG.warning(msg) + else: + LOG.exception(msg) return itertools.chain(*sample_iters) def generate_samples(self, host_url, data): |