summaryrefslogtreecommitdiffstats
path: root/playbooks
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #6886 from ↵OpenShift Merge Robot2018-01-271-1/+2
|\ | | | | | | | | | | | | | | | | smarterclayton/playbooks/gcp/openshift-cluster/build_base_image.yml Automatic merge from submit-queue. Add GPG keys in the base image and don't install docker Base image fails to accept the keys.
| * Add GPG keys in the base image and don't install dockerClayton Coleman2018-01-251-1/+2
| |
* | Merge pull request #6896 from mgugino-upstream-stage/fix-upgrade-image-tagOpenShift Merge Robot2018-01-261-3/+24
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Correct 3.7 to 3.9 upgrade openshift_image_tag Due to complexities upgrading two versions at the same time, openshift_image_tag was being set incorrectly during control_plane upgrades. This commit ensures that openshift_image_tag is set correctly during this process. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1536839
| * | Correct 3.7 to 3.9 upgrade openshift_image_tagMichael Gugino2018-01-261-3/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to complexities upgrading two versions at the same time, openshift_image_tag was being set incorrectly during control_plane upgrades. This commit ensures that openshift_image_tag is set correctly during this process. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1536839
* | | Merge pull request #6862 from ewolinetz/deprecate_use_callbackOpenShift Merge Robot2018-01-261-8/+0
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Updating deprecations to use callback plugin Instead of just pausing during the run and relying on users to see it, instead format and print the deprecated variables at the end of play instead like this: ``` Initialization : Complete (0:00:23) [DEPRECATION WARNING]: The following are deprecated variables and will be no longer be used in the next minor release. Please update your inventory accordingly. openshift_hosted_logging_deploy openshift_hosted_metrics_deployer_prefix ``` The plugin changes are the same as the changes made in https://github.com/openshift/openshift-ansible/pull/6734
| * | Updating deprecations to use callback pluginEric Wolinetz2018-01-251-8/+0
| | |
* | | Merge pull request #6821 from spadgett/remove-asset-configScott Dodson2018-01-264-41/+57
|\ \ \ | | | | | | | | Remove old assetConfig from master-config.yaml
| * | | Remove old assetConfig from master-config.yamlSamuel Padgett2018-01-254-41/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `assetConfig` is no longer used now that the console is split into its own pod. - Remove `assetConfig` on upgrades to 3.9 - Stop writing logging and metrics URLs to assetConfig for 3.9
* | | | Merge pull request #6859 from abutcher/bz1536217OpenShift Merge Robot2018-01-252-0/+46
|\ \ \ \ | |/ / / |/| | | | | | | | | | | Automatic merge from submit-queue. Bug 1536217: Need to validate etcd serving certs before 3.9 upgrade
| * | | Redeploy etcd certificates during upgrade when etcd hostname not present in ↵Andrew Butcher2018-01-242-0/+46
| | | | | | | | | | | | | | | | etcd serving cert SAN.
* | | | Merge pull request #6858 from mtnbikenc/fix-1530403Scott Dodson2018-01-251-1/+5
|\ \ \ \ | | | | | | | | | | [1530403] Improve etcd group error message
| * | | | [1530403] Improve etcd group error messageRussell Teague2018-01-241-1/+5
| | | | | | | | | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1530403
* | | | | Merge pull request #6849 from vrutkovs/label-masters-with-upstream-standardOpenShift Merge Robot2018-01-252-20/+10
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Label masters with node-role.kubernetes.io/master This is controlled by `openshift_master_node_label` var in openshift_manage_node Related to https://bugzilla.redhat.com/show_bug.cgi?id=1535673
| * | | | Label masters with node-role.kubernetes.io/master. This PR also sets these ↵Vadim Rutkovsky2018-01-242-20/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | labels and scheduling status during upgrades Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | | | | Merge pull request #6842 from ↵Scott Dodson2018-01-249-1/+10
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | mgugino-upstream-stage/protect-upgrade-master-version Protect master installed version during node upgrades
| * | | | Protect master installed version during node upgradesMichael Gugino2018-01-239-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Master version is unprotected during all upgrades. During node-only upgrades, master version should remain protected to ensure correct image tag is set for node upgrades. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1536839
* | | | | Merge pull request #6603 from joelddiaz/fetch_subnet_azOpenShift Merge Robot2018-01-241-1/+17
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. un-hardcode default subnet az allow defining 'default_az' for one of the subnets in the vpc structure, and pull that default_az and set it to openshift_aws_subnet_az. this should allow one less variable to be defined/overridded when using non-default (us-east-1) regions
| * | | | un-hardcode default subnet azJoel Diaz2018-01-221-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | allow defining 'default_az' for one of the subnets in the vpc structure, and pull that default_az and set it to openshift_aws_subnet_az. this should allow one less variable to have to be defined/overridded (openshift_aws_subnet_az) when using non-default (us-east-1) regions update provisioning_vars.yml.example to show an example VPC structure
* | | | | 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 #6541 from smarterclayton/move_gcp_inScott Dodson2018-01-2416-91/+447
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Allow openshift-ansible image to deploy to GCP
| * | | | | Split control plane and component install in deploy_clusterClayton Coleman2018-01-237-88/+90
| | | | | |
| * | | | | PyYAML is required by openshift_facts on nodesClayton Coleman2018-01-231-0/+2
| | | | | |
| * | | | | Move origin-gce roles and playbooks into openshift-ansibleClayton Coleman2018-01-2310-5/+357
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This moves all core functionality into the openshift-ansible repo, adds the necessary equivalent entrypoint to the openshift-ansible installer image, and ensures the dynamic inventory mechanisms in openshift-ansible continue to work. Notable changes from origin-gce: * playbook extensions changed to .yml * dynamic inventory subdirectory created to prevent accidental use * use the custom entrypoint entrypoint-gcp for this image * move tasks into openshift_gcp role
* | | | | | Merge pull request #6802 from mgugino-upstream-stage/fix-etcd-scaleupOpenShift Merge Robot2018-01-234-3/+56
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix etcd scaleup playbook Currently, etcd scaleup playbook has no way to account for newly added prerequisites.yml play. This commit allows adding new etcd hosts via scaleup play and accounts for etcd hosts that are standalone or part of nodes or masters group.
| * | | | | Fix etcd scaleup playbookMichael Gugino2018-01-224-3/+56
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, etcd scaleup playbook has no way to account for newly added prerequisites.yml play. This commit allows adding new etcd hosts via scaleup play and accounts for etcd hosts that are standalone or part of nodes or masters group.
* | | | | Merge pull request #6817 from mtnbikenc/fix-1506750OpenShift Merge Robot2018-01-232-3/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-222-3/+3
| |/ / / /
* | | | | Merge pull request #6812 from sdodson/bz1396350OpenShift Merge Robot2018-01-231-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Ensure that node names are lowerecased before matching The name field is always lowercase whereas the nodename may not have been in the past. Ensure that we lowercase it before looking for matching nodes. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1396350
| * | | | Ensure that node names are lowerecased before matchingScott Dodson2018-01-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name field is always lowercase whereas the nodename may not have been in the past. Ensure that we lowercase it before looking for matching nodes. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1396350
* | | | | Merge pull request #6504 from mbruzek/mbruzek-openshift-openstackRussell Teague2018-01-231-6/+8
|\ \ \ \ \ | | | | | | | | | | | | The provision_install file ends in yml not yaml! Ansible requirement …
| * \ \ \ \ Merge branch 'master' into mbruzek-openshift-openstackMatt Bruzek2018-01-18153-520/+854
| |\ \ \ \ \
| * | | | | | The provision_install file ends in yml not yaml! Ansible requirement ↵Matt Bruzek2017-12-151-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | clarification.
* | | | | | | Merge pull request #6796 from mgugino-upstream-stage/fix-etcd-upgrade-sanityScott Dodson2018-01-221-0/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix etcd upgrade sanity
| * | | | | | | Fix etcd-upgrade sanity checksMichael Gugino2018-01-191-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit ensures that only oo_etcd_to_config and oo_masters_to_config are checked during sanity_checks. This will prevent hosts that have not had facts gathered (such as nodes) will not be processed for correct inventory and runtime facts. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1536317
* | | | | | | | Merge pull request #6533 from joelddiaz/prereqs_uninstallOpenShift Merge Robot2018-01-225-0/+50
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. allow uninstalling AWS objects created by prerequisite playbook add deprovisioners/uninstallers for objects created via playbooks/aws/openshift-cluster/prerequisites.yml specifically: security groups, vpcs, and any ssh keys introduce openshift_aws_enable_uninstall_shared_objects to protect AWS objects that would be shared in the case of multiple clusters co-existing in one AWS account. right now it protects the ssh keys, but it can/should be used to protect against deleting the shared IAM instance profile as well. default this variable to False to be on the safe side when uninstalling/deprovisioning.
| * | | | | | | | allow uninstalling AWS objects created by prerequisite playbookJoel Diaz2018-01-225-0/+50
| | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add deprovisioners/uninstallers for objects created via playbooks/aws/openshift-cluster/prerequisites.yml specifically: security groups, vpcs, and any ssh keys introduce openshift_aws_enable_uninstall_shared_objects to protect AWS objects that would be shared in the case of multiple clusters co-existing in one AWS account. right now it protects the ssh keys, but it can/should be used to protect against deleting the shared IAM instance profile as well. default this variable to False to be on the safe side when uninstalling/deprovisioning. add some documentation on using deprovisioning playbooks
* | | | | | | | Merge pull request #6814 from mgugino-upstream-stage/move-up-base-packagesOpenShift Merge Robot2018-01-226-46/+60
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Install base_packages earlier 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.
| * | | | | | | Install base_packages earlierMichael Gugino2018-01-226-46/+60
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 master scaleup playMichael Gugino2018-01-192-10/+39
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow playbooks/openshift-master/scaleup.yml to call prerequisites.yml at the proper time. Related-to: https://github.com/openshift/openshift-ansible/pull/6784
* | | | | | Merge pull request #6784 from mgugino-upstream-stage/fix-node-scaleupMichael Gugino2018-01-1913-17/+43
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix node scaleup plays
| * | | | | | Fix node scaleup playsMichael Gugino2018-01-1813-17/+43
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | | | Merge pull request #6691 from abutcher/oc-adm-csr-cnAndrew Butcher2018-01-191-36/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Use master group's ansible_ssh_user when accepting nodes.
| * | | | | | Remove duplication in node acceptance playbook and setup master groups so ↵Andrew Butcher2018-01-161-36/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | that we can use the first master's ansible_ssh_user when delegating.
* | | | | | | Fix OpenStack readmeTomas Sedovic2018-01-191-1/+1
| |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ansible-playbook command in the OpenStack readme is missing a trailing backslash after the dynamic inventory. This prevents being able to copy/paste the command into the terminal and just run it as is.
* | | | | | Merge pull request #6771 from tomassedovic/fix-readmeOpenShift Merge Robot2018-01-191-5/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Drop the testing repo var from openstack readme This was introduced as a temporary fix for the origin+centos deployments. It is no longer necessary and it's confusing to newcomers.
| * | | | | | Drop the testing repo var from openstack readmeTomas Sedovic2018-01-181-5/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This was introduced as a temporary fix for the origin+centos deployments. It is no longer necessary and it's confusing to newcomers.
* | | | | | Merge pull request #6760 from mgugino-upstream-stage/fix-docker-atomic-proxiesOpenShift Merge Robot2018-01-181-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Ensure atomic_proxies are configured with docker In some setups, system containers may be used in conjunction with package_docker. This commit ensures that the atomic proxies are configured while using package_docker and containerized components. This commit also ensures containerized_groups are based on openshift_is_containerized instead of 'containerized' variable directly to account for possible use of atomic hosts which users may not specify containerized in inventory directly. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1535323
| * | | | | Ensure atomic_proxies are configured with dockerMichael Gugino2018-01-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some setups, system containers may be used in conjunction with package_docker. This commit ensures that the atomic proxies are configured while using package_docker and containerized components. This commit also ensures containerized_groups are based on openshift_is_containerized instead of 'containerized' variable directly to account for possible use of atomic hosts which users may not specify containerized in inventory directly. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1535323
* | | | | | Merge pull request #6076 from abutcher/bz1511357Scott Dodson2018-01-171-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Bug 1511357: Task "Reload generated facts" failed while scaling up containerized masters