summaryrefslogtreecommitdiffstats
path: root/roles/openshift_node
Commit message (Collapse)AuthorAgeFilesLines
* 99-origin-dns.sh: Only modify resolv.conf if upstream DNS is already configuredSuren A. Chilingaryan2018-03-181-1/+2
|
* Fix NetworkManager hook to support OpenShift running on non-default network ↵Suren A. Chilingaryan2018-03-072-5/+20
| | | | interface
* Move installation of packages before container_runtime to ensure bind mounts ↵Kenny Woodson2018-02-133-8/+13
| | | | are avaialable.
* Merge pull request #5251 from celebdor/configurable_node_startOpenShift Merge Robot2018-02-132-5/+12
|\ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Make node start options configurable Thanks to this, the node startup can be customized. Change-Id: Ifb93bf892d7de52a575b8d0231328d44f733d0aa Signed-off-by: Antoni Segura Puimedon <celebdor@gmail.com>
| * Make node start options configurableAntoni Segura Puimedon2017-12-212-5/+12
| | | | | | | | | | | | | | Thanks to this, the node startup can be customized. Change-Id: Ifb93bf892d7de52a575b8d0231328d44f733d0aa Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
* | Merge pull request #7103 from vrutkovs/origin-dns-prepend-cluster-localOpenShift Merge Robot2018-02-121-2/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. origin-dns: make sure cluster.local DNS server is listed first This would ensure glibc resolver won't discard this server when existing DNS server list is longer than 2. This sed would also update existing configuration, where cluster.local was appended Fixes bug https://bugzilla.redhat.com/show_bug.cgi?id=1544001
| * | origin-dns: make sure cluster.local DNS server is listed firstVadim Rutkovsky2018-02-121-2/+3
| | | | | | | | | | | | | | | | | | | | | This would ensure glibc resolver won't discard this server when existing DNS server list is longer than 2. This sed would also update existing configuration, where cluster.local was appended
* | | Merge pull request #7073 from abutcher/aws-node-hostnameOpenShift Merge Robot2018-02-121-8/+0
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Update user-data for FQDN instance hostnames * Remove node service start from bootstrap.yml. We start and enable the node service in user-data and we want the node service to start after NetworkManager so that the instance will have its final hostname. * Restart systemd-hostnamed before restarting NetworkManager. In testing this has ensured that instances receive an FQDN hostname which is required for kube.
| * | Remove node start from bootstrap.yml.Andrew Butcher2018-02-121-8/+0
| | |
* | | Merge pull request #7031 from mgugino-upstream-stage/fix-syscon-varsOpenShift Merge Robot2018-02-081-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | 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>
* | | Fix docker rpm upgrade install task wordingMichael Gugino2018-02-061-1/+1
|/ / | | | | | | This commit corrects wording of task name.
* | dynamic inventory bug when group exists but its emptyMangirdas2018-02-021-0/+1
| |
* | add glusterblock support for ansibleMangirdas2018-01-313-1/+43
| | | | | | | | Co-authored-by: Christina Kyriakidou <Ckyriaki@Redhat.com>
* | node: specify bind option to /root/.dockerGiuseppe Scrivano2018-01-251-0/+1
| | | | | | | | | | | | | | | | | | Without the option, runc fails with "no such device" when trying to create the mount point. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1534933 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #6551 from mgugino-upstream-stage/swapoffScott Dodson2018-01-243-62/+11
|\ \ | | | | | | Create swapoff module
| * | Create swapoff moduleMichael Gugino2018-01-243-62/+11
| | | | | | | | | | | | | | | Create custome ansible module 'swapoff' to idempotently disable swap in /etc/fstab and also run swapoff.
* | | Merge pull request #6843 from rajatchopra/bugdnsmasqScott Dodson2018-01-242-2/+2
|\ \ \ | |/ / |/| | Change dnsmasq Requires to Wants
| * | Change dnsmasq Requires to Wants. ↵Rajat Chopra2018-01-232-2/+2
| | | | | | | | | | | | | | | | | | https://bugzilla.redhat.com/show_bug.cgi?id=1532960 Signed-off-by: Rajat Chopra <rchopra@redhat.com>
* | | Merge pull request #6827 from vrutkovs/node-uppercase-certificatesOpenShift Merge Robot2018-01-231-1/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Automatic merge from submit-queue. Lowercase node names when creating certificates Additional fix for https://github.com/openshift/openshift-ansible/pull/6812
| * | Lowercase node names when creating certificatesVadim Rutkovsky2018-01-231-1/+1
| | |
* | | Merge pull request #6783 from mgugino-upstream-stage/node-syscon-add-mountsScott Dodson2018-01-232-0/+31
|\ \ \ | |/ / |/| | Add ability to mount volumes into system container nodes
| * | Add ability to mount volumes into system container nodesMichael Gugino2018-01-182-0/+31
| | | | | | | | | | | | | | | | | | | | | This commit adds the ability to mount volumes into system containerized nodes. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1534933
* | | Add Azure to support openshift_cloudprovider_kindweherdh2018-01-181-0/+6
|/ /
* | Merge pull request #6426 from danwinship/cleanup-ipamScott Dodson2018-01-131-0/+6
|\ \ | | | | | | Clean up host-local IPAM data while nodes are drained
| * | Clean up host-local IPAM data while nodes are drainedDan Winship2017-12-141-0/+6
| | |
* | | Add defaults for openshift_pkg_versionMichael Gugino2018-01-102-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This variable may or may not be defined by the users. During deployments, it will be set to '-{{ openshift_version }}' if undefined. During upgrades, it will remain undefined. This commit ensures that if the variable is undefined, empty strings '' are set.
* | | Merge pull request #6553 from ↵Michael Gugino2018-01-101-24/+14
|\ \ \ | | | | | | | | | | | | | | | | mgugino-upstream-stage/node-reduce-package-commands Install node packages in one task instead of 3
| * | | Install node packages in one task instead of 3Michael Gugino2018-01-041-24/+14
| | | | | | | | | | | | | | | | | | | | This commit reduces the number of package tasks from 3 to 1.
* | | | 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.