Fix typos

README_vagrant.md

@@ -41,7 +41,7 @@ vagrant provision
 
 Environment Variables
 ---------------------
-The following environment variables can be overriden:
+The following environment variables can be overridden:
 - ``OPENSHIFT_DEPLOYMENT_TYPE`` (defaults to origin, choices: origin, openshift-enterprise)
 - ``OPENSHIFT_NUM_NODES`` (the number of nodes to create, defaults to 2)
 

bin/cluster

@@ -314,7 +314,7 @@ This wrapper is overriding the following ansible variables:
 
   * ANSIBLE_SSH_PIPELINING:
       If not set in the environment, this wrapper will set it to `True`.
-      If you experience issue with Ansible ssh pipelining, you can disable it by explicitely set this environment variable to `False`.
+      If you experience issues with Ansible SSH pipelining, you can disable it by explicitly setting this environment variable to `False`.
 '''
     )
     parser.add_argument('-v', '--verbose', action='count',

docs/best_practices_guide.adoc

@@ -76,7 +76,7 @@ def add_person(first_name, last_name, age=None):
 
 
 === PyLint
-http://www.pylint.org/[PyLint] is used in an attempt to keep the python code as clean and as managable as possible. The build bot runs each pull request through PyLint and any warnings or errors cause the build bot to fail the pull request.
+http://www.pylint.org/[PyLint] is used in an attempt to keep the python code as clean and as manageable as possible. The build bot runs each pull request through PyLint and any warnings or errors cause the build bot to fail the pull request.
 
 '''
 [[PyLint-rules-MUST-NOT-be-disabled-on-a-whole-file]]

docs/style_guide.adoc

@@ -103,7 +103,7 @@ Ansible role variables are defined as variables contained in (or passed into) a
 [cols="2v,v"]
 |===
 | <<Role-variables-MUST-have-a-prefix-of-atleast-3-characters-See.below.for.specific.naming.rules, Rule>>
-| Role variables MUST have a prefix of atleast 3 characters. See below for specific naming rules.
+| Role variables MUST have a prefix of at least 3 characters. See below for specific naming rules.
 |===
 
 ==== Role with 3 (or more) words in the name

filter_plugins/openshift_master.py

@@ -75,7 +75,7 @@ class IdentityProviderBase(object):
 
         valid_mapping_methods = ['add', 'claim', 'generate', 'lookup']
         if self.mapping_method not in valid_mapping_methods:
