summaryrefslogtreecommitdiffstats
path: root/roles/openshift_node_upgrade
Commit message (Collapse)AuthorAgeFilesLines
* Combine openshift_node and openshift_node_upgradeMichael Gugino2017-11-1625-786/+0
| | | | | | | | | 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 #5968 from mgugino-upstream-stage/version-scrubScott Dodson2017-11-151-2/+0
|\ | | | | Removed old version code
| * Removed old version codeMichael Gugino2017-11-141-2/+0
| | | | | | | | | | | | This commit removes any references to versions < 1.5/3.5 We assume the version is always greater than or equal to 1.5/3.5.
* | Merge pull request #6094 from ↵OpenShift Merge Robot2017-11-152-0/+20
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgugino-upstream-stage/syscontainers-docker-login-module Automatic merge from submit-queue. Alternative method to create docker registry auth creds Currently, the command 'docker login' is run when using oreg with authentication. On some hosts, such as hosts configured to use system containers, the docker service is not running. 'docker login' will fail without the docker service running. This commit adds a module to idempotently add credentials to the registry credentials file without the use of 'docker login' Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1511374
| * Alternative method to create docker registry auth credsMichael Gugino2017-11-142-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the command 'docker login' is run when using oreg with authentication. On some hosts, such as hosts configured to use system containers, the docker service is not running. 'docker login' will fail without the docker service running. This commit adds a module to idempotently add credentials to the registry credentials file without the use of 'docker login' Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1511374
* | Nuke /var/lib/dockershim/sandbox/* while nodes are drainedScott Dodson2017-11-141-0/+6
|/
* Add retry logic to docker auth credentialsMichael Gugino2017-10-301-0/+3
| | | | | | This commit enables retry on docker login commands. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1506931
* Fix broken debug_levelMichael Gugino2017-10-114-3/+4
| | | | | | | | | | | | | | | | | | | | Currently, debug_level is documented as a way to change the debug output level for both masters and nodes. debug_level does not currently have any effect. This commit removes debug_level from openshift_facts and properly sets openshift_master_debug_level and openshift_node_debug_level to the value of debug_level specified in the inventory. This commit also reorganizes some set_fact tasks needed during master upgrades to put all work-around set-facts for undefined variables in one place, allowing for easier cleanup in the future. This includes an entry for openshift_master_debug_level. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1500164
* node: make node service PartOf=openvswitch.service when openshift-sdn is usedDan Williams2017-10-051-0/+1
| | | | | | | | | | | | | Commit 7f805f9a0c41477365dd88b0ac73f0d221bd654a causes the behavior seen in https://bugzilla.redhat.com/show_bug.cgi?id=1453113 because openshift-node is no longer restarted when openvswitch is, due to the change from Requires to Wants. Turns out that making the openshift node service PartOf the OVS service can achieve the same result and ensure openshift-node gets restarted whenever OVS does, which ensures that networking doesn't break underneath the node. Suggested by Giuseppe Scrivano
* Fix registry auth variableMichael Gugino2017-09-291-1/+1
| | | | | | | | There is currently a bug in registry auth credential creation logic for openshift_node and openshift_node_upgrade This commit fixes the logic.
* Remove some reminants of Atomic EnterpriseScott Dodson2017-09-261-1/+1
|
* Merge pull request #5205 from mgugino-upstream-stage/docker-etc-containers-regsOpenShift Merge Robot2017-09-251-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue Move additional/block/insecure registires to /etc/containers/... Move additional/block/insecure registires to /etc/containers/registries.conf This commit moves additional/block/insecure registries to /etc/containers/registries.conf and comments existing lines in /etc/sysconfig/docker. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1460930
| * Move additional/block/insecure registires to /etc/containers/registries.confMichael Gugino2017-09-211-1/+1
| | | | | | | | | | | | | | | | This commit moves additional/block/insecure registries to /etc/containers/registries.conf and comments existing lines in /etc/sysconfig/docker. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1460930
* | Fix registry_auth logic for upgradesMichael Gugino2017-09-224-1/+48
|/ | | | | | | | | | | | | | | Currently, the logic for registry authentication is not implemented correctly to account for upgrades of containerized hosts. Additionally, the logic to account for multiple runs of openshift-ansible might cause registry authentication credentials to not be mounted inside of containerized hosts. This commit adds the necessary logic to ensure containerized hosts retain registry credentials. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1494470
* Remove openshift-commonMichael Gugino2017-09-067-12/+14
| | | | | | | | | Most of this role's purpose was to set facts. The vast majority of these facts were simply redefining user-supplied variables. This commit also removes various artifacts leftover from previous versions, as well as variables that seem to be entirely unused.
* Remove remaining references to openshift-master.serviceClayton Coleman2017-08-082-3/+0
| | | | Prevents playbooks from accidentally restarting the master service.
* move common tasks to a single file included by both systemd_units.ymlJan Chaloupka2017-07-288-73/+85
|
* backport 'Add systemctl daemon-reload handler to openshift_node' #4403 to ↵Jan Chaloupka2017-07-272-10/+14
| | | | openshift_node_upgrade
* Sync all openshift.common.use_openshift_sdn uses in yaml filesJan Chaloupka2017-07-262-2/+7
| | | | | | | | | | | Most occurrences are in a form: ```yaml openshift.common.use_openshift_sdn | default(true) | bool ``` Let's make all occurences this way given the use_openshift_sdn is set to true anyway. See https://github.com/openshift/openshift-ansible/blob/0c350dcc7d06d62be5ba3a8e468dff85cdd96dd7/roles/openshift_facts/library/openshift_facts.py#L2035
* Set TimeoutStartSec=300Scott Dodson2017-07-251-0/+1
| | | | | | On nodes with thousands of services it may take a very long time to establish all of the network routing rules. The longest we've seen is about 180s
* Revert "set KillMode to process in node service file"Scott Dodson2017-07-251-1/+0
|
* Add drain retries after 60 second delayScott Dodson2017-07-181-0/+5
|
* Wrap additional service changes in retriesScott Dodson2017-07-181-1/+7
|
* Wrap docker stop in retriesScott Dodson2017-07-181-1/+7
|
* Add retries to node restart handlersScott Dodson2017-07-181-2/+10
|
* Test docker restart with retries 3 delay 30Steve Milner2017-07-181-1/+1
|
* set KillMode to process in node service fileJan Chaloupka2017-07-131-0/+1
|
* Merge pull request #4624 from ↵Jan Chaloupka2017-07-081-0/+1
|\ | | | | | | | | ingvagabund/increase-timeout-when-waiting-for-master-and-nodes increase implicit 300s default timeout to explicit 600s
| * increase implicit 300s default timeout to explicit 600sJan Chaloupka2017-06-281-0/+1
| |
* | pre-pull images before stopping dockerJan Chaloupka2017-07-032-15/+14
| |
* | Fix upgradeScott Dodson2017-06-301-0/+11
| |
* | Use default ports for dnsmasq and node dnsScott Dodson2017-06-303-3/+2
| |
* | Run dns on the node and use that for dnsmasqScott Dodson2017-06-307-4/+43
|/
* Merge pull request #4441 from sdodson/retry-docker-startupScott Dodson2017-06-271-1/+5
|\ | | | | Retry docker startup once
| * Retry docker startup onceScott Dodson2017-06-131-1/+5
| | | | | | | | | | Large environments frequently see docker fail to start up after an upgrade. Not sure why but we can retry once.
* | Add openshift_node_dnsmasq role to upgradeScott Dodson2017-06-181-0/+3
|/
* Add node unit file on upgradeSteve Milner2017-06-063-0/+17
| | | | | | | | When rpm_upgrade is used for upgrading the node's unit file will be overridden. systemd is then reloaded if the files are templated successfully. Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1455843
* upgrade: Reload systemd before restartSteve Milner2017-05-311-0/+8
| | | | | | | | | The installer now lays down updated systemd unit files in /etc/systemd/system/ which override the units that come with the rpms. Because of this we need to reload systemd before restarting services. Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1455843
* allow to configure oreg_url specifically for node or master. refs #4233Tobias Brunner2017-05-191-2/+2
| | | | | | | | | This commit allows to specify imageConfig.format specifically for master or for nodes. One use case of this could be if you want to use customer builder images. In this case imageConfig.format only needs to be changed in the master-config.yml but not in the node-config.yml.
* Merge pull request #4094 from ashcrow/status-doesnt-existJan Chaloupka2017-05-191-1/+1
|\ | | | | Ensure good return code for specific until loops
| * Ensure good return code for specific until loopsSteve Milner2017-05-041-1/+1
| |
* | Merge pull request #4165 from sdodson/BZ1450167OpenShift Bot2017-05-181-0/+6
|\ \ | | | | | | Merged by openshift-bot
| * | Reset selinux context on /var/lib/origin/openshift.common.volumesScott Dodson2017-05-111-0/+6
| | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1450167
* | | node, systemd: change Requires to Wants for openvswitchGiuseppe Scrivano2017-05-171-1/+1
|/ / | | | | | | | | | | | | | | | | Sometimes the node container is not started on a container-engine restart. Use a weaker dependency on openvswitch that is causing this issue Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1451192 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Update systemd units to use proper container service nameSteve Milner2017-05-093-10/+10
|/ | | | | | | - If using a system container: container-engine - If using a package install: docker Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1448800
* System container dockerSteve Milner2017-05-031-1/+3
| | | | | | | | | | | | | | | This change allows for the use of either the traditional package install of docker OR a system container install of docker. Two new inventory options, openshift_docker_use_system_container and openshift_docker_systemcontainer_image_registry_override, have been added which are be used to install with a system container. By default this option is commented out. One new fact has been added: - docker.service_name: docker by default, container-engine-docker for system container
* Add bool for proper conditional handlingRussell Teague2017-04-261-1/+1
|
* Update swap disable tasksRussell Teague2017-04-211-7/+12
| | | | | | * Lines are commented out vice being removed * Comment is added indicating why the change happened * Variable openshift_disable_swap added to allow user control
* Merge pull request #3943 from sdodson/fix-node-upgrade-masterJan Chaloupka2017-04-213-34/+26
|\ | | | | Stop all services prior to upgrading, start all services after
| * Stop all services prior to upgrading, start all services afterScott Dodson2017-04-183-34/+26
| |