summaryrefslogtreecommitdiffstats
path: root/roles/openshift_facts
Commit message (Collapse)AuthorAgeFilesLines
...
* Fix broken debug_levelMichael Gugino2017-10-111-1/+0
| | | | | | | | | | | | | | | | | | | | 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
* Merge pull request #5577 from ashcrow/1496725OpenShift Merge Robot2017-09-291-1/+3
|\ | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. openshift_facts: coerce docker_use_system_container to bool Use ``safe_get_bool()`` when reading ``docker_use_system_container`` to ensure that ansible accepted inputs are turned into real boolean results. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1496725 3.6 Backport: https://github.com/openshift/openshift-ansible/pull/5578
| * openshift_facts: coerce docker_use_system_container to boolSteve Milner2017-09-281-1/+3
| | | | | | | | | | | | | | | | | | Use safe_get_bool() when reading docker_use_system_container to ensure that ansible accepted inputs are turned into real boolean results. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1496725 Signed-off-by: Steve Milner <smilner@redhat.com>
* | Migrate enterprise registry logic to docker roleMichael Gugino2017-09-271-24/+0
|/ | | | | | | | | | | | | Currently, the enterprise registry to forcefully added in openshift_facts. Recently, the docker role has been modified to consume registry variables directly, bypassing openshift_facts. This commit cleans up unused code in openshift_facts, and migrates enterprise registry logic to the docker role. Fixes: https://github.com/openshift/openshift-ansible/issues/5557
* Merge pull request #5483 from sdodson/bz1396350OpenShift Merge Robot2017-09-221-1/+1
|\ | | | | | | | | | | | | Automatic merge from submit-queue Ensure that hostname is lowercase Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1396350
| * Ensure that hostname is lowercaseScott Dodson2017-09-211-1/+1
| | | | | | | | Fixes Bug 1396350
* | Creating structure to warn for use of deprecated variables and set them in a ↵Eric Wolinetz2017-09-201-65/+71
|/ | | | single location before they are no longer honored
* Cleanup old deployment typesMichael Gugino2017-09-201-32/+7
| | | | | | | | | | | Previously, openshift-ansible supported various types of deployments using the variable "openshift_deployment_type" Currently, openshift-ansible only supports two deployment types, "origin" and "openshift-enterprise". This commit removes all logic and references to deprecated deployment types.
* Scaffold out the entire build defaults hashTim Bielawa2017-09-131-2/+4
| | | | | Some functions called later may expect sub-keys to exist which will not with the current default empty-dict.
* Remove openshift-commonMichael Gugino2017-09-061-117/+3
| | | | | | | | | 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 meta depends from clockMichael Gugino2017-08-291-8/+0
| | | | This role removes openshift_fact meta depends from clock role.
* Add hostname/nodename length checkMichael Gugino2017-08-161-1/+3
| | | | | | | | | OpenShift will not allow nodes to register if the nodename is > 63 chars. This commit verifies that nodename is not set to a potential hostname that is greater than 63 characters. This commit also updates logic to check `openshift_hostname` to check for > 63 chars. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1467790
* Removing tasks from module openshift_facts.Kenny Woodson2017-08-046-139/+1
|
* Merge pull request #4898 from giuseppe/cri-oOpenShift Bot2017-08-041-0/+2
|\ | | | | Merged by openshift-bot
| * cri-o: skip Docker version testGiuseppe Scrivano2017-08-031-0/+2
| | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Default gte_3_7 to falseScott Dodson2017-08-031-1/+1
| |
* | Update version checks to tolerate 3.7Steve Kuznetsov2017-08-031-1/+7
|/ | | | Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
* Refactor openshift_facts BIOS vendor discoveryRussell Teague2017-07-261-5/+1
| | | | | | | | | | | | | In Ansible 2.4, `get_file_content` is moving from ansible.module_utils.facts to ansible.module_utils.facts.utils However, the method is no longer needed as the BIOS vendor fact can be retrieved from a standard Ansible fact, `ansible_system_vendor`. i.e.: "ansible_system_vendor": "Google",
* If proxy in effect, add etcd host IP addresses to NO_PROXY list on mastersTim Bielawa2017-07-211-0/+7
| | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1466783
* Hopefully finally fix the no_proxy settingsTim Bielawa2017-07-191-69/+14
| | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1467776 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1466783 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1472244 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1468424 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1470165 Some of there were 'fixed' before but that commit was merged too soon. This actually fixes the needful.
* Other playbooks maybe expecting this to be at least an empty string. I think ↵Tim Bielawa2017-07-171-0/+1
| | | | they default it to an empty list if its not found.
* Fix NO_PROXY environment variable settingTim Bielawa2017-07-171-32/+68
|
* Only add entries to NO_PROXY settings if a NO_PROXY value is setTim Bielawa2017-07-071-8/+9
| | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1468424 Incorrect indentation caused base svc/cluster.local/hostname entries to be added to the NO_PROXY settings regardless of if the user was configuring NO_PROXY or not.
* Fix missing service domain .svc in NO_PROXY settingsTim Bielawa2017-07-051-1/+3
| | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1467776 Adds the tld '.svc' to the docker NO_PROXY list
* Always convert no_proxy from string into a listScott Dodson2017-07-031-18/+27
| | | | | | | Always add dns_domain, hostname to no_proxy list Fixes BZ1414748 Fixes BZ1414749
* Correct version comparisons to ensure proper evaluationRussell Teague2017-06-291-6/+6
| | | | | | | When a user supplies the openshift_image_tag inventory fact as '3.5', the evaluation of `version >= LooseVersion('3.5.0')` will return False. Changing the evaluation against only the X.X release will properly return True.
* Merge pull request #4195 from ewolinetz/service_catalogScott Dodson2017-06-261-0/+24
|\ | | | | Creation of service_catalog and broker roles
| * Picking change from sdodsonewolinetz2017-06-231-0/+1
| |
| * Adding volume fact for etcd for openshift ansible service brokerewolinetz2017-06-231-1/+24
| |
* | system_containers: Add openshift_ to other system_container varsSteve Milner2017-06-201-4/+4
| |
* | system_containers: Add openshift_ to use_system_containers varSteve Milner2017-06-202-5/+5
|/
* Push to the registry via dnsScott Dodson2017-06-181-0/+1
| | | | | | Configures OPENSHIFT_DEFAULT_REGISTRY=docker-registry.default.svc Adds 'cluster.local' to dns search on nodes via dispatcher script Adds '.svc' to NO_PROXY defaults
* Merge pull request #4179 from sosiouxme/20170512-singletonize-slow-stuffOpenShift Bot2017-06-141-4/+19
|\ | | | | Merged by openshift-bot
| * singletonize some role tasks that repeat a lotLuke Meyer2017-06-061-4/+19
| |
* | Do not attempt to override openstack nodenameJason DeTiberus2017-06-011-5/+7
|/ | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1392519
* move etcd upgrade related code into etcd_upgrade roleJan Chaloupka2017-05-181-0/+4
|
* Add docker package for container-engine installSteve Milner2017-05-141-1/+1
| | | | | The package is required so that openshift_facts is able to use the docker client to gain information on the versions.
* Merge pull request #3969 from jarrpa/glusterfs-registry-tooOpenShift Bot2017-05-101-1/+3
|\ | | | | Merged by openshift-bot
| * GlusterFS: Allow swapping an existing registry's backend storageJose A. Rivera2017-05-041-1/+3
| | | | | | | | Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* | Remove vim configuration from Python filesRodolfo Carvalho2017-05-091-1/+0
| | | | | | | | | | | | | | In a project where contributors are free to use whatever editor they want and we have linting tools that verify the proper formatting of Python files, it should not be required to have a vim-specific line in Python files.
* | Fix 1448368, and some other minors issuesGan Huang2017-05-051-5/+4
|/
* Merge pull request #4049 from ashcrow/system-container-dockerOpenShift Bot2017-05-031-1/+9
|\ | | | | Merged by openshift-bot
| * System container dockerSteve Milner2017-05-031-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Refactor system fact gathering to avoid dictionary size change during iteration.Andrew Butcher2017-05-021-3/+5
|/
* Standardize all Origin versioning on 3.6Russell Teague2017-04-251-6/+7
|
* Allow for GlusterFS to provide registry storageJose A. Rivera2017-04-101-0/+4
| | | | Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* openshift_facts: Allow examples_content_version to be set to v1.6Jose A. Rivera2017-04-031-1/+3
| | | | Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* Merge pull request #3554 from php-coder/install_python3_pkgOpenShift Bot2017-03-314-6/+18
|\ | | | | Merged by openshift-bot
| * openshift_facts: install python3-dbus package on Fedora nodes.Slava Semushin2017-03-244-6/+18
| |
| * Do not require python-six via openshift_factsRodolfo Carvalho2017-03-201-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | It turns out the code in roles/openshift_health_checker/openshift_checks/__init__.py had a mistake in the import statement, and not the case that `add_metaclass` is not available in the embedded `six` from `ansible.module_utils`. The fix is to correct the import and do not require python-six to be installed on target hosts. Note: now that we have some unit tests in the role, errors like that would have been caught.
generated by cgit v1.2.1 (git 2.18.0) at 2024-05-03 16:29:14 +0000