summaryrefslogtreecommitdiffstats
path: root/utils
diff options
context:
space:
mode:
authorBrenton Leanhardt <bleanhar@redhat.com>2015-11-30 10:49:48 -0500
committerBrenton Leanhardt <bleanhar@redhat.com>2015-11-30 10:49:48 -0500
commitf790d7d49055b3eeb382fa4e90884f73162849cc (patch)
tree2d8f521afc2a61a46c27e9fd002fa10df65a994c /utils
parentc25735c88df27867cda0b45622106cb1119b2a7d (diff)
parent385ca96f5aaf8987820a5c7a25349ab7bedf9318 (diff)
downloadopenshift-f790d7d49055b3eeb382fa4e90884f73162849cc.tar.gz
openshift-f790d7d49055b3eeb382fa4e90884f73162849cc.tar.bz2
openshift-f790d7d49055b3eeb382fa4e90884f73162849cc.tar.xz
openshift-f790d7d49055b3eeb382fa4e90884f73162849cc.zip
Merge pull request #980 from brenton/unshedulable
Fixing 'unscheduleable' typo
Diffstat (limited to 'utils')
-rw-r--r--utils/src/ooinstall/cli_installer.py6
-rw-r--r--utils/src/ooinstall/openshift_ansible.py12
-rw-r--r--utils/test/cli_installer_tests.py14
3 files changed, 16 insertions, 16 deletions
diff --git a/utils/src/ooinstall/cli_installer.py b/utils/src/ooinstall/cli_installer.py
index d7c06745e..0b38f706c 100644
--- a/utils/src/ooinstall/cli_installer.py
+++ b/utils/src/ooinstall/cli_installer.py
@@ -90,7 +90,7 @@ change this later using etcd from Red Hat Enterprise Linux 7.
Any Masters configured as part of this installation process will also be
configured as Nodes. This is so that the Master will be able to proxy to Pods
-from the API. By default this Node will be unscheduleable but this can be changed
+from the API. By default this Node will be unschedulable but this can be changed
after installation with 'oadm manage-node'.
The OpenShift Node provides the runtime environments for containers. It will
@@ -274,8 +274,8 @@ https://docs.openshift.org/latest/install_config/install/advanced_install.html#m
if len(masters) == len(nodes):
message = """
No dedicated Nodes specified. By default, colocated Masters have their Nodes
-set to unscheduleable. Continuing at this point will label all nodes as
-scheduleable.
+set to unschedulable. Continuing at this point will label all nodes as
+schedulable.
"""
confirm_continue(message)
diff --git a/utils/src/ooinstall/openshift_ansible.py b/utils/src/ooinstall/openshift_ansible.py
index 9afc9a644..4aa60922d 100644
--- a/utils/src/ooinstall/openshift_ansible.py
+++ b/utils/src/ooinstall/openshift_ansible.py
@@ -67,10 +67,10 @@ def generate_inventory(hosts):
for node in nodes:
# TODO: Until the Master can run the SDN itself we have to configure the Masters
# as Nodes too.
- scheduleable = True
+ schedulable = True
if node in masters:
- scheduleable = False
- write_host(node, base_inventory, scheduleable)
+ schedulable = False
+ write_host(node, base_inventory, schedulable)
if not getattr(proxy, 'preconfigured', True):
base_inventory.write('\n[lb]\n')
@@ -112,7 +112,7 @@ def write_inventory_vars(base_inventory, multiple_masters, proxy):
base_inventory.write("openshift_master_cluster_public_hostname={}\n".format(proxy.public_hostname))
-def write_host(host, inventory, scheduleable=True):
+def write_host(host, inventory, schedulable=True):
global CFG
facts = ''
@@ -126,8 +126,8 @@ def write_host(host, inventory, scheduleable=True):
facts += ' openshift_public_hostname={}'.format(host.public_hostname)
# TODO: For not write_host is handles both master and nodes.
# Technically only nodes will ever need this.
- if not scheduleable:
- facts += ' openshift_scheduleable=False'
+ if not schedulable:
+ facts += ' openshift_schedulable=False'
installer_host = socket.gethostname()
if installer_host in [host.connect_to, host.hostname, host.public_hostname]:
facts += ' ansible_connection=local'
diff --git a/utils/test/cli_installer_tests.py b/utils/test/cli_installer_tests.py
index c951b6580..90b6b15a3 100644
--- a/utils/test/cli_installer_tests.py
+++ b/utils/test/cli_installer_tests.py
@@ -616,7 +616,7 @@ class AttendedCliTests(OOCliFixture):
#pylint: disable=too-many-arguments,too-many-branches
def _build_input(self, ssh_user=None, hosts=None, variant_num=None,
- add_nodes=None, confirm_facts=None, scheduleable_masters_ok=None,
+ add_nodes=None, confirm_facts=None, schedulable_masters_ok=None,
master_lb=None):
"""
Builds a CLI input string with newline characters to simulate
@@ -658,7 +658,7 @@ class AttendedCliTests(OOCliFixture):
# TODO: support option 2, fresh install
if add_nodes:
- if scheduleable_masters_ok:
+ if schedulable_masters_ok:
inputs.append('y')
inputs.append('1') # Add more nodes
i = 0
@@ -712,7 +712,7 @@ class AttendedCliTests(OOCliFixture):
inventory = ConfigParser.ConfigParser(allow_no_value=True)
inventory.read(os.path.join(self.work_dir, '.ansible/hosts'))
self.assertEquals('False',
- inventory.get('nodes', '10.0.0.1 openshift_scheduleable'))
+ inventory.get('nodes', '10.0.0.1 openshift_schedulable'))
self.assertEquals(None,
inventory.get('nodes', '10.0.0.2'))
self.assertEquals(None,
@@ -790,7 +790,7 @@ class AttendedCliTests(OOCliFixture):
add_nodes=[('10.0.0.2', False)],
ssh_user='root',
variant_num=1,
- scheduleable_masters_ok=True,
+ schedulable_masters_ok=True,
confirm_facts='y')
self._verify_get_hosts_to_run_on(mock_facts, load_facts_mock,
@@ -830,13 +830,13 @@ class AttendedCliTests(OOCliFixture):
inventory = ConfigParser.ConfigParser(allow_no_value=True)
inventory.read(os.path.join(self.work_dir, '.ansible/hosts'))
self.assertEquals('False',
- inventory.get('nodes', '10.0.0.1 openshift_scheduleable'))
+ inventory.get('nodes', '10.0.0.1 openshift_schedulable'))
self.assertEquals('False',
- inventory.get('nodes', '10.0.0.2 openshift_scheduleable'))
+ inventory.get('nodes', '10.0.0.2 openshift_schedulable'))
self.assertEquals(None,
inventory.get('nodes', '10.0.0.3'))
self.assertEquals('False',
- inventory.get('nodes', '10.0.0.4 openshift_scheduleable'))
+ inventory.get('nodes', '10.0.0.4 openshift_schedulable'))
return