summaryrefslogtreecommitdiffstats
path: root/playbooks/common
Commit message (Collapse)AuthorAgeFilesLines
* consolidate etcd_upgrade roleJan Chaloupka2017-09-213-15/+21
|
* Merge pull request #5380 from ↵OpenShift Merge Robot2017-09-191-0/+8
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * Only install base openshift package on masters and nodesMichael Gugino2017-09-121-0/+8
| | | | | | | | | | | | | | | | | | | | | | 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.
* | more retries on repoquery_cmdLuke Meyer2017-09-191-0/+4
| |
* | Merge pull request #5430 from ashcrow/always-required-new-variablesOpenShift Merge Robot2017-09-181-15/+0
|\ \ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue Always required new variables Related to https://bugzilla.redhat.com/show_bug.cgi?id=1451023
| * | openshift_sanitize_inventory: Check for required varsSteve Milner2017-09-151-15/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | Merge pull request #5392 from ↵OpenShift Merge Robot2017-09-181-15/+22
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | pull openshift_master deps out into a playJan Chaloupka2017-09-131-15/+22
| | |
* | | Correct firewall install for openshift-nfsRussell Teague2017-09-151-1/+1
| | |
* | | Merge pull request #5360 from sdodson/embedded-etcdScott Dodson2017-09-141-4/+7
|\ \ \ | | | | | | | | Clarify requirement of having etcd group
| * | | Clarify requirement of having etcd groupScott Dodson2017-09-111-4/+7
| | | |
* | | | Merge pull request #5396 from abutcher/etcd-migrate-openshift-ipScott Dodson2017-09-141-3/+3
|\ \ \ \ | | | | | | | | | | Use openshift.common.ip rather than ansible_default_ipv4 in etcd migration playbook.
| * | | | Use openshift.common.ip rather than ansible_default_ipv4 in etcd migration ↵Andrew Butcher2017-09-131-3/+3
| | |/ / | |/| | | | | | | | | | playbook.
* | | | Merge pull request #5367 from abutcher/etcd-scaleup-master-client-urlsScott Dodson2017-09-141-0/+19
|\ \ \ \ | | | | | | | | | | Bug 1490304: Etcd scale-up playbook should add new member to etcdClientInfo of master-config.yaml
| * | | | Update master config with new client urls during etcd scaleup.Andrew Butcher2017-09-121-0/+19
| | | | |
* | | | | Merge pull request #5389 from tremble/storage_migrationScott Dodson2017-09-141-9/+9
|\ \ \ \ \ | | | | | | | | | | | | Don't assume storage_migration control variables are already boolean
| * | | | | Don't assume storage_migration control variables are already booleanMark Chappell2017-09-131-9/+9
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | openshift_upgrade_pre_storage_migration_enabled openshift_upgrade_pre_storage_migration_fatal openshift_upgrade_post_storage_migration_enabled openshift_upgrade_post_storage_migration_enabled because the 4 variables are not already defaulted/evaluated there's no guarantee that they're actually the boolean type at this point. Example: if they're passed in on the command line
* | | | | Merge pull request #5386 from ashcrow/fix-wording-in-upgrade-noticeOpenShift Bot2017-09-131-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | | Merged by openshift-bot
| * | | | upgrade: Updates warning on missing required variablesSteve Milner2017-09-121-1/+1
| |/ / / | | | | | | | | | | | | Signed-off-by: Steve Milner <smilner@redhat.com>
* / / / double the required disk space for etcd backupJan Chaloupka2017-09-121-1/+1
|/ / /
* | | Merge pull request #5353 from mgugino-upstream-stage/public_hostname_checkScott Dodson2017-09-121-0/+4
|\ \ \ | | | | | | | | Add openshift_public_hostname length check
| * | | Add openshift_public_hostname length checkMichael Gugino2017-09-111-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the variable openshift_hostname is checked to ensure it is less than 63 characters. The variable openshift_public_hostname should also be checked for this length. This commit checks the length of openshift_public_hostname to ensure it is 63 characters or less. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1467790
* | | | Merge pull request #5340 from sdodson/bz1489913Scott Dodson2017-09-111-3/+4
|\ \ \ \ | |_|/ / |/| | | Revert back to hostnamectl and previous default of not setting hostname
| * | | Revert back to hostnamectl and previous default of not setting hostnameScott Dodson2017-09-081-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old default was "{{ not openshift.common.version_gte_3_1_or_1_1 }}" just default to false instead, master branch will never be used with 3.0 hostname module doesn't work on atomic host Fixes BZ 1489913
* | | | Merge pull request #5344 from mgugino-upstream-stage/fix-pkg-versionsOpenShift Bot2017-09-111-0/+5
|\ \ \ \ | | | | | | | | | | Merged by openshift-bot
| * | | | Fix rpm version logic for hostsMichael Gugino2017-09-081-0/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the variable openshift_pkg_version is defined by the first master unless specified by the user. The role openshift_version attempts to set this value on all other hosts after setting it on the first master. During normal installation, openshift_pkg_version is not initially defined, therefor each host will utilize the most recent rpm available. This creates a problem during later operations of a cluster if a node is added or replaced by a new node. This commit ensures that the value for openshift_pkg_version set by the first master is respected by subsequent hosts. Fixes: #5341
* | | | Merge pull request #5256 from ashcrow/1451023Scott Dodson2017-09-111-0/+15
|\ \ \ \ | |_|_|/ |/| | | upgrade: Verify required network items are set
| * | | upgrade: Verify required network items are setSteve Milner2017-09-081-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When upgrading osm_cluster_network_cidr, osm_host_subnet_length, and openshift_portal_net must be set to avoid SDN initialization errors. This was found when the default parameters were changed between Openshift versions. This meant users who upgraded and did not specify either mentioned variable at install/upgrade time ended up getting SDN errors post upgrade. When osm_cluster_network_cidr, osm_host_subnet_length, and openshift_portal_net are not set the upgrade will fail telling the user that the variables must be set and how to find the current values in the current install. References: https://github.com/openshift/openshift-ansible/commit/b50b4ea0b03feb9431abd7294fe4fb6b549ddfc0 Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451023
* | | | Merge pull request #5342 from mtnbikenc/fix-include-through-symlinkScott Dodson2017-09-111-1/+1
|\ \ \ \ | | | | | | | | | | Correct include path to not follow symlink
| * | | | Correct include path to not follow symlinkRussell Teague2017-09-081-1/+1
| | |/ / | |/| |
* | | | Merge pull request #5332 from kwoodson/amibuild_node_configOpenShift Bot2017-09-111-0/+1
|\ \ \ \ | |_|_|/ |/| | | Merged by openshift-bot
| * | | ami build process calls openshift-node/config.ymlKenny Woodson2017-09-071-0/+1
| |/ /
* / / Fix include path for docker upgrade tasksRussell Teague2017-09-081-2/+4
|/ /
* | Merge pull request #5211 from kwoodson/provisioning_fixesScott Dodson2017-09-071-0/+1
|\ \ | |/ |/| Provisioning updates.
| * Consolidating AWS roles and variables underneath openshift_aws role.Kenny Woodson2017-09-071-0/+1
| |
* | Merge pull request #5305 from mtnbikenc/breakout-master-additionalOpenShift Bot2017-09-072-3/+2
|\ \ | |/ |/| Merged by openshift-bot
| * Move master additional config out of baseRussell Teague2017-09-052-3/+2
| |
* | Merge pull request #5208 from mgugino-upstream-stage/remove-openshift_commonOpenShift Bot2017-09-0714-25/+71
|\ \ | | | | | | Merged by openshift-bot
| * | Remove openshift-commonMichael Gugino2017-09-0614-25/+71
| |/ | | | | | | | | | | | | | | | | Most of this role's purpose was to set facts. The vast majority of these facts were simply redefining user-supplied variables. This commit also removes various artifacts leftover from previous versions, as well as variables that seem to be entirely unused.
* / Create ansible role for deploying prometheus on openshiftZohar Galor2017-09-062-0/+12
|/ | | | | | | | | | | | | | A new role for installing prometheus on openshift. Depends on `openshift_hosted_prometheus_deploy` flag role creates: - prometheus namespace - prometheus clusterrolebinding and service account - pvs for prometheus, alertmanager and alertbuffer for internal nfs - prometheus pod with prometheus behind oauth-proxy, alertmanager and alert-buffer behind oauth-proxy - prometheus and alertmanager configmaps - prometheus and alerts services and direct routes - prometheus, alertmanager and alert-buffer pvcs
* Standardize etcd entry point playbooksRussell Teague2017-09-012-20/+0
|
* Break out master config into stand-alone playbookRussell Teague2017-09-013-6/+3
| | | | | | | | | Added byo/openshift-master/config.yml as a new entry point for performing master installation. Excluder disable/enable for master has been moved to common/openshift-master/config.yml to follow a standard pattern that could be used for both installation and scaleup playbooks.
* Merge pull request #5282 from mtnbikenc/breakout-node-configOpenShift Bot2017-09-015-62/+32
|\ | | | | Merged by openshift-bot
| * Break out node config into stand-alone playbookRussell Teague2017-08-315-62/+32
| | | | | | | | | | | | | | | | | | | | Added byo/openshift-node/config.yml as a new entry point for performing node installation. This playbook must be run after masters have already been deployed. Excluder disable/enable for both nodes and masters has been moved their respective config.yml. This created a common pattern that could be used for both installation and scaleup playbooks.
* | Merge pull request #5283 from mtnbikenc/move-aio-failOpenShift Bot2017-09-012-7/+10
|\ \ | | | | | | Merged by openshift-bot
| * | Move all-in-one fail check to evaluate_groups.ymlRussell Teague2017-08-312-7/+10
| |/
* | Merge pull request #5259 from mtnbikenc/fix-upgrade-auth-checkRussell Teague2017-08-311-0/+1
|\ \ | |/ |/| [BZ1486450] Only run migrate auth for < 3.7
| * Only run migrate auth for < 3.7Russell Teague2017-08-291-0/+1
| |
* | Merge pull request #5247 from mgugino-upstream-stage/fix-master-upgradesOpenShift Bot2017-08-301-4/+1
|\ \ | | | | | | Merged by openshift-bot
| * | Fix openshift_master upgradeMichael Gugino2017-08-291-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | Currently, openshift_master upgrade play imports tasks directly from the openshift_master role. This method does not honor role defaults. This commit changes sets openshift_master_config_dir if undefined.