summaryrefslogtreecommitdiff
path: root/playbooks/common
AgeCommit message (Collapse)Author
2017-09-26Merge pull request #5470 from ingvagabund/consolidate-etcd-common-roleOpenShift Merge Robot
Automatic merge from submit-queue Consolidate etcd common role The last PR toward a single etcd role
2017-09-26Merge pull request #5536 from ingvagabund/make-difference-output-list-3.7OpenShift Merge Robot
Automatic merge from submit-queue make difference filter output a list for Python3 Fixes: #3934
2017-09-26Merge pull request #5472 from mgugino-upstream-stage/split-hosted-v2OpenShift Merge Robot
Automatic merge from submit-queue Refactor openshift_hosted plays and role (version 2) Currently, openshift_hosted role duplicates some logic across separate task chains. This commit cleans up the openshift_hosted role and converts it to be primarily used with include_role to give better logic to the playbooks that utilize this role. This commit also refactors the playbook that calls various openshift_hosted roles into individual playbooks. This allows more granularity for advanced users. ----- This version of the patch set rolls back some of the refactoring (removal of running fact roles as a dependency) and focuses on just realigning the roles and plays. Original PR: https://github.com/openshift/openshift-ansible/pull/5284 Once this merges, I will close the old PR. Leaving it open for now for reference.
2017-09-26make difference filter output a list for Python3Jan Chaloupka
2017-09-26Set the etcd backend quota to 4GB by defaultJan Chaloupka
2017-09-25Merge pull request #5520 from ingvagabund/check-etcd-storage-backendOpenShift Merge Robot
Automatic merge from submit-queue check if the storage backend is set to etcd3 before upgrading to 3.7 SSIA
2017-09-25consolidate etcd_common roleJan Chaloupka
2017-09-25Merge pull request #5451 from ingvagabund/consolidate-etcd-upgradeOpenShift Merge Robot
Automatic merge from submit-queue Consolidate etcd upgrade Consolidates `etcd_upgrade` into the `etcd` role.
2017-09-25Refactor openshift_hosted plays and roleMichael Gugino
Currently, openshift_hosted role duplicates some logic across separate task chains. This commit cleans up the openshift_hosted role and converts it to be primarily used with include_role to give better logic to the playbooks that utilize this role. This commit also refactors the playbook that calls various openshift_hosted roles into individual playbooks. This allows more granularity for advanced users.
2017-09-25check if the storage backend is set to etcd3 before upgrading to 3.7Jan Chaloupka
2017-09-25Merge pull request #5513 from lucastheisen/masterOpenShift Merge Robot
Automatic merge from submit-queue resolve #5428: python-dbus not found `python-dbus` is not available in centos standard repos, but: > It appears python-dbus is just a reference to dbus-python and `dbus-python` is.
2017-09-24Merge pull request #5495 from abutcher/ca-redeploy-expired-etcdOpenShift Merge Robot
Automatic merge from submit-queue Improve CA redeploy restart logic Expired etcd certificates require special casing around restarts in the certificate redeploy playbooks. When etcd certificates are expired we can't restart masters or nodes. We also can't simply restart etcd because peers also had expired certificates so we must start/stop etcd when we detect expired etcd certificates. `openshift-ca.yml`: * No longer restart master services when etcd certificates were previously expired. * No longer restart node services when master or etcd certificates were previously expired. `etcd-ca.yml`: * No longer restart master services when etcd certificates were previously expired. Tested using [gen_expired_tls.sh](https://gist.github.com/abutcher/bdd20b9d582675d89fb22658689c49e4) on one of my master/etcd hosts to ensure that restart logic changes caused us to skip the right restarts and do a full start/stop of etcd in the `redeploy-certificates.yml` and `redeploy-etcd-certificates.yml` playbooks. Note: When this happens with a cluster and you want to replace all certificates, you can run: `redeploy-etcd-ca.yml`, `redeploy-openshift-ca.yml` (which will both skip restarts) and then run `redeploy-certificates.yml` which will now be able to full/stop start etcd.
2017-09-23Merge pull request #5507 from abutcher/etcd-scaleup-urlsOpenShift Merge Robot
Automatic merge from submit-queue Set master facts prior to adding new etcd client urls to master config. `openshift_master` role dependencies were moved out of the role in https://github.com/openshift/openshift-ansible/pull/5392 so we need to call `openshift_master_facts` prior to patching the master config in etcd scaleup. https://bugzilla.redhat.com/show_bug.cgi?id=1490304
2017-09-23Merge pull request #5460 from ingvagabund/consolidate-etcd-migrate-roleOpenShift Merge Robot
Automatic merge from submit-queue Consolidate etcd migrate role The PR is based on top of https://github.com/openshift/openshift-ansible/pull/5371 and https://github.com/openshift/openshift-ansible/pull/5451. Once both PRs are merged, I will rebase.
2017-09-23resolve #5428: python-dbus not foundLucas Theisen
2017-09-23Merge pull request #5413 from ingvagabund/disable-excluders-after-pre-checksOpenShift Merge Robot
Automatic merge from submit-queue Move some pre-checks before excluders are disabled Some pre-checks needs an OCP version which is detected by a set of tasks that need the excluders to be disabled. So at the best I can move some pre-checks before the excluders are disabled. However, there will be still some checks that can fail with excluders updated to the newer version. Bug: 1484304
2017-09-22Merge pull request #5498 from mgugino-upstream-stage/fedora-pyyamlScott Dodson
Add python3-PyYAML for Fedora installs
2017-09-22Add booleans to prevent unwanted install of nuage roles.Michael Gugino
Recently, some role dependencies were removed from meta depends into playbooks. Particularly, the nuage role requires several roles. Currently, only the nuage_master role requires openshift_use_nuage to be true. This commit requires the other nuage roles to reference the variable openshift_use_nuage before install.
2017-09-22Allow examples management to be disabledScott Dodson
2017-09-22Merge pull request #5176 from ewolinetz/deprecate_log_met_varsOpenShift Merge Robot
Automatic merge from submit-queue Creating structure to warn for use of deprecated variables and set th… …em in a single location before they are no longer honored Implementation of https://trello.com/c/XKOGHSkP/460-3-deprecate-openshifthostedloggingmetrics-installer-variables Tried to implement this in a way such that we can easily and cleanly add/remove variables in the future for any role we choose. Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1449812 https://bugzilla.redhat.com/show_bug.cgi?id=1464349
2017-09-22Set master facts prior to adding new etcd client urls to master config.Andrew Butcher
2017-09-22Remove debugging statements and pause moduleScott Dodson
How on earth did the bot merge this? The upgrade test should've stalled indefinitely.
2017-09-22Merge pull request #5478 from sdodson/bz1490677OpenShift Merge Robot
Automatic merge from submit-queue Default openshift_pkg_version to full version-release during upgrades Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1490677 The versioning scheme for 3.7 pre-releases has changed and now all versions are 3.7.0 and the release is incremented on builds, ie: 3.7.0-0.124.0 upgraded to 3.7.0-0.125.0. If we know we're an upgrade and they haven't requested a specific package version defer the defaulting of openshift_pkg_version until the upgrade playbooks and there set it to the available version including the release.
2017-09-22consolidate etcd_migrate roleJan Chaloupka
2017-09-21Merge pull request #5226 from ewolinetz/template_service_brokerOpenShift Merge Robot
Automatic merge from submit-queue Creating initial tsb role to consume and apply templates provided for… … tsb cc: @deads2k @sdodson Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1486623 https://bugzilla.redhat.com/show_bug.cgi?id=1470623 https://bugzilla.redhat.com/show_bug.cgi?id=1491626
2017-09-21Merge pull request #5371 from ingvagabund/consolidate-etcd-certs-rolesOpenShift Merge Robot
Automatic merge from submit-queue consolidate etcd certs roles This is a starter for consolidation of all etcd like roles into a single `etcd` action-based role. I have intentionally started with the simplest one to demonstrate the steps needed to make it so and to make the review easy enough for everyone.
2017-09-21Add python3-PyYAML for Fedora installsMichael Gugino
Fedora installs currently fail because nodes require the usage of python3. python3-PyYAML must be installed in order to support usage of python3. This commit adds the rpm python3-PyYAML to fedora nodes. Fixes: https://github.com/openshift/openshift-ansible/issues/5469
2017-09-21Do a full stop/start when etcd certificates had expired.Andrew Butcher
2017-09-21Improve CA playbook restart logic and skip restarts when related services ↵Andrew Butcher
had previously expired certificates.
2017-09-21consolidate etcd_upgrade roleJan Chaloupka
2017-09-21Merge pull request #5473 from abutcher/scaleup-network-valuesScott Dodson
Set network facts using first master's config during scaleup.
2017-09-21disable excluders after all pre-checksJan Chaloupka
2017-09-20Merge pull request #5363 from ewolinetz/default_aggregationOpenShift Merge Robot
Automatic merge from submit-queue Updating to always configure api aggregation with installation This moves the wiring of the aggregator up into the config playbook as we want to enable this by default with an installation. Resolves https://github.com/openshift/openshift-ansible/issues/5056
2017-09-20Merge pull request #5474 from simo5/noreconcileOpenShift Merge Robot
Automatic merge from submit-queue Do not reconcile in >= 3.7 Starting with 3.7 we use kube's RBAC which happens to do a forceful reconcile at server startup. Explicit reconciles are not needed anymore.
2017-09-20Merge pull request #5357 from ↵OpenShift Merge Robot
juanvallejo/jvallejo/add-health-checks-upgrade-path Automatic merge from submit-queue add health checks 3_6,3_7 upgrade path Related BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1483931 Adds health checks to `upgrade_control_plane` and `upgrade_nodes` in 3_6 and 3_7. cc @sosiouxme @rhcarvalho @brenton
2017-09-20Merge pull request #5014 from ↵OpenShift Merge Robot
juanvallejo/jvallejo/add-additonal-checks-upgrade-path Automatic merge from submit-queue Adding additonal checks upgrade path Depends on https://github.com/openshift/openshift-ansible/pull/4960 TODO - Possibly handle `upgrade` playbook context on `etcd_volume` check cc @sosiouxme @rhcarvalho
2017-09-20Default openshift_pkg_version to full version-release during upgradesScott Dodson
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1490677 The versioning scheme for 3.7 pre-releases has changed and now all versions are 3.7.0 and the release is incremented on builds, ie: 3.7.0-0.124.0 upgraded to 3.7.0-0.125.0. If we know we're an upgrade and they haven't requested a specific package version defer the defaulting of openshift_pkg_version until the upgrade playbooks and there set it to the available version including the release.
2017-09-20Creating structure to warn for use of deprecated variables and set them in a ↵Eric Wolinetz
single location before they are no longer honored
2017-09-20Updating to always configure api aggregation with installationEric Wolinetz
2017-09-20Do not reconcile in >= 3.7Simo Sorce
Starting with 3.7 we use kube's RBAC which happens to do a forceful reconcile at server startup. Explicit reconciles are not needed anymore. Also drop obsolete version checks and simplify 'when' conditional Signed-off-by: Simo Sorce <simo@redhat.com>
2017-09-20Creating initial tsb role to consume and apply templates provided for tsbEric Wolinetz
2017-09-20Set network facts using first master's config during scaleup.Andrew Butcher
2017-09-20Cleanup old deployment typesMichael Gugino
Previously, openshift-ansible supported various types of deployments using the variable "openshift_deployment_type" Currently, openshift-ansible only supports two deployment types, "origin" and "openshift-enterprise". This commit removes all logic and references to deprecated deployment types.
2017-09-19Merge pull request #5380 from ↵OpenShift Merge Robot
mgugino-upstream-stage/fix-openshift-version-pkg-install Automatic merge from submit-queue Only install base openshift package on masters and nodes Recent refactoring to remove openshift_common resulted in base openshift rpm's being installed on more hosts than previous. This situation results in hosts that would otherwise not need access to openshift repositories to require them. This patch set results in only openshift_masters and openshift_nodes to have the openshift base package installed.
2017-09-19more retries on repoquery_cmdLuke Meyer
2017-09-18Merge pull request #5430 from ashcrow/always-required-new-variablesOpenShift Merge Robot
Automatic merge from submit-queue Always required new variables Related to https://bugzilla.redhat.com/show_bug.cgi?id=1451023
2017-09-18Consolidate etcd certs rolesJan Chaloupka
This is a part of the etcd_ like role consolidationi into an action-based role. As part of the consilidation some roles have been removed and some replaced by include_role module. Resulting in reorder and shift of role dependencies from a role into a play.
2017-09-18Merge pull request #5392 from ↵OpenShift Merge Robot
ingvagabund/pull-openshift_master-deps-out-into-a-play Automatic merge from submit-queue Pull openshift_master deps out into a play The `openshift_master` role is called only in a single play. Thus, we can pull out all its dependencies without duplicating all dependency role invocations. Both `lib_openshift` and `lib_os_firewall` are required deps as they defined ansible modules used inside the `openshift_master` role. I have also rearranged definition of variables so variable used only inside a single role are part of the `include_role` statement. Atm, we can't use `include_role` due to https://github.com/ansible/ansible/issues/21890
2017-09-15openshift_sanitize_inventory: Check for required varsSteve Milner
Moved the checks for osm_cluster_network_cidr, osm_host_subnet_length, openshift_portal_net from upgrade to openshift_sanitize_inventory as we now consider it a required variable for install, updrade, or scale up. Signed-off-by: Steve Milner <smilner@redhat.com>
2017-09-15Correct firewall install for openshift-nfsRussell Teague