diff options
author | Brenton Leanhardt <bleanhar@redhat.com> | 2015-12-08 10:38:12 -0500 |
---|---|---|
committer | Brenton Leanhardt <bleanhar@redhat.com> | 2015-12-08 10:38:12 -0500 |
commit | 23b65d4220276d3072a925a14bb9ffb7c84960e5 (patch) | |
tree | 607d85b31f7c17395500d868431d1323eac77d43 /utils/src/ooinstall/oo_config.py | |
parent | 3363ce041ae9543e148d3b53887c6b95100dfd67 (diff) | |
parent | 0056156b4b31d10d15034536b280564be38ec04b (diff) | |
download | openshift-23b65d4220276d3072a925a14bb9ffb7c84960e5.tar.gz openshift-23b65d4220276d3072a925a14bb9ffb7c84960e5.tar.bz2 openshift-23b65d4220276d3072a925a14bb9ffb7c84960e5.tar.xz openshift-23b65d4220276d3072a925a14bb9ffb7c84960e5.zip |
Merge pull request #1035 from brenton/bz1287977
Bug 1287977 - Incorrect check output from atomic-openshift-installer …
Diffstat (limited to 'utils/src/ooinstall/oo_config.py')
-rw-r--r-- | utils/src/ooinstall/oo_config.py | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/utils/src/ooinstall/oo_config.py b/utils/src/ooinstall/oo_config.py index 1be85bc1d..031b82bc1 100644 --- a/utils/src/ooinstall/oo_config.py +++ b/utils/src/ooinstall/oo_config.py @@ -14,7 +14,8 @@ PERSIST_SETTINGS = [ 'variant_version', 'version', ] -REQUIRED_FACTS = ['ip', 'public_ip', 'hostname', 'public_hostname'] +DEFAULT_REQUIRED_FACTS = ['ip', 'public_ip', 'hostname', 'public_hostname'] +PRECONFIGURED_REQUIRED_FACTS = ['hostname', 'public_hostname'] class OOConfigFileError(Exception): @@ -208,7 +209,12 @@ class OOConfig(object): for host in self.hosts: missing_facts = [] - for required_fact in REQUIRED_FACTS: + if host.preconfigured: + required_facts = PRECONFIGURED_REQUIRED_FACTS + else: + required_facts = DEFAULT_REQUIRED_FACTS + + for required_fact in required_facts: if not getattr(host, required_fact): missing_facts.append(required_fact) if len(missing_facts) > 0: |