Browse Source

Merge pull request #2695 from smunilla/BZ1383961

Update installation summary for etcd members
Scott Dodson 8 years ago
parent
commit
f232cfd42b
2 changed files with 6 additions and 10 deletions
  1. 2 10
      utils/src/ooinstall/cli_installer.py
  2. 4 0
      utils/src/ooinstall/oo_config.py

+ 2 - 10
utils/src/ooinstall/cli_installer.py

@@ -108,11 +108,6 @@ a high-availability (HA) deployment. If you choose an HA deployment, then you
 are prompted to identify a *separate* system to act as the load balancer for
 are prompted to identify a *separate* system to act as the load balancer for
 your cluster once you define all masters and nodes.
 your cluster once you define all masters and nodes.
 
 
-If only one master is specified, an etcd instance is embedded within the
-OpenShift master service to use as the datastore. This can be later replaced
-with a separate etcd instance, if required. If multiple masters are specified,
-then a separate etcd cluster is configured with each master serving as a member.
-
 Any masters configured as part of this installation process are also
 Any masters configured as part of this installation process are also
 configured as nodes. This enables the master to proxy to pods
 configured as nodes. This enables the master to proxy to pods
 from the API. By default, this node is unschedulable, but this can be changed
 from the API. By default, this node is unschedulable, but this can be changed
@@ -253,11 +248,8 @@ def print_host_summary(all_hosts, host):
             click.echo("  - Load Balancer (Preconfigured)")
             click.echo("  - Load Balancer (Preconfigured)")
         else:
         else:
             click.echo("  - Load Balancer (HAProxy)")
             click.echo("  - Load Balancer (HAProxy)")
-    if host.is_master():
-        if host.is_etcd_member(all_hosts):
-            click.echo("  - Etcd Member")
-        else:
-            click.echo("  - Etcd (Embedded)")
+    if host.is_etcd():
+        click.echo("  - Etcd")
     if host.is_storage():
     if host.is_storage():
         click.echo("  - Storage")
         click.echo("  - Storage")
     if host.new_host:
     if host.new_host:

+ 4 - 0
utils/src/ooinstall/oo_config.py

@@ -120,6 +120,10 @@ class Host(object):
     def is_storage(self):
     def is_storage(self):
         return 'storage' in self.roles
         return 'storage' in self.roles
 
 
+    def is_etcd(self):
+        """ Does this host have the etcd role """
+        return 'etcd' in self.roles
+
     def is_etcd_member(self, all_hosts):
     def is_etcd_member(self, all_hosts):
         """ Will this host be a member of a standalone etcd cluster. """
         """ Will this host be a member of a standalone etcd cluster. """
         if not self.is_master():
         if not self.is_master():