summaryrefslogtreecommitdiffstats
path: root/utils/src/ooinstall/openshift_ansible.py
diff options
context:
space:
mode:
authorAndrew Butcher <abutcher@afrolegs.com>2016-09-01 13:47:37 -0400
committerGitHub <noreply@github.com>2016-09-01 13:47:37 -0400
commit84c220e5e68fa8ead1f15b9af928c42c12ce853b (patch)
treed40d6b80f152ef65ae4efbf1fca1cbc7780fd266 /utils/src/ooinstall/openshift_ansible.py
parent55683a581ab891e293c5a34c074c324009f1ba37 (diff)
parent54a494e0948c270a66b63dc9420d38bb5b68ff54 (diff)
downloadopenshift-84c220e5e68fa8ead1f15b9af928c42c12ce853b.tar.gz
openshift-84c220e5e68fa8ead1f15b9af928c42c12ce853b.tar.bz2
openshift-84c220e5e68fa8ead1f15b9af928c42c12ce853b.tar.xz
openshift-84c220e5e68fa8ead1f15b9af928c42c12ce853b.zip
Merge pull request #2367 from tbielawa/FixNewerPyLintErrors
Fix PyLint errors discovered when upgrading to newer version
Diffstat (limited to 'utils/src/ooinstall/openshift_ansible.py')
-rw-r--r--utils/src/ooinstall/openshift_ansible.py5
1 files changed, 3 insertions, 2 deletions
diff --git a/utils/src/ooinstall/openshift_ansible.py b/utils/src/ooinstall/openshift_ansible.py
index 09dd1ebc4..75d26c10a 100644
--- a/utils/src/ooinstall/openshift_ansible.py
+++ b/utils/src/ooinstall/openshift_ansible.py
@@ -4,9 +4,10 @@ import socket
import subprocess
import sys
import os
+import logging
import yaml
from ooinstall.variants import find_variant
-import logging
+
installer_log = logging.getLogger('installer')
CFG = None
@@ -232,7 +233,7 @@ def load_system_facts(inventory_file, os_facts_path, env_vars, verbose=False):
os_facts_path])
installer_log.debug("Going to subprocess out to ansible now with these args: %s", ' '.join(args))
status = subprocess.call(args, env=env_vars, stdout=FNULL)
- if not status == 0:
+ if status != 0:
installer_log.debug("Exit status from subprocess was not 0")
return [], 1