summaryrefslogtreecommitdiffstats
path: root/roles
Commit message (Collapse)AuthorAgeFilesLines
* Only try to yaml.load a file if it ends in .yml or .yaml in logging factsEric Wolinetz2018-02-081-3/+4
|
* 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 #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 #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-071-0/+0
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | 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-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | 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 #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 #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
| |/ / / /
* | | | | 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
| | | | | |
* | | | | | Allow OVS 2.7 in OCP 3.10Scott Dodson2018-02-072-2/+2
|/ / / / /
* | | | | Merge pull request #6993 from joelddiaz/deprovision_elbsKenny Woodson2018-02-076-29/+82
|\ \ \ \ \ | |_|/ / / |/| | | | add deprovisioning for ELB (and IAM certs)
| * | | | add deprovisioning for ELB (and IAM certs)Joel Diaz2018-02-026-29/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| | | | | |
* | | | | | Merge pull request #7035 from sosiouxme/20180206-ovs-2.9OpenShift Merge Robot2018-02-063-9/+17
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. health checks: tolerate ovs 2.9 fixes https://bugzilla.redhat.com/show_bug.cgi?id=1541313
| * | | | | | health checks: tolerate ovs 2.9Luke Meyer2018-02-063-9/+17
| | |_|_|/ / | |/| | | |
* | | | | | Merge pull request #7015 from sdodson/v310Scott Dodson2018-02-06197-31/+96215
|\ \ \ \ \ \ | | | | | | | | | | | | | | Initial support for 3.10
| * | | | | | Initial support for 3.10Scott Dodson2018-02-05197-31/+96215
| | | | | | |
* | | | | | | Fix docker rpm upgrade install task wordingMichael Gugino2018-02-061-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | This commit corrects wording of task name.
* | | | | | Merge pull request #6628 from ewolinetz/es_5.x_opt_inOpenShift Merge Robot2018-02-0653-116/+1690
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Es 5.x opt in FYI @richm @jcantrill
| * | | | | | Swapping container order for es podEric Wolinetz2018-02-052-73/+73
| | | | | | |
| * | | | | | Adding support for ES 5.x tech preview opt inEric Wolinetz2018-02-0553-79/+1653
| | | | | | |
* | | | | | | Merge pull request #6990 from joelddiaz/aws_master_grp_paramsKenny Woodson2018-02-061-21/+19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | parameterize various master/infra scale group bits
| * | | | | | | parameterize various master scale group bitsJoel Diaz2018-02-051-21/+19
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | allow selectively overriding master scale group bits: instance type: openshift_aws_master_group_instance_type scale group min size: openshift_aws_master_group_min_size scale group max size: openshift_aws_master_group_max_size scale group desired size: openshift_aws_master_group_desired_size and overriding infra scale group bits: instance type: openshift_aws_infra_group_instance_type scale group min size: openshift_aws_infra_group_min_size scale group max size: openshift_aws_infra_group_max_size scale group desired size: openshift_aws_infra_group_desired_size and overriding compute scale group bits: instance type: openshift_aws_compute_group_instance_type scale group min size: openshift_aws_compute_group_min_size scale group max size: openshift_aws_compute_group_max_size scale group desired size: openshift_aws_compute_group_desired_size also consolodate the health check for master/infra/compute since they are all the same
* | | | | | | Merge pull request #6975 from pgier/bz1540799OpenShift Merge Robot2018-02-062-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. bug 1540799: openshift_prometheus: update alertmanager config file flag Prometheus alertmanager as of 0.13.0 requires double dashes (posix) for command line flags. The double dashes should be backwards compat with earlier versions of alertmanager.
| * | | | | | | bug 1540799: openshift_prometheus: update alertmanager config file flagPaul Gier2018-02-052-2/+2
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prometheus alertmanager as of 0.13.0 requires double dashes (posix) for command line flags. The double dashes should be backwards compat with earlier versions of alertmanager.
* | | | | | | Merge pull request #7016 from kwoodson/daemonset_monitoringOpenShift Merge Robot2018-02-063-18/+34
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. daemonset config modifications to allow monitoring. Modifications that allow custom secrets, configmaps, and shared setup between containers for config and monitoring.
| * | | | | | | Updates to configure monitoring container.Kenny Woodson2018-02-053-18/+34
| | | | | | | |
* | | | | | | | Merge pull request #6933 from giuseppe/bz-1529478Scott Dodson2018-02-051-3/+3
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | cri-o: export variables defined in crio-network
| * | | | | | | cri-o: export variables defined in crio-networkGiuseppe Scrivano2018-01-301-3/+3
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | so that the cri-o process can use them. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1529478 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | | | Merge pull request #6731 from Klaas-/klaas-patch1OpenShift Merge Robot2018-02-051-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix broken softlink I think d3fefc32a727fe3c13159c4e9fe4399f35b487a8 moved dirs and failed to correct the symlinks inside
| * | | | | | Fix softlinks broken by d3fefc32a727fe3c13159c4e9fe4399f35b487a8Klaas Demter2018-01-261-1/+1
| | | | | | |
* | | | | | | Merge pull request #6985 from joelddiaz/deprovision_s3_bucketsKenny Woodson2018-02-052-0/+31
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | add S3 bucket cleanup
| * | | | | | | add S3 bucket cleanupJoel Diaz2018-02-022-0/+31
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default to just cleaning out all the objects in the S3 bucket (IFF openshift_aws_create_s3 is 'true'). If you really, trully want to delete the S3 bucket and free up the bucket name, you can set openshift_aws_really_delete_s3_bucket to 'true' ('false' by default).
* | | | | | | Merge pull request #6969 from mjudeikis/gluster-dynamic-invetory-bugOpenShift Merge Robot2018-02-042-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. dynamic inventory bug when group exists but its empty @jarrpa output from my test for the gluster block I use dynamic inventory. And the result of this is that ansible see all possible groups available, but some of them are empty [1]. Because group exists, fist condition is evaluated as true and it fails with the array is an empty error... ``` ... 'all': [u'54.144.5.175', u'52.200.175.54', u'54.237.208.252', u'52.55.224.72', u'52.207.103.246', u'34.227.95.191', u'34.238.127.160'], u'bastion': [u'34.227.95.191'], u'bastion.0': [u'34.227.95.191'], u'etcd': [u'34.238.127.160'], u'glusterfs': [u'52.200.175.54', u'52.207.103.246', u'54.144.5.175'], u'glusterfs.0': [u'52.200.175.54'], u'glusterfs.1': [u'52.207.103.246'], u'glusterfs.2': [u'54.144.5.175'], u'glusterfs_registry': [], u'infra': [u'52.55.224.72'], u'infra.0': [u'52.55.224.72'], u'masters': [u'34.238.127.160'], u'masters.0': [u'34.238.127.160'], ... ```
| * | | | | | | dynamic inventory bug when group exists but its emptyMangirdas2018-02-022-2/+2
| | | | | | | |
* | | | | | | | Merge pull request #6980 from mtnbikenc/fix-upgradesScott Dodson2018-02-043-2/+4
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix version handling in 3.8/3.9 control plane upgrades
| * | | | | | | | Fix version handling in 3.8/3.9 control plane upgradesRussell Teague2018-02-023-2/+4
| | |_|_|_|/ / / | |/| | | | | |
* | | | | | | | Merge pull request #6996 from mjudeikis/restart_fix_atomicOpenShift Merge Robot2018-02-032-1/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. check for restart on atomic fix atomic host storage
| * | | | | | | | dynamic inventory bug when group exists but its emptyMangirdas2018-02-022-1/+5
| | |/ / / / / / | |/| | | | | |