From 25b36d6fc93762fe289d9e57c2d87a28fa21ee05 Mon Sep 17 00:00:00 2001
From: Samuel Munilla <smunilla@redhat.com>
Date: Thu, 28 Apr 2016 13:45:37 -0400
Subject: a-o-i: More friendly proxy questions

Some UI improvements to help make it more clear to the user what information we
are looking for.
---
 utils/src/ooinstall/cli_installer.py | 24 +++++++++++++++---------
 utils/test/fixture.py                |  1 +
 2 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/utils/src/ooinstall/cli_installer.py b/utils/src/ooinstall/cli_installer.py
index 0163f6d82..eacf340aa 100644
--- a/utils/src/ooinstall/cli_installer.py
+++ b/utils/src/ooinstall/cli_installer.py
@@ -520,7 +520,7 @@ def error_if_missing_info(oo_cfg):
     if missing_info:
         sys.exit(1)
 
-def get_proxy_hostname_and_excludes():
+def get_proxy_hostnames_and_excludes():
     message = """
 If a proxy is needed to reach HTTP and HTTPS traffic please enter the name below.
 This proxy will be configured by default for all processes needing to reach systems outside
@@ -532,16 +532,22 @@ https://docs.openshift.com/enterprise/latest/install_config/http_proxies.html
 """
     click.echo(message)
 
-    message = "Specify the hostname for your proxy? (ENTER for none)"
-    proxy_hostname = click.prompt(message, default='')
+    message = "Specify your http proxy ? (ENTER for none)"
+    http_proxy_hostname = click.prompt(message, default='')
 
-    if proxy_hostname:
-        message = "List any hosts that should be excluded from your proxy. (ENTER for none)"
+    message = "Specify your https proxy ? (ENTER for none)"
+    https_proxy_hostname = click.prompt(message, default=http_proxy_hostname)
+
+    if http_proxy_hostname or https_proxy_hostname:
+        message = """
+All hosts in your openshift inventory will automatically be added to the NO_PROXY value.
+Please provide any additional hosts to be added to NO_PROXY. (ENTER for none)
+"""
         proxy_excludes = click.prompt(message, default='')
     else:
         proxy_excludes = ''
 
-    return proxy_hostname, proxy_excludes
+    return http_proxy_hostname, https_proxy_hostname, proxy_excludes
 
 def get_missing_info_from_user(oo_cfg):
     """ Prompts the user for any information missing from the given configuration. """
@@ -589,9 +595,9 @@ https://docs.openshift.com/enterprise/latest/admin_guide/install/prerequisites.h
         click.clear()
 
     if not oo_cfg.settings.get('openshift_http_proxy', None):
-        proxy_hostname, proxy_excludes = get_proxy_hostname_and_excludes()
-        oo_cfg.settings['openshift_http_proxy'] = proxy_hostname
-        oo_cfg.settings['openshift_https_proxy'] = proxy_hostname
+        http_proxy, https_proxy, proxy_excludes = get_proxy_hostnames_and_excludes()
+        oo_cfg.settings['openshift_http_proxy'] = http_proxy
+        oo_cfg.settings['openshift_https_proxy'] = https_proxy
         oo_cfg.settings['openshift_no_proxy'] = proxy_excludes
         click.clear()
 
diff --git a/utils/test/fixture.py b/utils/test/fixture.py
index 4e0977e5d..e01eaebaf 100644
--- a/utils/test/fixture.py
+++ b/utils/test/fixture.py
@@ -202,6 +202,7 @@ def build_input(ssh_user=None, hosts=None, variant_num=None,
 
     inputs.append('subdomain.example.com')
     inputs.append('proxy.example.com')
+    inputs.append('proxy-private.example.com')
     inputs.append('exclude.example.com')
 
     # TODO: support option 2, fresh install
-- 
cgit v1.2.3