From 24ebbf843134491d430e9d1140837f30a111c0be Mon Sep 17 00:00:00 2001 From: Emma Foley Date: Tue, 27 Feb 2018 18:07:08 +0000 Subject: [PATCH] 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 --- baro_tests/collectd.py | 6 +++--- 1 file 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( -- 2.16.6