summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #2993 from rhcarvalho/success-preflightRodolfo Carvalho2017-01-1716-3/+560
|\ | | | | Add RPM checks as a byo playbook
| * Rename subrole facts -> initRodolfo Carvalho2017-01-134-3/+2
| | | | | | | | | | | | | | | | Trying to improve the name, `init` needs to be loaded before calling other subroles. We don't make `init` a dependency of `common`, `masters` and `nodes` to avoid running the relatively slow `openshift_facts` multiple times.
| * Move Python modules into roleRodolfo Carvalho2017-01-138-3/+9
| | | | | | | | Allow reuse via role dependency.
| * Fix inconsistent task nameRodolfo Carvalho2017-01-121-1/+1
| |
| * 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-126-10/+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
| * Move playbook to BYORodolfo Carvalho2017-01-122-2/+2
| | | | | | | | Because that's the main playbook directory in use.
| * Refactor preflight check into rolesRodolfo Carvalho2017-01-1213-152/+212
| |
| * Make flake8 (py35) happy on bare exceptRodolfo Carvalho2017-01-121-3/+3
| |
| * Make callback plugin an always-on aggregate pluginRodolfo Carvalho2017-01-122-7/+6
| | | | | | | | | | This makes it so that one doesn't need to explicitly enable nor replace the default callback plugin.
| * Add RPM checks as an adhoc playbookRodolfo Carvalho2017-01-127-0/+525
| |
* | Merge pull request #3096 from abutcher/node_portsScott Dodson2017-01-163-4/+21
|\ \ | | | | | | Support openshift_node_port_range for configuring service NodePorts
| * | Support openshift_node_port_range for configuring service NodePortsClayton Coleman2017-01-103-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | Sets the appropriate config field if openshift_node_port_range is set and also configures filewalls on each node. firewalld already supports port ranges like "30000-32000", while iptables needs that value converted to the correct "30000:32000" form for use with `--dport`. If not set, no node ports are opened.
* | | Merge pull request #3085 from abutcher/no-nfs-group-defaultScott Dodson2017-01-161-5/+0
|\ \ \ | | | | | | | | Do not default registry storage kind to 'nfs' when 'nfs' host group exists.
| * | | Do not default registry storage kind to 'nfs' when 'nfs' group exists.Andrew Butcher2017-01-121-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes it possible to use openshift-ansible configured nfs for hosted components other than the registry while also configuring empty-dir registry storage (no openshift_hosted_registry_storage_kind set).
* | | | Merge pull request #3091 from ashcrow/modify-yamlAndrew Butcher2017-01-161-8/+19
|\ \ \ \ | | | | | | | | | | Updated modify_yaml with docstring and clarifications
| * | | | Updated modify_yaml with docstring and clarificationsSteve Milner2017-01-131-8/+19
| | | | |
* | | | | Merge pull request #3090 from jfchevrette/fix_pv_template_specAndrew Butcher2017-01-161-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Fix role openshift_storage_nfs_lvm keys should be lowercase in the pv spec template
| * | | | | keys should be lowercase according to the specJean-Francois Chevrette2017-01-121-2/+2
| | | | | |
* | | | | | Merge pull request #3088 from jfchevrette/fix_openshift_storage_nfs_lvmAndrew Butcher2017-01-162-2/+3
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Fix role openshift_storage_nfs_lvm
| * | | | | fix when statement indentation, cast to boolJean-Francois Chevrette2017-01-121-1/+1
| | | | | |
| * | | | | add openshift_facts as role dependencyJean-Francois Chevrette2017-01-121-1/+2
| |/ / / /
* | | | | Merge pull request #3089 from ashcrow/openshift-master-updatesAndrew Butcher2017-01-131-11/+8
|\ \ \ \ \ | |/ / / / |/| | | | openshift_master filter updates
| * | | | filter: Removed unused validation callsSteve Milner2017-01-121-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The parent class for filters in openshift_master defines a validate method which does nothing. This change removes calls to this method while leaving the validation implementations as is.
| * | | | Updated initializer usage in filtersSteve Milner2017-01-121-11/+11
|/ / / / | | | | | | | | | | | | | | | | | | | | Switched from explicit parent class usage to super(). See: https://docs.python.org/2/library/functions.html#super
* | | | Merge pull request #3087 from ashcrow/setuppyScott Dodson2017-01-122-2/+4
|\ \ \ \ | | | | | | | | | | Added setup.py to flake8 tests
| * | | | Added setup.py to flake8 testsSteve Milner2017-01-122-2/+4
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added setup.py to flake8 tests since it is code. Also minor updates to slightly modernize. Note: In the future {#} should probably be ported to {} as numbering is no longer needed (and disliked by many).
* | | | Merge pull request #3084 from tbielawa/issue_template_updateScott Dodson2017-01-121-12/+42
|\ \ \ \ | |/ / / |/| | | Another proposed update to the issue template
| * | | Another proposed update to the issue templateTim Bielawa2017-01-121-12/+42
|/ / /
* | | Merge pull request #3081 from lberk/masterScott Dodson2017-01-121-1/+1
|\ \ \ | |_|/ |/| | Fix typo in inventory README.md
| * | Fix typo in inventory README.mdLukas Berk2017-01-121-1/+1
|/ / | | | | | | libviert -> libvirt
* | Merge pull request #3076 from ashcrow/flatten-oo-filtersJason DeTiberus2017-01-111-896/+898
|\ \ | | | | | | WIP: oo_filters: Moved static methods to functions
| * | oo_filters: Moved static methods to functionsSteve Milner2017-01-111-896/+898
| | | | | | | | | | | | | | | This change moves the static methods into functions in an effort to reduce complexity and improve readability.
* | | Merge pull request #3079 from tbielawa/readme_release_docsJason DeTiberus2017-01-111-22/+40
|\ \ \ | | | | | | | | first swing at release version wording
| * | | first swing at release version wordingTim Bielawa2017-01-111-22/+40
|/ / / | | | | | | | | | clarify our branching and release compat. policy.
* | | 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
| |/ /
* | | Merge pull request #3074 from tbielawa/GH3070-drainuateScott Dodson2017-01-115-3/+20
|\ \ \ | | | | | | | | Add a fact to select --evacuate or --drain based on your OCP version
| * | | Add a fact to select --evacuate or --drain based on your OCP versionTim Bielawa2017-01-115-3/+20
| | | | | | | | | | | | | | | | Closes #3070
* | | | Merge pull request #3077 from mtnbikenc/fix-toxJason DeTiberus2017-01-111-2/+1
|\ \ \ \ | |_|/ / |/| | | Correct tox to run on Travis
| * | | Correct tox to run on TravisRussell Teague2017-01-111-2/+1
|/ / /
* | | Merge pull request #3075 from mtnbikenc/fix-yamllint-checkJason DeTiberus2017-01-111-5/+5
|\ \ \ | | | | | | | | [WIP] Correct return code compairison for yamllint
| * | | Correct return code compairison for yamllintRussell Teague2017-01-111-5/+5
| | | |
* | | | Merge pull request #3073 from sdodson/branchesScott Dodson2017-01-111-1/+1
|\ \ \ \ | |_|/ / |/| | | Update branch status
| * | | Update branch statusScott Dodson2017-01-111-1/+1
|/ / / | | | | | | | | | master is now applicable to v1.5 and v1.4
* | | Merge pull request #2881 from jcantrill/mv_metrics_to_hosted_metricsJason DeTiberus2017-01-118-17/+17
|\ \ \ | | | | | | | | rename openshift_metrics to openshift_hosted_metrics
| * | | rename openshift_metrics to openshift_hosted_metricsJeff Cantrill2017-01-118-17/+17
| |/ /
* | | Merge pull request #3067 from lhuard1A/update_ec2_inventoryJason DeTiberus2017-01-112-59/+280
|\ \ \ | | | | | | | | Update aws dynamic inventory
| * | | Update aws dynamic inventoryLénaïc Huard2017-01-102-59/+280
| | | | | | | | | | | | | | | | Fetch from https://raw.githubusercontent.com/ansible/ansible/devel/contrib/inventory/ec2.py