summaryrefslogtreecommitdiffstats
path: root/playbooks
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7124 from mgugino-upstream-stage/fix-version-multiOpenShift Merge Robot2018-02-144-38/+76
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix version multi Simplify double upgrade version logic Currently, double upgrade process (3.7 -> 3.9) for control plane attempts to run openshift_version role twice to set the appropriate values for upgrading each major version, 3.8 and 3.9. This commit instructs openshift_version to only inquire about the proper settings for 3.9, and hard-sets the appropriate values for 3.8. This allows a simplification of the openshift_version role, allowing for easier debugging. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542368
| * Simplify double upgrade version logicMichael Gugino2018-02-134-38/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, double upgrade process (3.7 -> 3.9) for control plane attempts to run openshift_version role twice to set the appropriate values for upgrading each major version, 3.8 and 3.9. This commit instructs openshift_version to only inquire about the proper settings for 3.9, and hard-sets the appropriate values for 3.8. This allows a simplification of the openshift_version role, allowing for easier debugging. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542368
* | Fix metadata access in OpenStack inventoryTomas Sedovic2018-02-141-1/+1
| | | | | | | | | | Some versions of shade (e.g. the one currently in RHEL) only allow a dict-like interface.
* | Merge pull request #6918 from SaravanaStorageNetwork/uninstall_playbookOpenShift Merge Robot2018-02-132-0/+12
|\ \ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Uninstall playbook for Glusterfs Uninstall playbook for Glusterfs
| * | uninstall playbook for GlusterFSSaravanakumar Arumugam2018-02-072-0/+12
| | | | | | | | | | | | Signed-off-by: Saravanakumar Arumugam <sarumuga@redhat.com>
* | | Merge pull request #7116 from mgugino-upstream-stage/fix-docker-upgradeOpenShift Merge Robot2018-02-132-1/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix docker_upgrade variable Currently, docker_upgrade is ignored during cluster upgrades. This commit ensures that the variable is respected. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1543714
| * | | Fix docker_upgrade variableMichael Gugino2018-02-122-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, docker_upgrade is ignored during cluster upgrades. This commit ensures that the variable is respected. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1543714
* | | | Merge pull request #7121 from sdodson/fix-registry-cert-redeployKenny Woodson2018-02-131-1/+1
|\ \ \ \ | | | | | | | | | | Invert logic to decide when to re-deploy certs
| * | | | Invert logic to decide when to re-deploy certsScott Dodson2018-02-121-1/+1
| | | | |
* | | | | Merge pull request #6951 from tomassedovic/fix-openstack-masterOpenShift Merge Robot2018-02-132-6/+11
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix OpenStack playbooks This shuffles a few playbooks around to fix the ordering between various facts and prerequisites.
| * | | | | Remove cluster_facts.yml from the install.ymlTomas Sedovic2018-02-121-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was moved there by accident. There's no reason to have it there explicitly.
| * | | | | Fix OpenStack playbooksTomas Sedovic2018-02-122-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This shuffles a few playbooks around to fix the ordering between various facts and prerequisites.
* | | | | | Merge pull request #7095 from jpeeler/upgrade-catOpenShift Merge Robot2018-02-121-0/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Add service catalog to be upgraded Currently service catalog was left running at the previous version. https://bugzilla.redhat.com/show_bug.cgi?id=1541247
| * | | | | | Add service catalog to be upgradedJeff Peeler2018-02-121-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently service catalog was left running at the previous version.
* | | | | | | Merge pull request #7113 from mgugino-upstream-stage/fix-etcd-scaleup-sanityMichael Gugino2018-02-121-0/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix etcd scaleup plays
| * | | | | | | Fix etcd scaleup playsMichael Gugino2018-02-121-0/+1
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit ensures that only the proper host groups have sanity checks run during etcd scaleup. This commit also adds additional debugging statements to sanity_checks.py to make it easier to debug when an error occurs. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1543771
* | | | | | | Merge pull request #7108 from spadgett/check-version-for-console-installOpenShift Merge Robot2018-02-121-1/+3
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Disable console install when not 3.9 or newer Fixes https://github.com/openshift/origin/issues/18573 /assign @sdodson @smarterclayton @stevekuznetsov
| * | | | | | Disable console install when not 3.9 or newerSamuel Padgett2018-02-121-1/+3
| | |_|/ / / | |/| | | |
* | | | | | Merge pull request #7107 from csrwng/cluster_operator_componentsOpenShift Merge Robot2018-02-121-0/+24
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Add playbook to install components for cluster operator Adds a playbook to install infra components for cluster operator
| * | | | | Add playbook to install components for cluster operatorCesar Wong2018-02-121-0/+24
| | |/ / / | |/| | |
* | | | | Merge pull request #6699 from bogdando/heat_user_dataOpenShift Merge Robot2018-02-121-0/+32
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. [openstack] custom user commands for cloud-init Allow to specify additional user commands executed on all Nova servers provisioned via Heat. An example use case is installing and starting os-collect-config agents to put Nova servers under the configuration management driven via the host openstack cloud Heat services. This allows to integrate with another deployment tools like TripleO. Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
| * | | | Fix openshift_openstack_provision_user_commandsBogdan Dobrelya2018-02-121-3/+3
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
| * | | | Fix wordingBogdan Dobrelya2018-02-091-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
| * | | | Fix cloud init runcmd templatingBogdan Dobrelya2018-02-091-6/+6
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
| * | | | Note ignored Heat user data changes for openstackBogdan Dobrelya2018-02-091-0/+4
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
| * | | | Clarify the ansible playbook vs cloud-initBogdan Dobrelya2018-02-091-3/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Document use cases for custom post-provision ansible hooks vs cloud-init runcmd shell commands. Rename to openshift_openstack_cloud_init_runcmd. Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
| * | | | [openstack] custom user commands for cloud-initBogdan Dobrelya2018-02-091-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow to specify additional user commands executed on all Nova servers provisioned via Heat. An example use case is installing and starting os-collect-config agents to put Nova servers under the configuration management driven via the host openstack cloud Heat services. This allows to integrate with another deployment tools like TripleO. Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* | | | | Merge pull request #7001 from mbruzek/cidr_pool_start_endOpenShift Merge Robot2018-02-121-1/+6
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Removing prefix, replacing with cidr, pool_start and pool_end vars The heat template was hardcoded with a /24 cidr and that limited customers to 251 ip addresses in the OpenStack subnet. This allows the user to configure the cidr and the allocation pool start and end. Addresses issue #6829 that I created last week. @tomassedovic please take a look
| * | | | Removing prefix and replacing with cidr, pool_start and pool_end variables.Matt Bruzek2018-02-021-1/+6
| | | | |
* | | | | Merge pull request #7078 from mgugino-upstream-stage/limit-hostsOpenShift Merge Robot2018-02-0930-7/+149
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Limit host scope during plays Many plays only target a select subset of hosts, especially oo_first_master for components such as logging and registry. This commit limits the scope of most plays to eliminate unnecessary task execution on node groups. This will result in great time savings for large deployments. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1516526
| * | | | Limit host scope during playsMichael Gugino2018-02-0930-7/+149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many plays only target a select subset of hosts, especially oo_first_master for components such as logging and registry. This commit limits the scope of most plays to eliminate unnecessary task execution on node groups. This will result in great time savings for large deployments. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1516526
* | | | | Fix upgrade-control plane post_control_plane.ymlMichael Gugino2018-02-091-1/+1
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An entry-point playbook was imported by mistake. This caused common init code to run again, which is undesireable. This commit changes the import to use the corresponding 'private' play which does not call the init code. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542855
* | | | Merge pull request #7074 from kwoodson/redeploy_certs_insecureOpenShift Merge Robot2018-02-081-16/+20
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Redeploy router certificates during upgrade only when secure. Wrap the upgrade logic for redeploying certificates into another block so that insecure registries do not perform any certificate tasks.
| * | | | Redeploy router certificates during upgrade only when secure.Kenny Woodson2018-02-081-16/+20
| | | | |
* | | | | Merge pull request #6926 from abutcher/etcd-ca-hostScott Dodson2018-02-0810-36/+2
|\ \ \ \ \ | | | | | | | | | | | | Determine which host is the etcd CA host
| * | | | | Determine which etcd host is the etcd_ca_host rather than assume it is the ↵Andrew Butcher2018-02-018-13/+1
| | | | | | | | | | | | | | | | | | | | | | | | first host in the etcd host group.
| * | | | | Attempt to back up generated certificates on every etcd host.Andrew Butcher2018-02-011-1/+1
| | | | | |
| * | | | | Remove pre upgrade verification step re: etcd ca host.Andrew Butcher2018-02-011-22/+0
| | | | | |
* | | | | | 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
| | | | | |
* | | | | | Merge pull request #7056 from sdodson/gcp-wait-for-connectionScott Dodson2018-02-081-0/+6
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Use wait_for_connection to validate ssh transport is alive
| * | | | | Use wait_for_connection to validate ssh transport is aliveScott Dodson2018-02-071-0/+6
| | | | | |
* | | | | | Merge pull request #7043 from vrutkovs/prerequisites-enable-docker-excluderOpenShift Merge Robot2018-02-081-0/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Setup docker excluder if requested before container_runtime is installed That would prevent possible container runtime upgrades during cluster config Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540800 Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
| * | | | | | Setup docker excluder if requested before container_runtime is installedVadim Rutkovsky2018-02-071-0/+6
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | That would prevent possible container runtime upgrades during cluster config Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | | | | | Merge pull request #7017 from mtnbikenc/fix-1540537OpenShift Merge Robot2018-02-076-2/+8
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. [1540537] Add base package installation to upgrade playbooks 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
| * | | | | Add base package installation to upgrade playbooksRussell Teague2018-02-076-2/+8
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Merge pull request #7044 from zgalor/fix_uninstall_bugOpenShift Merge Robot2018-02-071-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix uninstall using openshift_prometheus_state=absent This was broken in https://github.com/openshift/openshift-ansible/pull/6811 bz: https://bugzilla.redhat.com/show_bug.cgi?id=1540806
| * | | | | Fix uninstall using openshift_prometheus_state=absentZohar Galor2018-02-071-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | This was broken in https://github.com/openshift/openshift-ansible/pull/6811 bz: https://bugzilla.redhat.com/show_bug.cgi?id=1540806
* | | | | Merge pull request #7052 from vrutkovs/rolling_restart_modeOpenShift Merge Robot2018-02-071-2/+2
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. 3.9 upgrade: fix typos in restart masters procedure * 'rolling_restart_mode' should be 'services', not 'service' * use 'state: restarted' to properly restart services Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540054 Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>