summaryrefslogtreecommitdiffstats
path: root/roles
Commit message (Collapse)AuthorAgeFilesLines
* Reduce code duplication using variableRodolfo Carvalho2017-01-122-44/+14
| | | | | We can use openshift_facts's service_type to determine the package names.
* Replace custom variables with openshift_factsRodolfo Carvalho2017-01-125-9/+6
| | | | | | | Note: on a simple example run of ansible-playbook against a single docker-based host, I saw the execution time jump from 7s to 17s. That's unfortunate, but it is probably better to reuse openshift_facts, than to come up with new variables.
* Catch DBus exceptions on class initializationRodolfo Carvalho2017-01-121-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | I came across this: Traceback (most recent call last): File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 2409, in <module> main() File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 2396, in main protected_facts_to_overwrite) File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 1855, in __init__ protected_facts_to_overwrite) File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 1892, in generate_facts defaults = self.get_defaults(roles, deployment_type, deployment_subtype) File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 1989, in get_defaults version_info = get_docker_version_info() File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 1290, in get_docker_version_info if is_service_running('docker'): File "/tmp/ansible_BhXpTE/ansible_module_openshift_facts.py", line 1249, in is_service_running bus = SystemBus() File "/usr/lib64/python2.7/site-packages/dbus/_dbus.py", line 194, in __new__ private=private) File "/usr/lib64/python2.7/site-packages/dbus/_dbus.py", line 100, in __new__ bus = BusConnection.__new__(subclass, bus_type, mainloop=mainloop) File "/usr/lib64/python2.7/site-packages/dbus/bus.py", line 122, in __new__ bus = cls._new_for_bus(address_or_type, mainloop=mainloop) dbus.exceptions.DBusException: org.freedesktop.DBus.Error.FileNotFound: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
* Refactor preflight check into rolesRodolfo Carvalho2017-01-1210-0/+293
|
* Merge pull request #3030 from kwoodson/yaml_editorScott Dodson2017-01-1111-0/+2118
|\ | | | | Ansible module for modifying yaml idempotently.
| * Adding ability to systematically modify yaml from ansible.Kenny Woodson2017-01-1111-0/+2118
| |
* | Add a fact to select --evacuate or --drain based on your OCP versionTim Bielawa2017-01-112-0/+17
|/ | | | Closes #3070
* rename openshift_metrics to openshift_hosted_metricsJeff Cantrill2017-01-117-16/+16
|
* Merge pull request #3007 from detiber/toxificationScott Dodson2017-01-102-25/+21
|\ | | | | More Toxification
| * More toxificationJason DeTiberus2017-01-102-25/+21
| | | | | | | | | | | | | | | | | | | | | | | | - Move pylint tests to tox - Move yamllint tests to tox - Create separate tox config (and setup.py) for root - bump ansible requirement - unify pylint config - add docs - remove git directory containing old testing tools - install python-six if not present for openshift-facts - add python-six as a dependency for openshift-ansible-utils
* | Merge pull request #3065 from ashcrow/rpmrebuilddbAndrew Butcher2017-01-101-0/+12
|\ \ | | | | | | Workaround for dnf+docker version race condition
| * | Workaround for dnf+docker version race conditionSteve Milner2017-01-101-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some versions of dnf the rpm database can require maintenance after usage. Unfortunately, this trips up get_version_output in the openshift_facts library as the maintenance may interrupt the STDOUT stream. This makes the resulting data returned from get_version_output malformed and unparsable. This change adds an rpm --rebuilddb command before requesting docker version output when the ansible package manager is dnf. This causes the STDOUT maintenance messages to no longer interrupt the expected yaml stream as well as releases locks in the rpm database.
* | | Merge pull request #3060 from abutcher/return-deps-to-rolesScott Dodson2017-01-103-3/+73
|\ \ \ | |_|/ |/| | Return role dependencies to roles for openshift_{hosted,master,node}
| * | Move role dependencies out of playbooks for openshift_master, openshift_node ↵Andrew Butcher2017-01-093-3/+73
| | | | | | | | | | | | and openshift_hosted.
* | | Merge pull request #3055 from abutcher/openshift_image_tag=latestScott Dodson2017-01-093-6/+7
|\ \ \ | | | | | | | | Fix openshift_image_tag=latest.
| * | | Fix openshift_image_tag=latest.Andrew Butcher2017-01-093-6/+7
| | |/ | |/|
* | | Merge pull request #2909 from mtnbikenc/firewalldScott Dodson2017-01-095-18/+26
|\ \ \ | |/ / |/| | Enable firewalld by default
| * | Enable firewalld by defaultRussell Teague2016-12-145-18/+26
| | |
* | | Merge pull request #3015 from smarterclayton/future_versions_masterScott Dodson2017-01-084-5/+17
|\ \ \ | | | | | | | | Add future versions to openshift_facts
| * | | Add future versions to openshift_factsClayton Coleman2017-01-044-5/+17
| | | |
* | | | Merge pull request #3054 from sdodson/fix-repo-boolsScott Dodson2017-01-061-2/+2
|\ \ \ \ | |_|_|/ |/| | | Fix repo defaults
| * | | Fix repo defaultsScott Dodson2017-01-061-2/+2
| | | |
* | | | Merge pull request #3039 from tbielawa/gh3020Scott Dodson2017-01-062-4/+9
|\ \ \ \ | |/ / / |/| | | Add required python-six package to installation
| * | | Add required python-six package to installationTim Bielawa2017-01-042-4/+9
| | | | | | | | | | | | | | | | | | | | * Moves package list into a defaults/main.yml file * Closes #3020
* | | | Re-add when condition which was removed mistakenly in #3036Maciej Szulik2017-01-051-0/+1
| | | |
* | | | Merge pull request #3036 from soltysh/docker_registry_volumeScott Dodson2017-01-051-6/+5
|\ \ \ \ | | | | | | | | | | Fix jsonpath expected output when checking registry volume secrets
| * | | | Fix jsonpath expected output when checking registry volume secretsMaciej Szulik2017-01-051-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In k8s 1.5 (see rebase PR: origin 12143) jsonpath output is slightly different than previously. Causing this line to fail. We need to pick secret or secretName when checking this task failure condition.
* | | | | logging-deployer pull fixes from origin-aggregated-logging/#317Scott Dodson2017-01-052-5/+0
| | | | |
* | | | | Merge pull request #3033 from abutcher/openshift-enable-origin-repoScott Dodson2017-01-051-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Cast openshift_enable_origin_repo to bool.
| * | | | | Cast openshift_enable_origin_repo to bool.Andrew Butcher2017-01-041-2/+2
| | |_|/ / | |/| | |
* | | | | Merge pull request #3008 from bigbank-as/feature/nfs-reclaim-optionScott Dodson2017-01-053-1/+17
|\ \ \ \ \ | | | | | | | | | | | | Add osnl_volume_reclaim_policy variable to nfs_lvm role
| * | | | | Add osnl_volume_reclaim_policy variable to nfs_lvm roleAndo Roots2016-12-223-1/+17
| |/ / / / | | | | | | | | | | | | | | | | | | | | This allows to choose which volume reclamation policy to use when provisioning storage with `openshift_storage_nfs_lvm`.
* | | | | Merge pull request #3040 from sdodson/issue3025Scott Dodson2017-01-041-2/+2
|\ \ \ \ \ | |_|_|/ / |/| | | | Enable repos defined in openshift_additional_repos by default
| * | | | Enable repos defined in openshift_additional_repos by defaultScott Dodson2017-01-041-2/+2
| | | | | | | | | | | | | | | | | | | | Fixes #3025
* | | | | Update CFME template to point to GA buildSatoe Imaishi2017-01-041-5/+5
|/ / / /
* / / / Add templates for CFME Beta pod imagesSatoe Imaishi2016-12-223-0/+505
|/ / /
* / / python3 support, add tox for better local testing against multiple python ↵Jason DeTiberus2016-12-203-20/+27
|/ / | | | | | | versions
* | Merge pull request #2829 from tbielawa/cert_expiry_updatesTim Bielawa2016-12-193-9/+60
|\ \ | | | | | | Cert expiry updates
| * | Update the openshift-certificate-expiry README to reflect latestTim Bielawa2016-12-191-7/+13
| | | | | | | | | | | | changes
| * | Check embedded etcd certs now, tooTim Bielawa2016-12-151-4/+44
| | | | | | | | | | | | | | | * Addresses RFE in https://bugzilla.redhat.com/show_bug.cgi?id=1389264
| * | Include 'total' and 'ok' in check resultsTim Bielawa2016-12-152-1/+6
| |/ | | | | | | * Addresses https://bugzilla.redhat.com/show_bug.cgi?id=1389263
* / Deprecate node 'evacuation' with 'drain'Tim Bielawa2016-12-161-1/+1
|/ | | | * https://trello.com/c/TeaEB9fX/307-3-deprecate-node-evacuation
* Fix access_modes initializationLuis Fernandez Alvarez2016-12-131-3/+9
|
* Merge pull request #2964 from mtnbikenc/linting-refactorTim Bielawa2016-12-1240-345/+354
|\ | | | | YAML Linting with CI checking
| * YAML LintingRussell Teague2016-12-1240-345/+354
| | | | | | | | | | * Added checks to make ci for yaml linting * Modified y(a)ml files to pass lint checks
* | Limit node certificate SAN to node hostnames/ips.Andrew Butcher2016-12-091-7/+7
|/
* Merge pull request #2902 from dgoodwin/fix-metrics-urlScott Dodson2016-12-088-36/+36
|\ | | | | Fix metricsPublicURL only being set correctly on first master.
| * Fix metricsPublicURL only being set correctly on first master.Devan Goodwin2016-12-018-36/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | Problem was caused by facts not being set for that master. To fix this patch cleans up the calculation of metricsPublicURL in general. Because this value is used in openshift_master to template into the master config file, we now define these facts more clearly in openshift_master_facts, and add a dependency on this to openshift_metrics. The calculation of default sub-domain is also changed to remove it from system facts (as neither of these are facts about the system) and instead use plain variables.
* | Merge pull request #2937 from dgoodwin/upgrade-prepullScott Dodson2016-12-084-21/+22
|\ \ | | | | | | Pre-pull master/node/ovs images during upgrade.
| * | Pre-pull master/node/ovs images during upgrade.Devan Goodwin2016-12-074-21/+22
| | | | | | | | | | | | | | | | | | | | | | | | We did this for install but not upgrade, leading to situations where the service restarts after upgrade could take much longer than expected as docker pulls down the new image. Now the images are present when we restart services and should allow them to come back online much more quickly, equivalent to rpm service restarts.