csa/devops/ansible-patches/openshift.git - Unnamed repository; edit this file 'description' to name the repository.

	Commit message (Collapse)	Author	Age	Files	Lines
*	docker_creds: fix python3 exception	Giuseppe Scrivano	2018-01-10	1	-1/+1
\| \| \| \| \| \| \| \|	Solves this exception with python3: TypeError: a bytes-like object is required, not 'str Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
*	docker: use image from CentOS and Fedora registries	Giuseppe Scrivano	2018-01-10	1	-6/+6
\| \| \| \|	Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
*	crio: use Docker and CentOS registries for the image	Giuseppe Scrivano	2018-01-10	1	-19/+8
\|
*	Merge pull request #6674 from mgugino-upstream-stage/remove-becomes2	Scott Dodson	2018-01-10	28	-56/+36
\|\ \| \| \| \|	Remove become statements
\| *	Chmod temp dirs created on localhost	Michael Gugino	2018-01-09	8	-6/+33
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	After remove become:no statements on local_action tasks, we need to ensure that the proper file permssions are applied to local temp directories. This reason for this is that the 'fetch' module does not use 'become' for the localhost, just the remote host. Additionally, users may not wish for the localhost to become during a fetch. local_action will execute with whatever permissions are specified in inventory or via cli.
\| *	Remove become statements	Michael Gugino	2018-01-09	26	-50/+3
\| \| \| \| \| \| \| \| \| \|	This commit removes become:no statements that break the installer in various ways.
* \|	Merge pull request #6671 from mgugino-upstream-stage/upgrade-cp-scope	OpenShift Merge Robot	2018-01-10	17	-248/+210
\|\ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. Limit host group scope on control-plane upgrades This commit limits common init code to exclude oo_nodes_to_config during upgrade_control_plane runs.
\| * \|	Limit host group scope on control-plane upgrades	Michael Gugino	2018-01-09	7	-2/+29
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This commit limits common init code to exclude oo_nodes_to_config during upgrade_control_plane runs.
\| * \|	Refactor version and move some checks into sanity_checks.py	Michael Gugino	2018-01-09	11	-246/+181
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This commit changes how we handle openshift_version role. Most of the version initialization code is only run on the first master now. All other hosts have values set from the master. Aftwards, we run some basic RPM queries to ensure that the correct version is available on the other nodes. Containerized needs to do their own image checks elsewhere.
* \| \|	Merge pull request #6602 from ewolinetz/es_full_cluster_restart	OpenShift Merge Robot	2018-01-10	6	-1/+117
\|\ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. Adding logic to do a full cluster restart if we are incrementing our … …major versions of ES This will help with the upgrade from 2.x to 5.x for ES, it also fixes something I came across with the handler on 3.7 where it checks the prior deployed version of the ES pod rather than the new one.
\| * \| \|	Adding logic to disable and reenable external communication to ES during ↵	Eric Wolinetz	2018-01-05	2	-2/+35
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	full restart
\| * \| \|	Adding logic to do a full cluster restart if we are incrementing our major ↵	Eric Wolinetz	2018-01-03	6	-1/+84
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	versions of ES
* \| \| \|	Merge pull request #5853 from imcsk8/flannel-iptables	OpenShift Merge Robot	2018-01-09	2	-0/+14
\|\ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. Add iptables rules for flannel [WIP] When using flannel there are iptables rules that need to be added as stated here: https://access.redhat.com/documentation/en-us/reference_architectures/2017/html-single/deploying_red_hat_openshift_container_platform_3.4_on_red_hat_openstack_platform_10/#run_ansible_installer Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1493955
\| * \| \| \|	Add iptables save handler	Ivan Chavero	2018-01-08	1	-0/+4
\| \| \| \| \|
\| * \| \| \|	Fix wrong indentation	Ivan Chavero	2017-10-27	1	-2/+2
\| \| \| \| \|
\| * \| \| \|	Fix yaml indentation	Ivan Chavero	2017-10-27	1	-2/+2
\| \| \| \| \|
\| * \| \| \|	Add iptables rules for flannel	Ivan Chavero	2017-10-23	1	-0/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[WIP] When using flannel there are iptables rules that need to be added as stated here: https://access.redhat.com/documentation/en-us/reference_architectures/2017/html-single/deploying_red_hat_openshift_container_platform_3.4_on_red_hat_openstack_platform_10/#run_ansible_installer Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1493955
* \| \| \| \|	Merge pull request #6658 from mgugino-upstream-stage/containerized_bools	OpenShift Merge Robot	2018-01-09	4	-4/+4
\|\ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. ensure containerized bools are cast
\| * \| \| \| \|	ensure containerized bools are cast	Michael Gugino	2018-01-08	4	-4/+4
\| \| \| \| \| \|
* \| \| \| \| \|	Merge pull request #6646 from giuseppe/fix-container-engine-auth	OpenShift Merge Robot	2018-01-09	1	-0/+6
\|\ \ \ \ \ \ \| \|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. container-engine: move registry_auth.yml before pull so that the atomic pull takes into account the credentials if required. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
\| * \| \| \| \|	container-engine: move registry_auth.yml before pull	Giuseppe Scrivano	2018-01-08	1	-0/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	so that the atomic pull takes into account the credentials if required. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* \| \| \| \| \|	Merge pull request #6618 from ture-karlsson/master	Scott Dodson	2018-01-09	1	-0/+4
\|\ \ \ \ \ \ \| \|_\|_\|_\|_\|/ \|/\| \| \| \| \|	Provide example on how to use osm_etcd_image
\| * \| \| \| \|	Provide example on how to use osm_etcd_image in a disconnected and ↵	Ture Karlsson	2018-01-05	1	-0/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	containerized installation Example of how to use osm_etcd_image in accordance to: https://docs.openshift.com/container-platform/3.7/install_config/install/rpm_vs_containerized.html
* \| \| \| \| \|	Merge pull request #6634 from vrutkovs/openshift_binary_for_upgrades	Scott Dodson	2018-01-09	1	-0/+2
\|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \|	upgrades: set openshift_client_binary fact when running on oo_first_master host
\| * \| \| \| \| \|	upgrades: set openshift_client_binary fact when running on oo_first_master host	Vadim Rutkovsky	2018-01-06	1	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This sets openshift_client_binary var for the first master, as some roles use this var along with first_master_client_binary. Not sure if its worth setting this var for the faulty roles instead though. Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* \| \| \| \| \| \|	Merge pull request #6650 from ↵	OpenShift Merge Robot	2018-01-09	1	-1/+1
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	vrutkovs/containerized_upgrade_set_openshift_use_openshift_sdn Automatic merge from submit-queue. upgrades: use openshift_node_use_openshift_sdn when trying to pre-pull the image This affects 3.8/3.9 upgrades for containerized hosts, if nodes are separate from master.
\| * \| \| \| \| \| \|	Use openshift_node_use_openshift_sdn when doing a containerized node upgrade	Vadim Rutkovsky	2018-01-09	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* \| \| \| \| \| \| \|	Merge pull request #6659 from joelddiaz/ami_and_docker_storage_setup	Kenny Woodson	2018-01-09	2	-1/+6
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	docker storage setup for ami building
\| * \| \| \| \| \| \| \|	docker storage setup for ami building	Joel Diaz	2018-01-08	2	-1/+6
\| \| \|_\|_\|_\|/ / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	add host to g_new_node_hosts so that plays run against the AMI instance update example vars so that overlay2 is used by default for docker storage
* \| \| \| \| \| \| \|	Merge pull request #6660 from mgugino-upstream-stage/fix-logging-static	Michael Gugino	2018-01-08	1	-1/+3
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix: change import_role to include_role
\| * \| \| \| \| \| \| \|	Fix: change import_role to include_role	Michael Gugino	2018-01-08	1	-1/+3
\| \|/ / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It appears that when one role dynamically imports another, usage of import_role inside the dynamically included role is not possible. If something is included with include_role (dynamic), all tasks therein must also use include_role (dynamic).
* \| \| \| \| \| \| \|	Merge pull request #6651 from mgugino-upstream-stage/containerized-groups	Michael Gugino	2018-01-08	4	-7/+16
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Build containerized host group dynamically
\| * \| \| \| \| \| \| \|	Build containerized host group dynamically	Michael Gugino	2018-01-08	4	-7/+16
\| \| \|/ / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Currently, we are using some inventory variables to determine what host groups should be considered containerized. This is problematic and has several edge cases. This commit removes the variable l_containerized_host_groups and builds a dynamic group of hosts named 'oo_hosts_containerized_managed_true' based on the value of 'containerized'
* \| \| \| \| \| \| \|	Merge pull request #6653 from mgugino-upstream-stage/fix-crio-bool	Michael Gugino	2018-01-08	10	-15/+15
\|\ \ \ \ \ \ \ \ \| \|_\|/ / / / / / \|/\| \| \| \| \| \| \|	Properly cast crio boolean variables to bool
\| * \| \| \| \| \| \|	Properly cast crio boolean variables to bool	Michael Gugino	2018-01-08	10	-15/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Variables that are specifically booleans should be cast to bool. This is because users may sometimes pass them as string values. This is particularly prevalent when using ini-style inventories. Affected-by: https://github.com/ansible/ansible/issues/34591 Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1531592
* \| \| \| \| \| \| \|	Merge pull request #6580 from tomassedovic/openstack-fixes	OpenShift Merge Robot	2018-01-08	3	-2/+10
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. Openstack fixes This includes a few fixes for the OpenStack provider. It should fix #6555 and possibly also #6560.
\| * \| \| \| \| \| \| \|	Import prerequisites.yml for OpenStack	Tomas Sedovic	2018-01-02	1	-0/+3
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Set the correct path to the openstack.conf file	Tomas Sedovic	2018-01-02	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The file exists in `/etc/origin/cloudprovider/openstack.conf` not `/etc/origin/openstack.conf`.
\| * \| \| \| \| \| \| \|	Return a openshift_node_labels as a dict	Tomas Sedovic	2018-01-02	1	-0/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The OpenStack dynamic inventory was setting the `openshift_node_labels` value as a string which causes a failure with the `lib_utils_oo_dict_to_keqv_list` filter. Fixes #6555
\| * \| \| \| \| \| \| \|	Fix yaml syntax error in the sample inventory	Tomas Sedovic	2018-01-02	1	-2/+2
\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #6649 from mgugino-upstream-stage/fix-pre-packages	Michael Gugino	2018-01-08	1	-2/+2
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	install base_packages on oo_all_hosts
\| * \| \| \| \| \| \| \| \|	install base_packages on oo_all_hosts	Michael Gugino	2018-01-08	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This commit ensures base packages are installed for oo_all_hosts, which is what we were doing previously. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1530516
* \| \| \| \| \| \| \| \| \|	Merge pull request #6549 from mgugino-upstream-stage/node-meta-depends2	OpenShift Merge Robot	2018-01-08	5	-8/+2
\|\ \ \ \ \ \ \ \ \ \ \| \|_\|_\|/ / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. Remove last of openshift_node role meta-depends Remove last non-taskless meta-depends from openshift_node role.
\| * \| \| \| \| \| \| \| \|	Remove last of openshift_node role meta-depends	Michael Gugino	2018-01-02	5	-8/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove last non-taskless meta-depends from openshift_node role. Remove variable 'openshift_node_upgrade_in_progress' as it is no longer used.
* \| \| \| \| \| \| \| \| \|	Merge pull request #6548 from kwoodson/configurable_ami_drive_size	Kenny Woodson	2018-01-08	1	-5/+1
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Adding ability to update ami drive size.
\| * \| \| \| \| \| \| \| \| \|	Adding ability to update ami drive size.	Kenny Woodson	2017-12-21	1	-5/+1
\| \|/ / / / / / / / /
* \| \| \| \| \| \| \| \| \|	Merge pull request #6624 from vrutkovs/containerized-avoid-replacing-node-unit	OpenShift Merge Robot	2018-01-08	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatic merge from submit-queue. Don't overwrite node's systemd units for containerized install Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1527849 Perphaps this block should be removed, unless I'm missing some other case for it, as systemd units are being updated in ../systemd_units.yml.
\| * \| \| \| \| \| \| \| \|	Don't overwrite node's systemd units for containerized install	Vadim Rutkovsky	2018-01-05	1	-1/+1
\| \| \|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Systemd units are being updated in ../systemd_units.yml Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* \| \| \| \| \| \| \| \|	Merge pull request #6641 from sdodson/logging-loops	Michael Gugino	2018-01-08	1	-4/+4
\|\ \ \ \ \ \ \ \ \ \| \|_\|_\|/ / / / / / \|/\| \| \| \| \| \| \| \|	Switch back to dynamic include_role in logging loops
\| * \| \| \| \| \| \| \|	Switch back to dynamic include_role in logging loops	Scott Dodson	2018-01-07	1	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We'd switched to import_role to avoid increased memory consumption but we must use include_role whenever we loop.