summaryrefslogtreecommitdiffstats
path: root/roles/container_runtime
Commit message (Collapse)AuthorAgeFilesLines
* Fix docker_upgrade variableMichael Gugino2018-02-122-8/+14
| | | | | | | | | Currently, docker_upgrade is ignored during cluster upgrades. This commit ensures that the variable is respected. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1543714
* additional mounts: specify 'type' in container_runtime_crio_additional_mountsVadim Rutkovsky2018-02-121-0/+1
|
* Adding ability to provide additional mounts to crio system container.Kenny Woodson2018-02-072-0/+12
|
* 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 #6905 from mgugino-upstream-stage/fix-crt-typoOpenShift Merge Robot2018-01-271-1/+1
|\ | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix container_runtime variable typo 'Centos' should be 'CentOS' as it corresponds to the ansible variable that determines the linux distro.
| * Fix container_runtime variable typoMichael Gugino2018-01-261-1/+1
| | | | | | | | | | | | 'Centos' should be 'CentOS' as it corresponds to the ansible variable that determines the linux distro.
* | Removing dependency on the extra stroage device.Kenny Woodson2018-01-252-1/+5
|/
* Merge pull request #6682 from giuseppe/fix-registriesMichael Gugino2018-01-231-25/+14
|\ | | | | CRI-O, Docker: Use images from the CentOS and Fedora registry
| * docker: use image from CentOS and Fedora registriesGiuseppe Scrivano2018-01-101-6/+6
| | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * crio: use Docker and CentOS registries for the imageGiuseppe Scrivano2018-01-101-19/+8
| |
* | Ensure atomic_proxies are configured with dockerMichael Gugino2018-01-171-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 #6623 from vrutkovs/avoid-repoquery-on-containerizedOpenShift Merge Robot2018-01-111-9/+16
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. docker_upgrade_check: skip repoquery calls on containerized setups Fedora/CentOS Atomic hosts don't have repoquery installed - also there is no need to check available Docker rpms in the repos for Atomic Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1527256 Note, that this solution is pretty naive - a better idea would be splitting the playbooks for rpm and containerized installs for clarity
| * docker_upgrade_check: skip repoquery calls on containerized setupsVadim Rutkovsky2018-01-051-9/+16
| | | | | | | | | | Fedora/CentOS Atomic hosts don't have repoquery installed - also there is no need to check available Docker rpms in the repos for Atomic
* | Merge pull request #6646 from giuseppe/fix-container-engine-authOpenShift Merge Robot2018-01-091-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 pullGiuseppe Scrivano2018-01-081-0/+6
| | | | | | | | | | | | | | | | | | | | | so that the atomic pull takes into account the credentials if required. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | Properly cast crio boolean variables to boolMichael Gugino2018-01-082-2/+2
|/ / | | | | | | | | | | | | | | | | | | | | 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 #6627 from sdodson/import_roleScott Dodson2018-01-053-4/+4
|\ \ | | | | | | Migrate to import_role for static role inclusion
| * | Migrate to import_role for static role inclusionScott Dodson2018-01-053-4/+4
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Ansible 2.2, the include_role directive came into existence as a Tech Preview. It is still a Tech Preview through Ansible 2.4 (and in current devel branch), but with a noteable change. The default behavior switched from static: true to static: false because that functionality moved to the newly introduced import_role directive (in order to stay consistent with include* being dynamic in nature and `import* being static in nature). The dynamic include is considerably more memory intensive as it will dynamically create a role import for every host in the inventory list to be used. (Also worth noting, there is at the time of this writing an object allocation inefficiency in the dynamic include that can in certain situations amplify this effect considerably) This change is meant to mitigate the pressure on memory for the Ansible control host. We need to evaluate where it makes sense to dynamically include roles and revert back to dynamic inclusion if and where it makes sense to do so.
* | crio: create /etc/sysconfig/crio-storageGiuseppe Scrivano2018-01-051-0/+6
| | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | crio: configure proxy variablesGiuseppe Scrivano2018-01-052-0/+14
|/ | | | | | Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1529478 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* Cast openshift_docker_use_system_container to boolMichael Gugino2018-01-021-1/+1
| | | | | | | | | | | | | openshift_docker_use_system_container might be passed in via ini inventory as 'openshift_docker_use_system_container=false' This condition will be interpreted as a string type, instead of boolean. Casting openshift_docker_use_system_container as bool will achieve the users desired intent. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1528943
* Merge pull request #6497 from kwoodson/remove_docker_credsOpenShift Merge Robot2017-12-231-0/+2
|\ | | | | | | | | | | | | Automatic merge from submit-queue. Adding no_log to registry_auth. We were seeing credentials show up in our jenkins logs. We'd prefer if they were not logged. I'm not sure the repercussions but these are sensitive and probably should be `no_log`.
| * Adding no_log to registry_auth.Kenny Woodson2017-12-151-0/+2
| |
* | Remove references to deployment_typeMichael Gugino2017-12-211-0/+1
| | | | | | | | | | | | | | Move openshift_deployment_type check into sanity_check action plugin. Remove compatibility for deployment_type. deployment_type has been deprecated for some time now.
* | Move validate_hosts to prerequisites.ymlMichael Gugino2017-12-201-2/+0
| | | | | | | | | | | | Move more checks outside of init/main.yml for speeding up upgrades and other operational plays that need to run.
* | Remove openshift.common.{is_atomic|is_containerized}Michael Gugino2017-12-205-10/+10
| | | | | | | | | | We set these variables using facts in init, no need to duplicate the logic all around the codebase.
* | Merge pull request #6524 from kwoodson/docker_storage_setup_overlayScott Dodson2017-12-203-0/+36
|\ \ | | | | | | Adding support for docker-storage-setup on overlay
| * | Adding support for docker-storage-setup on overlayKenny Woodson2017-12-193-0/+36
| |/
* | Relocate filter plugins to lib_utilsMichael Gugino2017-12-183-4/+3
|/ | | | | | | | | | | | | | This commit relocates filter_plugings to lib_utils, changes the namespacing to prevent unintended use of older versions that may be present in filter_plugins/ directory on existing installs. Add lib_utils to meta depends for roles Also consolidate some plugins into lib_utils from various other areas. Update rpm spec, obsolete plugin rpms.
* Deprecate using Ansible tests as filtersRussell Teague2017-12-146-22/+47
|
* Merge pull request #6449 from giuseppe/crio-change-socket-pathScott Dodson2017-12-141-1/+1
|\ | | | | crio: change socket path to /var/run/crio/crio.sock
| * crio: change socket path to /var/run/crio/crio.sockGiuseppe Scrivano2017-12-121-1/+1
| | | | | | | | | | | | it is required for OpenShift 3.9 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #6467 from giuseppe/use-fedoraproject-latestRussell Teague2017-12-141-1/+1
|\ \ | | | | | | containers: use registry.fedoraproject.org/latest instead of registry.fedoraproject.org/f2.X
| * | docker: use Fedora /latest/ instead of hardcoding the versionGiuseppe Scrivano2017-12-131-1/+1
| |/ | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Refactor upgrade codepaths step 1Michael Gugino2017-12-121-0/+67
|/ | | | | | | | | | | | | | | | This commit refactors some upgrade code paths. Touched areas are: 1) Reduces usage of 'oo_all_hosts' in various places, especially when running upgrade_control_plane. 2) Reuses common code across the various upgrade* playbooks. 3) Moves docker upgrade checks into container_runtime_role. 4) Combines smaller playbooks and plays to reduce file sprawl.
* container-engine: skip openshift_docker_log_driver when it is FalseGiuseppe Scrivano2017-12-081-1/+1
| | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* container-engine: log-opts is a dictionary in the daemon.json fileGiuseppe Scrivano2017-12-082-1/+2
| | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* crio: define and use l_is_node_system_containerGiuseppe Scrivano2017-12-082-1/+4
| | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* Merge pull request #6362 from mgugino-upstream-stage/crt-playsScott Dodson2017-12-0713-269/+254
|\ | | | | Implement container_runtime playbooks and changes
| * Implement container_runtime playbooks and changesMichael Gugino2017-12-0713-269/+254
| | | | | | | | | | | | This commit refactors some duplicate code, removes usage of set_fact where not needed, and reorganizes container_runtime role to use include_role.
* | Include Deprecation: Convert to include_tasksRussell Teague2017-12-061-2/+1
|/
* Implement container runtime roleMichael Gugino2017-12-0119-0/+1097