diff options
| author |   Emma Foley <emma.l.foley@intel.com> | 2018-02-27 18:07:08 +0000 |
|---|---|---|
| committer | Emma Foley <emma.l.foley@intel.com> | 2018-02-27 18:13:09 +0000 |
| commit | 24ebbf843134491d430e9d1140837f30a111c0be (patch) | |
| tree | 0dc05ebd31212cc26a21dc35c6e31692e2e2c97e | |
| parent | 7f4ef61d68c3ed96759f3553b2793e0ce6a7e4a5 (diff) | |
Fix syntax error in baro_test/collectd.py:mcelog_install
The version of CentOS that's used no longer matches the version that
is explicitly checked in collectd.py:mcelog_install.
As a result the code on L637-L642[1] is now being called.
This code was syntactically incorrect, however, was previously not being
called, witht eh update to the version, it is now being executed and
causing failures in deployment [2].
[1] https://github.com/opnfv/barometer/blob/c10400426df872be3236677dcb749c755f4a559c/baro_tests/collectd.py#L637-L642
[2] https://build.opnfv.org/ci/job/functest-apex-baremetal-daily-master/809/consoleText
Change-Id: I35274d4ede17510c3ae03c756eda7bc9f78b4e17
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
| -rw-r--r-- | baro_tests/collectd.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/baro_tests/collectd.py b/baro_tests/collectd.py index 304b87b8..bf310f01 100644 --- a/baro_tests/collectd.py +++ b/baro_tests/collectd.py @@ -636,9 +636,9 @@ def mcelog_install(): centos_release = node.run_cmd('uname -r') if '3.10.0-514.26.2.el7.x86_64' not in centos_release: logger.info( - 'Mcelog will not be enabled ' - + 'on node-{0}, '.format(node.get_dict()['name']) - + 'unsupported CentOS release found ({1}).'.format( + 'Mcelog will not be enabled on node-{}, ' + + ' unsupported CentOS relase found ({}).'.format( + node.get_dict()['name'], centos_release)) else: logger.info( |