summaryrefslogtreecommitdiffstats
path: root/utils/src
diff options
context:
space:
mode:
authorDevan Goodwin <dgoodwin@redhat.com>2015-11-30 15:20:54 -0400
committerDevan Goodwin <dgoodwin@redhat.com>2015-11-30 15:23:16 -0400
commitcdd9aa7543f869aab5914fb816a66ed0debb0930 (patch)
treeb0a8f55f44b21df29d931b4da15638aced6ad54e /utils/src
parent4872c5e1d298cba9901e2ecf207580b7d4fdece7 (diff)
downloadopenshift-cdd9aa7543f869aab5914fb816a66ed0debb0930.tar.gz
openshift-cdd9aa7543f869aab5914fb816a66ed0debb0930.tar.bz2
openshift-cdd9aa7543f869aab5914fb816a66ed0debb0930.tar.xz
openshift-cdd9aa7543f869aab5914fb816a66ed0debb0930.zip
Error out if no load balancer specified.
Diffstat (limited to 'utils/src')
-rw-r--r--utils/src/ooinstall/cli_installer.py24
1 files changed, 10 insertions, 14 deletions
diff --git a/utils/src/ooinstall/cli_installer.py b/utils/src/ooinstall/cli_installer.py
index 9a447406f..249a8a7d9 100644
--- a/utils/src/ooinstall/cli_installer.py
+++ b/utils/src/ooinstall/cli_installer.py
@@ -201,11 +201,11 @@ def collect_master_lb(hosts):
"""
message = """
Setting up High Availability Masters requires a load balancing solution.
-Please provide a host that will be configured as a proxy. This can either be
-an existing load balancer configured to balance all masters on port 8443 or a
-new host that will have HAProxy installed on it.
+Please provide a the FQDN of a host that will be configured as a proxy. This
+can be either an existing load balancer configured to balance all masters on
+port 8443 or a new host that will have HAProxy installed on it.
-If the host provided does is not yet configured a reference haproxy load
+If the host provided does is not yet configured, a reference haproxy load
balancer will be installed. It's important to note that while the rest of the
environment will be fault tolerant this reference load balancer will not be.
It can be replaced post-installation with a load balancer with the same
@@ -318,25 +318,21 @@ def check_hosts_config(oo_cfg, unattended):
if len(masters) > 1:
master_lb = [host for host in oo_cfg.hosts if host.master_lb]
if len(master_lb) > 1:
- click.echo('More than one Master load balancer specified. Only one is allowed.')
+ click.echo('ERROR: More than one Master load balancer specified. Only one is allowed.')
sys.exit(1)
elif len(master_lb) == 1:
if master_lb[0].master or master_lb[0].node:
- click.echo('The Master load balancer is configured as a master or node. Please correct this.')
+ click.echo('ERROR: The Master load balancer is configured as a master or node. Please correct this.')
sys.exit(1)
- # Check for another host with same connect_to?
else:
message = """
-WARNING: No master load balancer specified in config. If you proceed you will
-need to provide a load balancing solution of your choice to balance the
-API (port 8443) on all master hosts.
+ERROR: No master load balancer specified in config. You must provide the FQDN
+of a load balancer to balance the API (port 8443) on all Master hosts.
https://docs.openshift.org/latest/install_config/install/advanced_install.html#multiple-masters
"""
- if unattended:
- click.echo(message)
- else:
- confirm_continue(message)
+ click.echo(message)
+ sys.exit(1)
nodes = [host for host in oo_cfg.hosts if host.node]
# TODO: This looks a little unsafe, maybe look for dedicated nodes only: