summaryrefslogtreecommitdiffstats
path: root/roles/openshift_node
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #6658 from mgugino-upstream-stage/containerized_boolsOpenShift Merge Robot2018-01-091-1/+1
|\ | | | | | | | | Automatic merge from submit-queue. ensure containerized bools are cast
| * ensure containerized bools are castMichael Gugino2018-01-081-1/+1
| |
* | Merge pull request #6650 from ↵OpenShift Merge Robot2018-01-091-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 upgradeVadim Rutkovsky2018-01-091-1/+1
| | | | | | | | | | | | Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | | Merge pull request #6653 from mgugino-upstream-stage/fix-crio-boolMichael Gugino2018-01-086-8/+8
|\ \ \ | |_|/ |/| | Properly cast crio boolean variables to bool
| * | Properly cast crio boolean variables to boolMichael Gugino2018-01-086-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 #6549 from mgugino-upstream-stage/node-meta-depends2OpenShift Merge Robot2018-01-081-2/+0
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 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-dependsMichael Gugino2018-01-021-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Remove last non-taskless meta-depends from openshift_node role. Remove variable 'openshift_node_upgrade_in_progress' as it is no longer used.
* | | Don't overwrite node's systemd units for containerized installVadim Rutkovsky2018-01-051-1/+1
| |/ |/| | | | | | | | | Systemd units are being updated in ../systemd_units.yml Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | Merge pull request #6586 from ↵OpenShift Merge Robot2018-01-031-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgugino-upstream-stage/cast-bool-openshift_docker_use_system_container Automatic merge from submit-queue. Cast openshift_docker_use_system_container to bool 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
| * | Cast openshift_docker_use_system_container to boolMichael Gugino2018-01-021-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 #6581 from mgugino-upstream-stage/fix-syscontainer-reg-authOpenShift Merge Robot2018-01-021-1/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Add docker auth credentials to system container install This commit adds docker auth credentials mount to system container systemd unit file. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1514324
| * | | Add docker auth credentials to system container installMichael Gugino2018-01-021-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds docker auth credentials mount to system container systemd unit file. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1514324
* | | | Merge pull request #6583 from mgugino-upstream-stage/aws-creds-locationOpenShift Merge Robot2018-01-021-3/+3
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Correct kublet_args cloud-provider directories This commit updates kublet_args to correct cloudprovider configuration file locaitons. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1527203
| * | | Correct kublet_args cloud-provider directoriesMichael Gugino2018-01-021-3/+3
| |/ / | | | | | | | | | | | | | | | | | | This commit updates kublet_args to correct cloudprovider configuration file locaitons. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1527203
* / / Remove bootstrap.yml from main.yml in openshift_node roleMichael Gugino2017-12-211-4/+0
|/ / | | | | | | | | | | This commit utilizes include_role for bootstrapping the node instead of conditional include of tasks now that the node role has no meta includes that have tasks.
* / Remove references to deployment_typeMichael Gugino2017-12-212-2/+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.
* Remove openshift.common.{is_atomic|is_containerized}Michael Gugino2017-12-2017-26/+26
| | | | | We set these variables using facts in init, no need to duplicate the logic all around the codebase.
* Relocate filter plugins to lib_utilsMichael Gugino2017-12-184-7/+4
| | | | | | | | | | | | | | 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.
* Remove openshift_node_facts roleMichael Gugino2017-12-188-12/+24
| | | | | This commit removes the remainder of openshift_node_facts role.
* Merge pull request #6456 from mgugino-upstream-stage/node-factsMichael Gugino2017-12-178-17/+64
|\ | | | | Remove openshift_node_facts part 1
| * Remove openshift_node_facts part 1Michael Gugino2017-12-148-17/+64
| | | | | | | | | | This commit removes some items from openshift_facts for the openshit_node role.
* | Deprecate using Ansible tests as filtersRussell Teague2017-12-1418-32/+34
| |
* | Merge pull request #6449 from giuseppe/crio-change-socket-pathScott Dodson2017-12-141-2/+2
|\ \ | |/ |/| crio: change socket path to /var/run/crio/crio.sock
| * crio: change socket path to /var/run/crio/crio.sockGiuseppe Scrivano2017-12-121-2/+2
| | | | | | | | | | | | it is required for OpenShift 3.9 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | upgrade node mark 2Michael Gugino2017-12-1211-174/+219
| |
* | Refactor node upgrade to include less serial tasksMichael Gugino2017-12-129-189/+196
| | | | | | | | | | | | | | | | | | | | | | | | | | This commit moves the pulling of images, packages, and updating config files into a non-serialized play. The serialized play is now in charge of marking unschedulable, draining, stopping and restarting services, and marking schedulable. If rpm install / container download takes 60s per host, this will save 3 hours and 10 minutes at 200 hosts per cluster and forks of 20 hosts.
* | Refactor upgrade codepaths step 1Michael Gugino2017-12-121-15/+0
|/ | | | | | | | | | | | | | | | 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.
* openshift_node: reintroduce restart of CRI-O.Giuseppe Scrivano2017-12-081-0/+9
| | | | | | change introduced with 39cf5084f18e2e0adca46b925660a6f2c38d227c Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* Remove openshift.common.service_typeMichael Gugino2017-12-0717-46/+50
| | | | | | | | This commit removes openshift.common.service_type in favor of openshift_service_type. This commit also removes r_openshift_excluder_service_type from plays in favor of using the role's defaults.
* Merge pull request #6380 from sdodson/oc-admScott Dodson2017-12-071-3/+3
|\ | | | | Remove all uses of openshift.common.admin_binary
| * Remove all uses of openshift.common.admin_binaryScott Dodson2017-12-071-3/+3
| | | | | | | | Replace with `oc adm`
* | Merge pull request #6362 from mgugino-upstream-stage/crt-playsScott Dodson2017-12-073-16/+0
|\ \ | | | | | | Implement container_runtime playbooks and changes
| * | Implement container_runtime playbooks and changesMichael Gugino2017-12-073-16/+0
| |/ | | | | | | | | | | 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-1/+1
|/
* Fix oreg_auth_credentials_create register varMichael Gugino2017-12-051-2/+6
| | | | | | | | | | | | There is a variable collision for the *oreg_auth_credentials_create variables in openshift_master and openshift_node registry_create tasks. This commit ensures standard and alternative oreg auth credential placement tasks don't use the same register variable. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1520866
* Implement container runtime roleMichael Gugino2017-12-0111-32/+27
|
* retry package operationsLuke Meyer2017-11-3011-0/+28
| | | | | | When a package install/update fails due to network blips or other spotty availability, retry it. If the failure is a real failure (e.g. package is really not there) it still fails after 3 tries (Ansible default).
* Merge pull request #6269 from mgugino-upstream-stage/clock-dependsMichael Gugino2017-11-281-2/+0
|\ | | | | Remove openshift_clock from meta depends
| * Remove openshift_clock from meta dependsMichael Gugino2017-11-271-2/+0
| | | | | | | | | | This commit adds openshift_clock role to required plays instead of using meta_depends.
* | Combine openshift_node and openshift_node_dnsmasqMichael Gugino2017-11-2711-3/+251
|/ | | | | This commit combines these two roles. This will prevent openshift_node_facts from running twice.
* Merge pull request #6198 from mgugino-upstream-stage/openvswitch-sys-containerMichael Gugino2017-11-272-2/+3
|\ | | | | Remove is_openvswitch_system_container from facts
| * Remove is_openvswitch_system_container from factsMichael Gugino2017-11-222-2/+3
| | | | | | | | Simply logic for this variable.
* | Include Deprecation - openshift-nodeRussell Teague2017-11-226-30/+27
|/
* Fix system_images_registry variableMichael Gugino2017-11-223-4/+10
| | | | Simplify the logic for this variable.
* Fix node system container varMichael Gugino2017-11-213-3/+5
| | | | Fix logic, make it simpler.
* Combine openshift_node and openshift_node_upgradeMichael Gugino2017-11-167-0/+326
| | | | | | | | | Currently, having openshift_node and openshift_node_upgrade as two distinct roles has created a duplication across handlers, templates, and some tasks. This commit combines the roles to reduce duplication and bugs encountered by not putting code in both places.
* Merge pull request #5990 from kwoodson/ensure_node_startedOpenShift Merge Robot2017-11-161-0/+8
|\ | | | | | | | | | | | | Automatic merge from submit-queue. Ensure node service is started. When bootstrapping nodes we need to restart networkmanager so that the /etc/resolv.conf is properly populated. We also restart the node service after making changes to the unit files.
| * Ensure node service is started.Kenny Woodson2017-11-151-0/+8
| |
* | Merge pull request #5968 from mgugino-upstream-stage/version-scrubScott Dodson2017-11-151-2/+0
|\ \ | |/ |/| Removed old version code