summaryrefslogtreecommitdiffstats
path: root/roles/openshift_facts
Commit message (Collapse)AuthorAgeFilesLines
* Add a fact to select --evacuate or --drain based on your OCP versionTim Bielawa2017-01-111-0/+15
| | | | Closes #3070
* Merge pull request #3007 from detiber/toxificationScott Dodson2017-01-101-14/+7
|\ | | | | More Toxification
| * More toxificationJason DeTiberus2017-01-101-14/+7
| | | | | | | | | | | | | | | | | | | | | | | | - 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
* | 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 #3015 from smarterclayton/future_versions_masterScott Dodson2017-01-081-0/+8
|\ | | | | Add future versions to openshift_facts
| * Add future versions to openshift_factsClayton Coleman2017-01-041-0/+8
| |
* | Add required python-six package to installationTim Bielawa2017-01-042-4/+9
|/ | | | | * Moves package list into a defaults/main.yml file * Closes #3020
* python3 support, add tox for better local testing against multiple python ↵Jason DeTiberus2016-12-201-13/+14
| | | | versions
* Fix access_modes initializationLuis Fernandez Alvarez2016-12-131-3/+9
|
* Switch back to using etcd rather than etcd3Scott Dodson2016-12-061-3/+3
|
* Conditionalize master config update for admission_plugin_config.Andrew Butcher2016-12-051-2/+2
|
* Cleanup ovs file and restart docker on every upgrade.Devan Goodwin2016-11-301-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | In 3.3 one of our services lays down a systemd drop-in for configuring Docker networking to use lbr0. In 3.4, this has been changed but the file must be cleaned up manually by us. However, after removing the file docker requires a restart. This had big implications particularly in containerized environments where upgrade is a very fragile series of upgrading and service restarts. To avoid double docker restarts, and thus double service restarts in containerized environments, this change does the following: - Skip restart during docker upgrade, if it is required. We will restart on our own later. - Skip containerized service restarts when we upgrade the services themselves. - Clean shutdown of all containerized services. - Restart Docker. (always, previously this only happened if it needed an upgrade) - Ensure all containerized services are restarted. - Restart rpm node services. (always) - Mark node schedulable again. At the end of this process, docker0 should be back on the system.
* refactor handling of scheduler defaultsJason DeTiberus2016-11-291-32/+16
|
* update tests and flake8/pylint fixesJason DeTiberus2016-11-291-73/+117
|
* Merge pull request #2842 from mscherer/check_dbus_moduleScott Dodson2016-11-281-2/+10
|\ | | | | Verify the presence of dbus python binding
| * Verify the presence of dbus python bindingMichael Scherer2016-11-231-2/+10
| | | | | | | | | | | | While the proper fix is to have it installed by default, this commit will also permit to have a better error message in the case the module is not present (as running on python 3)
* | Merge pull request #2836 from abutcher/BZ1393645Scott Dodson2016-11-281-13/+29
|\ \ | |/ |/| Merge admission plugin configs
| * Merge kube_admission_plugin_config with admission_plugin_configSamuel Munilla2016-11-221-13/+29
| | | | | | | | | | | | | | | | Move the values in kube_admission_plugin_config up one level per the new format from 1.3: "The kubernetesMasterConfig.admissionConfig.pluginConfig should be moved and merged into admissionConfig.pluginConfig."
* | Refactor to use Ansible package moduleRussell Teague2016-11-171-6/+5
|/ | | | | The Ansible package module will call the correct package manager for the underlying OS.
* Merge pull request #2763 from yfauser/byo_cni_plugin_fixesAndrew Butcher2016-11-161-0/+2
|\ | | | | [#2698] Change to allow cni deployments without openshift SDN
| * Change to allow cni deployments without openshift SDNyfauser2016-11-081-0/+2
| | | | | | | | | | | | | | The roles/openshift_facts main task did not pass the cni plugin variable to the later role playbooks. The master.yaml and node.yaml templates did not allow for a cni configuration without either installing openshift sdn or nuage. This change will allow to use os_sdn_network_plugin_name=cni and set openshift_use_openshift_sdn=false for deployments that use a cni plugin that doesn't need and want openshift sdn to be installed
* | Add updates for containerizedScott Dodson2016-11-141-3/+3
| |
* | Merge pull request #2600 from mscherer/python3_portingScott Dodson2016-11-101-8/+17
|\ \ | | | | | | Port openshift_facts to py3
| * | Port openshift_facts to py3Michael Scherer2016-10-141-8/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - fix configparser import - convert bytes to str using to_native - replace method that did got removed in py3 with six equivalent With ansible 2.2, python 3 start to be supported enough to run non trivial playbooks, and so a test run of openshift-ansible on F24 showed that openshift_facts.py needed to be ported.
* | | Merge pull request #2777 from dgoodwin/jenkins-role-bindingsScott Dodson2016-11-101-2/+8
|\ \ \ | | | | | | | | Reconcile role bindings for jenkins pipeline during upgrade.
| * | | Reconcile role bindings for jenkins pipeline during upgrade.Devan Goodwin2016-11-101-2/+8
| | | | | | | | | | | | | | | | https://github.com/openshift/origin/issues/11170 for more info.
* | | | Fix typos in openshift_facts gce cloud providerScott Dodson2016-11-091-2/+2
|/ / / | | | | | | | | | | | | Fixes Bug 1390160 Fixes BZ1390160
* | / Revert "Revert openshift.node.nodename changes"Scott Dodson2016-11-081-7/+20
| |/ |/|
* | Merge pull request #2753 from jkhelil/proxy_settingsScott Dodson2016-11-081-2/+2
|\ \ | | | | | | Always add local dns domain to no_proxy
| * | Always add local dns domain to no_proxyjawed2016-11-081-2/+2
| | |
* | | Merge pull request #2751 from sdodson/BZ1388253Scott Dodson2016-11-081-20/+7
|\ \ \ | | | | | | | | Revert openshift.node.nodename changes
| * | | Revert "Fix the nodeName of the OpenShift nodes on OpenStack"Scott Dodson2016-11-071-1/+1
| | | | | | | | | | | | | | | | This reverts commit aaaf82ba6032d0b1e9c36a39a7eda25b8c5f4b84.
| * | | Revert "Fix OpenStack cloud provider"Scott Dodson2016-11-071-10/+0
| | | | | | | | | | | | | | | | This reverts commit 1f2276fff1e41c1d9440ee8b589042ee249b95d7.
| * | | Revert "Check that OpenStack hostnames are resolvable"Scott Dodson2016-11-071-9/+6
| | | | | | | | | | | | | | | | This reverts commit 3cc2d6b019fa8a05ba480a7425cd654aa051355f.
* | | | Refactor default sdn_cluster_network_cidr and sdn_host_subnet_lengthScott Dodson2016-11-071-4/+9
| |/ / |/| | | | | | | | | | | Inventory values highest priority, then anything in master-config.yaml, then defaults last.
* | | Merge pull request #2652 from smunilla/BZ1320952_default_for_clusterNetworkCIDRScott Dodson2016-11-071-5/+19
|\ \ \ | |/ / |/| | Update defaults for clusterNetworkCIDR & hostSubnetLength
| * | Update defaults for clusterNetworkCIDR & hostSubnetLengthSamuel Munilla2016-11-031-5/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per https://github.com/openshift/openshift-docs/issues/1700: The default values for pod networking have changed: - clusterNetworkCIDR now defaults to 10.128.0.0/14 (10.128.0.0 - 10.131.255.255) rather than 10.1.0.0/16. - hostSubnetLength now defaults to 9 rather than 8 (meaning each node will be assigned a /23 subnet rather than a /24) Fixes Bug 1320952
* | | default to multizone GCE configSeth Jennings2016-11-041-0/+3
|/ /
* | Remove commit offset strings from parsed versionsTim Bielawa2016-10-311-2/+22
| |
* | Merge pull request #2670 from abutcher/hosted-registry-insecureScott Dodson2016-10-261-2/+21
|\ \ | | | | | | Bug 1388016 - The insecure-registry address was removed during upgrade
| * | Default hosted_registry_insecure true when insecure registry present in ↵Andrew Butcher2016-10-261-2/+21
| | | | | | | | | | | | existing /etc/sysconfig/docker.
* | | Fix race condtion in openshift_factsSamuel Munilla2016-10-251-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | If, for some reason, two facts processes were run simultaneously on the same host, creating the directory could cause an exception. This should help with that. Fixes Bug 1385449
* | | Merge pull request #2615 from tbielawa/node-labels-fixTim Bielawa2016-10-251-3/+36
|\ \ \ | |/ / |/| | Set node-labels in kubeletArguments
| * | Add node-labels to kubeletArgumentsTim Bielawa2016-10-201-3/+36
| | | | | | | | | | | | * Node labels are parsed from openshift_node_labels if they exist
* | | Fix typosRodolfo Carvalho2016-10-191-1/+1
|/ /
* / update handling of use_dnsmasqJason DeTiberus2016-10-131-2/+2
|/
* Merge pull request #2561 from lhuard1A/check_openstack_hostname_are_resovableScott Dodson2016-10-101-6/+9
|\ | | | | Check that OpenStack hostnames are resolvable
| * Check that OpenStack hostnames are resolvableLénaïc Huard2016-10-061-6/+9
| | | | | | | | If they’re not, then use IP instead
* | Merge pull request #2546 from abutcher/master-debug-levelTim Bielawa2016-10-051-0/+1
|\ \ | |/ |/| Bug 1366522 - The value of "debug_level" isn't applied to /etc/sysconfig/atomic-openshift-master
| * Move openshift.common.debug.level to openshift_facts.Andrew Butcher2016-10-051-0/+1
| |