summaryrefslogtreecommitdiffstats
path: root/playbooks/init
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7061 from gnewson/Change-etcd-checkOpenShift Merge Robot2018-02-081-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Changing the check for the number of etcd nodes This playbook is called (via std_include.yml) when the scale up playbook for either master or etcd is called. In the scenario where you are scaling up the number of masters/etcd nodes it is feasible, if not likely, that the number of etcd nodes is not 1, 3 or 5 and this check therefore causes a scale up to fail. The two example scenarios that are driving this change are: You have a cluster with 3 master nodes (each running etcd) and one of those masters fail. The master node is removed from both the OpenShift cluster and the etcd cluster and the inventory updated to reflect the state of the cluster minus the failed master node. You would then run the scale up playbook to add a new master / etcd master into the cluster using an inventory containing and etcd group of just 2 nodes. As above but the cluster has 5 master nodes. If you lose a master node and update the inventory to reflect that then the inventory will contain an etcd group with 4 nodes. @sdodson Previously submitted as https://github.com/openshift/openshift-ansible/pull/6979
| * Remove spaces introduced at the start of the lineGeoff Newson2018-02-071-1/+1
| | | | | | In making the initial change I introduce some spaces at the beginning of the line. Removing them.
| * Changing the check for the number of etcd nodesGeoff Newson2018-02-071-1/+1
| |
* | Add base package installation to upgrade playbooksRussell Teague2018-02-071-2/+3
|/ | | | | | | | Hosts will need python ipaddress module installed if it was not installed during initial installation. Bug 1540537 https://bugzilla.redhat.com/show_bug.cgi?id=1540537
* Don't use 'omit' for package moduleVadim Rutkovsky2018-02-021-1/+2
| | | | 'name' param is required and can't be skipped. 'when' is used instead
* Merge pull request #6927 from sdodson/1538974OpenShift Merge Robot2018-01-301-0/+8
|\ | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Don't deploy the console if disabled or registry subtype Whenever openshift_deployment_subtype == registry or the console is explicitly listed as a disabled feature don't deploy. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1538974
| * Don't deploy the console if disabled or registry subtypeScott Dodson2018-01-291-0/+8
| | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1538974
* | Merge pull request #6215 from sdodson/bz1511870OpenShift Merge Robot2018-01-291-0/+1
|\ \ | |/ |/| | | | | | | | | | | | | Automatic merge from submit-queue. Add kube service ipaddress to no_proxy list Computed as the first ip address in the portal_net Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1511870
| * Add kube service ipaddress to no_proxy listScott Dodson2018-01-241-0/+1
| | | | | | | | Computed as the first ip address in the portal_net
* | [1530403] Improve etcd group error messageRussell Teague2018-01-241-1/+5
|/ | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1530403
* Merge pull request #6749 from sdodson/allow-5-etcd-masterScott Dodson2018-01-241-1/+1
|\ | | | | Allow 5 etcd hosts
| * Allow 5 etcd hostsScott Dodson2018-01-161-1/+1
| |
* | Merge pull request #6817 from mtnbikenc/fix-1506750OpenShift Merge Robot2018-01-231-2/+2
|\ \ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. [1506750] Ensure proper hostname check override Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1506750
| * | [1506750] Ensure proper hostname check overrideRussell Teague2018-01-221-2/+2
| | |
* | | Install base_packages earlierMichael Gugino2018-01-223-41/+55
|/ / | | | | | | | | | | | | | | | | | | | | | | Currently, openshift_facts requires pyyaml to be installed. This package is installed via init/base_packages.yml, which is currently called after init/facts.yml. This results in a situation where installs will fail due to missing python dependency. This commit splits init/facts.yml into two, and allows base_packages.yml to be run before the openshift_facts.py plugin is executed.
* / Fix node scaleup playsMichael Gugino2018-01-184-6/+11
|/ | | | | | | | Currently, users have no way to run preqrequisites.yml on just newly added nodes during scaleup. This commit ensures only the new nodes are changed during scaleup as well as ensure prerequisites are run.
* Add defaults for openshift_pkg_versionMichael Gugino2018-01-101-2/+2
| | | | | | | | | | | | This variable may or may not be defined by the users. During deployments, it will be set to '-{{ openshift_version }}' if undefined. During upgrades, it will remain undefined. This commit ensures that if the variable is undefined, empty strings '' are set.
* Merge pull request #6674 from mgugino-upstream-stage/remove-becomes2Scott Dodson2018-01-101-1/+0
|\ | | | | Remove become statements
| * Remove become statementsMichael Gugino2018-01-091-1/+0
| | | | | | | | | | This commit removes become:no statements that break the installer in various ways.
* | Limit host group scope on control-plane upgradesMichael Gugino2018-01-092-2/+5
| | | | | | | | | | This commit limits common init code to exclude oo_nodes_to_config during upgrade_control_plane runs.
* | Refactor version and move some checks into sanity_checks.pyMichael Gugino2018-01-092-12/+23
|/ | | | | | | | | | | | | This commit changes how we handle openshift_version role. Most of the version initialization code is only run on the first master now. All other hosts have values set from the master. Aftwards, we run some basic RPM queries to ensure that the correct version is available on the other nodes. Containerized needs to do their own image checks elsewhere.
* Merge pull request #6634 from vrutkovs/openshift_binary_for_upgradesScott Dodson2018-01-091-0/+2
|\ | | | | upgrades: set openshift_client_binary fact when running on oo_first_master host
| * upgrades: set openshift_client_binary fact when running on oo_first_master hostVadim Rutkovsky2018-01-061-0/+2
| | | | | | | | | | | | | | | | | | This sets openshift_client_binary var for the first master, as some roles use this var along with first_master_client_binary. Not sure if its worth setting this var for the faulty roles instead though. Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | install base_packages on oo_all_hostsMichael Gugino2018-01-081-2/+2
|/ | | | | | | This commit ensures base packages are installed for oo_all_hosts, which is what we were doing previously. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1530516
* Migrate to import_role for static role inclusionScott Dodson2018-01-052-3/+3
| | | | | | | | | | | | | | | | | | | | | | | In Ansible 2.2, the include_role directive came into existence as a Tech Preview. It is still a Tech Preview through Ansible 2.4 (and in current devel branch), but with a noteable change. The default behavior switched from static: true to static: false because that functionality moved to the newly introduced import_role directive (in order to stay consistent with include* being dynamic in nature and `import* being static in nature). The dynamic include is considerably more memory intensive as it will dynamically create a role import for every host in the inventory list to be used. (Also worth noting, there is at the time of this writing an object allocation inefficiency in the dynamic include that can in certain situations amplify this effect considerably) This change is meant to mitigate the pressure on memory for the Ansible control host. We need to evaluate where it makes sense to dynamically include roles and revert back to dynamic inclusion if and where it makes sense to do so.
* Remove references to deployment_typeMichael Gugino2017-12-213-34/+46
| | | | | | | Move openshift_deployment_type check into sanity_check action plugin. Remove compatibility for deployment_type. deployment_type has been deprecated for some time now.
* Move validate_hosts to prerequisites.ymlMichael Gugino2017-12-202-28/+0
| | | | | | Move more checks outside of init/main.yml for speeding up upgrades and other operational plays that need to run.
* Move sanity_checks into custom action pluginMichael Gugino2017-12-201-48/+12
| | | | | | | This commit moves sanity_checks tasks into a custom action plugin that is only run against a single host. This will result in a large reduction of tasks during initialization
* Remove openshift.common.{is_atomic|is_containerized}Michael Gugino2017-12-201-11/+16
| | | | | We set these variables using facts in init, no need to duplicate the logic all around the codebase.
* Relocate filter plugins to lib_utilsMichael Gugino2017-12-181-2/+2
| | | | | | | | | | | | | | This commit relocates filter_plugings to lib_utils, changes the namespacing to prevent unintended use of older versions that may be present in filter_plugins/ directory on existing installs. Add lib_utils to meta depends for roles Also consolidate some plugins into lib_utils from various other areas. Update rpm spec, obsolete plugin rpms.
* Merge pull request #6489 from mgugino-upstream-stage/fix-repos-v2Michael Gugino2017-12-151-2/+2
|\ | | | | Fix rhel_subscribe boolean
| * Fix rhel_subscribe booleanMichael Gugino2017-12-151-2/+2
| | | | | | | | | | | | | | This commit fixes a variable name to it's correct name used in a when condition. Also makes use of 'is defined' consistent.
* | Merge pull request #6484 from mtnbikenc/tests-as-filtersRussell Teague2017-12-151-3/+3
|\ \ | |/ |/| Deprecate using Ansible tests as filters
| * Deprecate using Ansible tests as filtersRussell Teague2017-12-141-3/+3
| |
* | Merge pull request #6483 from mgugino-upstream-stage/fix-reposScott Dodson2017-12-142-4/+2
|\ \ | | | | | | Move repo and subscribe to prerequisites
| * | Move repo and subscribe to prerequisitesMichael Gugino2017-12-142-4/+2
| |/ | | | | | | | | This commit refactors some steps in rhel subscribe and moves repos.yml from init/main to prerequisites.
* | Merge pull request #5970 from mtnbikenc/fix-1506750OpenShift Merge Robot2017-12-141-7/+27
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. 1506750 Validate node hostname and IP address - Replaces use of `pause` with a `fail` task. `pause` only runs on one host, therefore the check was not run for all nodes - Adds check for valid openshift_ip Fixes 1506750 https://bugzilla.redhat.com/show_bug.cgi?id=1506750
| * Validate node hostname and IP addressRussell Teague2017-12-131-7/+27
| | | | | | | | | | | | Fixes 1506750 https://bugzilla.redhat.com/show_bug.cgi?id=1506750
* | Commit to stabalize RHSM operations. This code is derived from contribChris Callegari2017-12-141-0/+8
|/
* Remove unneeded embedded etcd logicMichael Gugino2017-12-081-6/+1
| | | | | Removing some remaining embedded etcd facts except for the migration plays.
* Fix ami building.Kenny Woodson2017-12-061-0/+2
|
* Implement container runtime roleMichael Gugino2017-12-012-4/+7
|
* retry package operationsLuke Meyer2017-11-301-0/+4
| | | | | | When a package install/update fails due to network blips or other spotty availability, retry it. If the failure is a real failure (e.g. package is really not there) it still fails after 3 tries (Ansible default).
* Remove openshift.common.cli_imageMichael Gugino2017-11-271-1/+0
| | | | | This commit removes openshift.common.cli_image in favor of openshift_cli_image.
* Simplify is_master_system_container logicMichael Gugino2017-11-271-2/+0
| | | | Simplify the variable is_master_system_container
* Merge pull request #6198 from mgugino-upstream-stage/openvswitch-sys-containerMichael Gugino2017-11-271-2/+0
|\ | | | | Remove is_openvswitch_system_container from facts
| * Remove is_openvswitch_system_container from factsMichael Gugino2017-11-221-2/+0
| | | | | | | | Simply logic for this variable.
* | Merge pull request #6227 from adelton/issue-5517Scott Dodson2017-11-241-0/+1
|\ \ | |/ |/| Workaround the fact that package state=present with dnf fails for already installed but excluded packages.
| * With dnf repoquery and excluded packages, --disableexcludes=all is needed to ↵Jan Pazdziora2017-11-221-0/+1
| | | | | | | | list the package with --installed.
* | Fix system_images_registry variableMichael Gugino2017-11-221-15/+0
|/ | | | Simplify the logic for this variable.