-            raise errors.AnsibleFilterError("|failed unkown mapping method "
+            raise errors.AnsibleFilterError("|failed unknown mapping method "
                                             "for provider {0}".format(self.__class__.__name__))
         self._required = []
         self._optional = []

inventory/aws/hosts/ec2.ini

@@ -60,7 +60,7 @@ all_instances = False
 
 # By default, only EC2 instances in the 'running' state are returned. Specify
 # EC2 instance states to return as a comma-separated list. This
-# option is overriden when 'all_instances' is True.
+# option is overridden when 'all_instances' is True.
 # instance_states = pending, running, shutting-down, terminated, stopping, stopped
 
 # By default, only RDS instances in the 'available' state are returned.  Set

playbooks/common/openshift-cluster/upgrades/docker/upgrade_check.yml

@@ -36,7 +36,7 @@
 - set_fact:
     l_docker_upgrade: False
 
-# Make sure a docker_verison is set if none was requested:
+# Make sure a docker_version is set if none was requested:
 - set_fact:
     docker_version: "{{ avail_docker_version.stdout }}"
   when: pkg_check.rc == 0 and docker_version is not defined

playbooks/libvirt/openshift-cluster/terminate.yml

@@ -1,5 +1,5 @@
 ---
-# TODO: does not handle a non-existant cluster gracefully
+# TODO: does not handle a non-existent cluster gracefully
 
 - name: Terminate instance(s)
   hosts: localhost

playbooks/openstack/openshift-cluster/files/heat_stack.yaml

@@ -45,7 +45,7 @@ parameters:
   node_port_incoming:
     type: string
     label: Source of node port connections
-    description: Authorized sources targetting node ports
+    description: Authorized sources targeting node ports
     default: 0.0.0.0/0
 
   num_etcd:

roles/docker/tasks/udev_workaround.yml

@@ -14,7 +14,7 @@
   copy:
     content: |
       [Service]
-      #Need blank ExecStart to "clear" pre-exising one
+      #Need blank ExecStart to "clear" pre-existing one
       ExecStart=
       {{ udevw_udev_start_cmd.stdout }} --event-timeout=300
     dest: "{{ udevw_udevd_dir }}/override.conf"

roles/kube_nfs_volumes/library/partitionpool.py

@@ -60,7 +60,7 @@ options:
       - Example 3: size=200G:1,100G:2 says that the ratio of space occupied by 200 GiB
         partitions and 100GiB partition is 1:2. Therefore, on 1 TiB disk, 1/3
         (300 GiB) should be occupied by 200 GiB partitions. Only one fits there,
-        so only one is created (we always round nr. of partitions *down*). Teh rest
+        so only one is created (we always round nr. of partitions *down*). The rest
         (800 GiB) is split into eight 100 GiB partitions, even though it's more
         than 2/3 of total space - free space is always allocated as much as possible.
       - size=200G:1,100G:2 = 1x 200 GiB and 8x 100 GiB partitions (on 1 TiB disk).

roles/nuage_master/templates/nuage-openshift-monitor.j2

@@ -23,7 +23,7 @@ enterpriseAdminUser: {{ nuage_master_adminusername }}
 enterpriseAdminPassword: {{ nuage_master_adminuserpasswd }}
 # Location where logs should be saved
 log_dir: {{ nuage_mon_rest_server_logdir }}
-# Monitor rest server paramters
+# Monitor rest server parameters
 # Logging level for the nuage openshift monitor
 # allowed options are: 0 => INFO, 1 => WARNING, 2 => ERROR, 3 => FATAL
 logLevel: {{ nuage_mon_log_level }}

roles/openshift_facts/library/openshift_facts.py

@@ -59,7 +59,7 @@ def migrate_docker_facts(facts):
         facts['docker']['hosted_registry_network'] = facts['node'].pop('portal_net')
 
     # log_options was originally meant to be a comma separated string, but
-    # we now prefer an actual list, with backward compatability:
+    # we now prefer an actual list, with backward compatibility:
     if 'log_options' in facts['docker'] and \
             isinstance(facts['docker']['log_options'], basestring):
         facts['docker']['log_options'] = facts['docker']['log_options'].split(",")

roles/openshift_hosted_logging/tasks/cleanup_logging.yaml

@@ -46,8 +46,8 @@
 
   - name: "Remove deployer template"
     command: "{{ openshift.common.client_binary }} --config={{ mktemp.stdout }}/admin.kubeconfig delete template logging-deployer-template -n openshift"
-    register: delete_ouput
-    failed_when: delete_ouput.rc == 1 and 'exists' not in delete_ouput.stderr
+    register: delete_output
+    failed_when: delete_output.rc == 1 and 'exists' not in delete_output.stderr
 
 
   - name: Delete temp directory

roles/openshift_manageiq/tasks/main.yaml

@@ -8,7 +8,7 @@
     cp {{ openshift.common.config_base }}/master/admin.kubeconfig {{manage_iq_tmp_conf}}
   changed_when: false
 
-- name: Add Managment Infrastructure project
+- name: Add Management Infrastructure project
   command: >
     {{ openshift.common.admin_binary }} new-project
     management-infra

roles/os_firewall/library/os_firewall_manage_iptables.py

@@ -50,8 +50,8 @@ class IpTablesCreateJumpRuleError(IpTablesError):
         self.chain = chain
 
 
-# TODO: impliment rollbacks for any events that where successful and an
-# exception was thrown later. for example, when the chain is created
+# TODO: implement rollbacks for any events that were successful and an
+# exception was thrown later. For example, when the chain is created
 # successfully, but the add/remove rule fails.
 class IpTablesManager(object): # pylint: disable=too-many-instance-attributes
     def __init__(self, module):

utils/site_assets/oo-install-bootstrap.sh

@@ -67,7 +67,7 @@ pip install --no-index -f file:///$(readlink -f deps) ansible 2>&1 >> $OO_INSTAL
 # TODO: these deps should technically be handled as part of installing ooinstall
 pip install --no-index -f file:///$(readlink -f deps) click 2>&1 >> $OO_INSTALL_LOG
 pip install --no-index ./src/ 2>&1 >> $OO_INSTALL_LOG
-echo "Installation preperation done!" 2>&1 >> $OO_INSTALL_LOG
+echo "Installation preparation done!" 2>&1 >> $OO_INSTALL_LOG
 
 echo "Using `ansible --version`" 2>&1 >> $OO_INSTALL_LOG
 

utils/src/ooinstall/cli_installer.py

@@ -969,7 +969,7 @@ def upgrade(ctx, latest_minor, next_major):
             sys.exit(0)
         playbook = mapping['major_playbook']
         new_version = mapping['major_version']
-        # Update config to reflect the version we're targetting, we'll write
+        # Update config to reflect the version we're targeting, we'll write
         # to disk once Ansible completes successfully, not before.
         oo_cfg.settings['variant_version'] = new_version
         if oo_cfg.settings['variant'] == 'enterprise':