summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #3086 from bparees/build_nodesScott Dodson2017-01-178-38/+105
|\ | | | | add configuration for build default+overrides settings
| * add configuration for build default+overrides settingsBen Parees2017-01-168-38/+105
| |
* | Merge pull request #3064 from giuseppe/fix-etcdctl-from-containerScott Dodson2017-01-171-1/+2
|\ \ | | | | | | use etcdctl from the container when containerized=True
| * | use etcdctl from the container when containerized=TrueGiuseppe Scrivano2017-01-101-1/+2
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | 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