summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7068 from datarace/logging-fixScott Dodson2018-02-084-15/+15
|\ | | | | changed oc to {{ openshift_client_binary }}
| * changed oc to {{ openshift_client_binary }}datarace2018-02-084-15/+15
| |
* | Merge pull request #7031 from mgugino-upstream-stage/fix-syscon-varsOpenShift Merge Robot2018-02-082-1/+5
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix master and node system container variables This commit corrects service names and adds DOCKER_SERVICE variable to openshift_master system container creation. This will ensure services will restart correctly and in the proper order. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542324
| * | Fix master and node system container variablesMichael Gugino2018-02-062-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit corrects service names and adds DOCKER_SERVICE variable to openshift_master system container creation. This will ensure services will restart correctly and in the proper order. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542324
* | | 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 #7079 from fabianvf/1524805-cloudforms-pull-policyScott Dodson2018-02-081-1/+1
|\ \ \ \ | | | | | | | | | | Bug 1524805- CFME example now works disconnected
| * | | | Bug 1524805- CFME example now works disconnectedFabian von Feilitzsch2018-02-081-1/+1
|/ / / /
* | | | Merge pull request #7055 from fabianvf/bz1541625Scott Dodson2018-02-081-2/+2
|\ \ \ \ | | | | | | | | | | Bug 1541625- properly cast provided ip address to unicode
| * | | | Bug 1541625- properly cast provided ip address to unicodeFabian von Feilitzsch2018-02-071-2/+2
| | | | |
* | | | | 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 #7060 from mwoodson/masterOpenShift Merge Robot2018-02-081-1/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. aws ami: make it so the tags from the orinal AMI are used with the newly created AMI
| * | | | | | aws ami: make it so the tags from the orinal AMI are used with the newly ↵Matt Woodson2018-02-071-1/+14
| | |_|/ / / | |/| | | | | | | | | | | | | | | | created AMI
* | | | | | Merge pull request #7057 from ashcrow/remove-master-from-ami-node-bootstrapKenny Woodson2018-02-082-3/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | openshift_node: Remove master from aws node building
| * | | | | | openshift_node: Remove master from aws node buildingSteve Milner2018-02-072-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since it is expected to be re-added at some point in the future the lines have been commented out rather than deleted. Signed-off-by: Steve Milner <smilner@redhat.com>
* | | | | | | 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 #7063 from sdodson/wc39Scott Dodson2018-02-071-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use v3.9 web-console image for now
| * | | | | | | Use v3.9 web-console image for nowScott Dodson2018-02-071-1/+1
| | |_|/ / / / | |/| | | | |
* | | | | | | Merge pull request #6938 from vshn/revert-heketi-image-optionOpenShift Merge Robot2018-02-071-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Revert "GlusterFS: Remove image option from heketi command" This reverts commit 3d3853836d71c37a9c81aea8e606c94052439789 which was supposedly a fix for an issue with topology initialization, but that turned out to be a separate issue fixed in commit b8879b5c6731bd51e590b470b1a51fce73db0ee1. By default "heketi-cli setup-openshift-heketi-storage" uses the "heketi/heketi:dev" image while, for OpenShift Container Platform installations, the default image for all other operations is "rhgs3/rhgs-volmanager-rhel7". In environments where only a limited set of Docker registries is available, i.e. only the Red Hat registry but not Docker Hub, using "heketi/heketi:dev" does not work or would require manual intervention. The discussion surrounding the removal of "--image" in PR#5769 involved statements that "--image" was removed from Heketi. As of January 30, 2018 the master branch contains the option, as does the upstream Docker image (Heketi version v5.0.0-225-gffb9fea) and the aforementioned "rhgs3/rhgs-volmanager-rhel7" image. Tested with OpenShift Container Platform 3.7.23.
| * | | | | | | Revert "GlusterFS: Remove image option from heketi command"Michael Hanselmann2018-01-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3d3853836d71c37a9c81aea8e606c94052439789 which was supposedly a fix for an issue with topology initialization, but that turned out to be a separate issue fixed in commit b8879b5c6731bd51e590b470b1a51fce73db0ee1. By default "heketi-cli setup-openshift-heketi-storage" uses the "heketi/heketi:dev" image while, for OpenShift Container Platform installations, the default image for all other operations is "rhgs3/rhgs-volmanager-rhel7". In environments where only a limited set of Docker registries is available, i.e. only the Red Hat registry but not Docker Hub, using "heketi/heketi:dev" does not work or would require manual intervention. The discussion surrounding the removal of "--image" in PR#5769 involved statements that "--image" was removed from Heketi. As of January 30, 2018 the master branch contains the option, as does the upstream Docker image (Heketi version v5.0.0-225-gffb9fea) and the aforementioned "rhgs3/rhgs-volmanager-rhel7" image. Tested with OpenShift Container Platform 3.7.23.
* | | | | | | | 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 #7032 from spadgett/console-livenessScott Dodson2018-02-072-11/+11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Detect config changes in console liveness probe
| * | | | | | | Detect config changes in console liveness probeSamuel Padgett2018-02-062-11/+11
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Use an md5sum to check if the console config has changed and automatically restart the container.
* | | | | | | Merge pull request #7044 from zgalor/fix_uninstall_bugOpenShift Merge Robot2018-02-072-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-072-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>
| * | | | | | 3.9 upgrade: fix typos in restart masters procedureVadim Rutkovsky2018-02-071-2/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 'rolling_restart_mode' should be 'services', not 'service' * use 'state: restarted' to properly restart services Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | | | | | Merge pull request #7007 from abutcher/master-cert-idempotenceScott Dodson2018-02-071-17/+8
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | Correct master certificates list checked by openshift_master_certificates
| * | | | | Correct the list of certificates checked in openshift_master_certificates ↵Andrew Butcher2018-02-051-17/+8
| | | | | | | | | | | | | | | | | | | | | | | | s.t. masters do not incorrectly report that master certs are missing.
* | | | | | Merge pull request #7045 from vrutkovs/quick-installer-new-playbook-pathsOpenShift Merge Robot2018-02-073-35/+108
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Quick installer: run prerequistes first and update path to main playbook Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1542895
| * | | | | | quick installer: disable broken test_get_hosts_to_run_on6 testVadim Rutkovsky2018-02-071-13/+12
| | | | | | |
| * | | | | | Quick installer: run prerequistes first and update path to main playbookVadim Rutkovsky2018-02-073-25/+99
| | |/ / / / | |/| | | |
* | | | | | Merge pull request #7004 from cooktheryan/master-tag-fixOpenShift Merge Robot2018-02-071-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. tag fix without ose- incorrect tag when trying to deploy efs
| * | | | | | tag fix without ose-Ryan Cook2018-02-021-1/+1
| | |/ / / / | |/| | | |
* | | | | | Automatic commit of package [openshift-ansible] release [3.9.0-0.41.0].Justin Pierce2018-02-072-2/+8
| |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | Merge pull request #7050 from jarrpa/gluster-doc-updateOpenShift Merge Robot2018-02-071-11/+11
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. GlusterFS: Minor documentation update
| * | | | | GlusterFS: Minor documentation updateJose A. Rivera2018-02-071-11/+11
| | | | | |
* | | | | | Merge pull request #7040 from sdodson/bz1542399OpenShift Merge Robot2018-02-071-0/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Make sure to include upgrade_pre when upgrading master nodes Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1542399
| * | | | | | Make sure to include upgrade_pre when upgrading master nodesScott Dodson2018-02-061-0/+3
| | | | | | |
* | | | | | | Merge pull request #7051 from sdodson/ovs27-ocp310Scott Dodson2018-02-072-2/+2
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Allow OVS 2.7 in OCP 3.10
| * | | | | | Allow OVS 2.7 in OCP 3.10Scott Dodson2018-02-072-2/+2
|/ / / / / /
* | | | | | Automatic commit of package [openshift-ansible] release [3.9.0-0.40.0].Justin Pierce2018-02-072-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | Merge pull request #6993 from joelddiaz/deprovision_elbsKenny Woodson2018-02-077-29/+91
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | add deprovisioning for ELB (and IAM certs)
| * | | | | add deprovisioning for ELB (and IAM certs)Joel Diaz2018-02-027-29/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add playbooks to handle deleting ELBs and any IAM certs that may have been created during provisioning. redo ELB creation to remove arbitrary wait and just retry until ELB creation succeeds
* | | | | | Merge pull request #6688 from jsanda/termination-grace-periodOpenShift Merge Robot2018-02-071-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. [6632] fix indentation of terminationGracePeriodSeconds var This is a PR to fix [6632](https://github.com/openshift/openshift-ansible/issues/6632).
| * | | | | | [6632] fix indentation of terminationGracePeriodSeconds varJohn Sanda2018-01-101-1/+1
| | | | | | |