summaryrefslogtreecommitdiffstats
path: root/roles
Commit message (Collapse)AuthorAgeFilesLines
* Use existing OPENSHIFT_DEFAULT_REGISTRY setting during masters scaleupTim Bielawa2017-07-285-6/+16
| | | | | | | | | | | | | | | Fixes a bug which reported that AFTER a cluster upgrade from OCP 3.5 to 3.6, any masters which are later added via the scaleup playbooks are setting a value for OPENSHIFT_DEFAULT_REGISTRY which is inconsistent with the already configured masters. * OPENSHIFT_DEFAULT_REGISTRY value is saved from oo_first_master * The new scaleup master has a fact set noting it is a scaleup host * The saved OPENSHIFT_DEFAULT_REGISTRY value is used instead of the default for 3.6, which is to use a hard-coded registry value of 'docker-registry.default.svc:5000' Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1469336
* Merge pull request #4899 from rhcarvalho/misc-cleanup-normalize-inputOpenShift Bot2017-07-282-6/+19
|\ | | | | Merged by openshift-bot
| * Normalize list of checks passed to action pluginRodolfo Carvalho2017-07-271-5/+15
| | | | | | | | | | | | | | | | | | This allows users to pass either a string with comma-separated check names or a proper list of strings, whatever is more convenient in the context. For consistency, the same normalization is applied to the 'openshift_disable_check' variable.
| * Make LoggingCheck.run return the correct typeRodolfo Carvalho2017-07-271-1/+4
| | | | | | | | | | | | The run method is expected to return a dict. Even though we should not run LoggingCheck by itself, it is still possible to do it and without this change we get an unhandled exception.
* | Revert "Add health checks to upgrade playbook"Rodolfo Carvalho2017-07-282-98/+5
| |
* | Merge pull request #4372 from ↵Rodolfo Carvalho2017-07-282-5/+98
|\ \ | | | | | | | | | | | | juanvallejo/jvallejo/verify-disk-memory-before-upgrade Add health checks to upgrade playbook
| * | fixes after rebasing with #4485juanvallejo2017-07-262-3/+2
| | |
| * | add pre-flight checks to ugrade pathjuanvallejo2017-07-262-5/+99
| | |
* | | Merge pull request #4897 from ↵Scott Dodson2017-07-272-10/+14
|\ \ \ | | | | | | | | | | | | | | | | ingvagabund/backport-4403-to-openshift-node-upgrade-role backport 'Add systemctl daemon-reload handler to openshift_node' #4403 to openshift_node_upgrade
| * | | backport 'Add systemctl daemon-reload handler to openshift_node' #4403 to ↵Jan Chaloupka2017-07-272-10/+14
| | | | | | | | | | | | | | | | openshift_node_upgrade
* | | | Merge pull request #4843 from kwoodson/sc_require_provScott Dodson2017-07-273-12/+6
|\ \ \ \ | | | | | | | | | | Remove default and force argument for storageclass provisioner.
| * | | | Remove default provisioner.Kenny Woodson2017-07-243-12/+6
| | | | |
* | | | | Merge pull request #4479 from mtnbikenc/contiv-warningsScott Dodson2017-07-274-7/+7
|\ \ \ \ \ | | | | | | | | | | | | Resolve Ansible deprecation warnings in Contiv roles
| * | | | | Resolve deprecation warnings in Contiv rolesRussell Teague2017-06-164-7/+7
| | | | | |
* | | | | | Merge pull request #4906 from ewolinetz/logging_unique_template_varsScott Dodson2017-07-278-9/+9
|\ \ \ \ \ \ | | | | | | | | | | | | | | Updating template parameter replica to be more unique to avoid var sc…
| * | | | | | Updating template parameter replica to be more unique to avoid var scope ↵ewolinetz2017-07-278-9/+9
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | creeping
* / | | | | Block etcdv3 migration for supported configurationsScott Dodson2017-07-271-0/+4
|/ / / / /
* | | | | Merge pull request #4882 from sdodson/fix-node-resolv-confScott Dodson2017-07-271-11/+18
|\ \ \ \ \ | |_|_|/ / |/| | | | Simplify generation of /etc/origin/node/resolv.conf
| * | | | Simplify generation of /etc/origin/node/resolv.confScott Dodson2017-07-261-11/+18
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Don't attempt to copy /etc/resolv.conf whenever there's no watermark, just write out the list of nameservers that network manager tells us about like we do for origin-upstream-dns.conf Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1474707
* | | | Merge pull request #4874 from ingvagabund/set-default-valuesJan Chaloupka2017-07-276-7/+16
|\ \ \ \ | | | | | | | | | | Sync all openshift.common.use_openshift_sdn uses in yaml files
| * | | | Sync all openshift.common.use_openshift_sdn uses in yaml filesJan Chaloupka2017-07-266-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most occurrences are in a form: ```yaml openshift.common.use_openshift_sdn | default(true) | bool ``` Let's make all occurences this way given the use_openshift_sdn is set to true anyway. See https://github.com/openshift/openshift-ansible/blob/0c350dcc7d06d62be5ba3a8e468dff85cdd96dd7/roles/openshift_facts/library/openshift_facts.py#L2035
* | | | | Merge pull request #4868 from ewolinetz/metrics_fix_caCertScott Dodson2017-07-261-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | Fixing route spec caCertificate to be correctly capitalized
| * | | | Fixing route spec caCertificate to be correctly capitalizedewolinetz2017-07-251-1/+1
| | | | |
* | | | | Merge pull request #4844 from ewolinetz/bz1463081Scott Dodson2017-07-262-6/+14
|\ \ \ \ \ | | | | | | | | | | | | Updating how storage type is determined, adding bool filter in opensh…
| * | | | | Updating how storage type is determined, adding bool filter in ↵ewolinetz2017-07-242-6/+14
| | | | | | | | | | | | | | | | | | | | | | | | openshift_logging_elasticsearch
* | | | | | Merge pull request #4816 from abutcher/containerized-lb-image-tagScott Dodson2017-07-261-0/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Pass first master's openshift_image_tag to openshift_loadbalancer for containerized haproxy installation.
| * | | | | | Pass first master's openshift_image_tag to openshift_loadbalancer for ↵Andrew Butcher2017-07-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | containerized haproxy installation.
* | | | | | | Merge pull request #4485 from sosiouxme/20170616-store-args-in-check-propertiesScott Dodson2017-07-2643-544/+494
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | openshift_checks: store args in check properties
| * | | | | | | openshift_checks: refactor to internalize task_varsLuke Meyer2017-07-2539-480/+439
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move task_vars into instance variable so we don't have to pass it around everywhere. Also store tmp. Make sure both are filled in on execute_module. In the process, is_active became an instance method, and task_vars is basically never used directly outside of test code.
| * | | | | | | openshift_checks: get rid of deprecated module_executorLuke Meyer2017-07-254-25/+10
| | | | | | | |
| * | | | | | | openshift_checks: improve comments/namesLuke Meyer2017-07-2513-51/+57
| | | | | | | |
* | | | | | | | Merge pull request #4693 from enoodle/create_router_certificate_by_defaultScott Dodson2017-07-262-4/+4
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Router wildcard certificate created by default
| * | | | | | | | add default value for router path in the certErez Freiberger2017-07-251-2/+2
| | | | | | | | |
| * | | | | | | | Router wildcard certificate created by defaultErez Freiberger2017-07-252-2/+2
| | | | | | | | |
* | | | | | | | | Fixing podpresets perms for service-catalog-controllerewolinetz2017-07-261-1/+5
| | | | | | | | |
* | | | | | | | | Merge pull request #4813 from sdodson/log-dumperScott Dodson2017-07-262-1/+21
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix log dumping on service failure
| * | | | | | | | | Fix log dumping on service failureScott Dodson2017-07-202-1/+21
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | Also, add log dumping to master service startup too
* | | | | | | | | Merge pull request #4855 from mwringe/hawkular-namespace-listener-updateScott Dodson2017-07-263-1/+41
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | Metrics: grant hawkular namespace listener role
| * | | | | | | | Metrics: grant hawkular namespace listener rolemwringe2017-07-243-1/+41
| | | | | | | | |
* | | | | | | | | Merge pull request #4687 from mrooding/fix-etcd-conditional-check-failureJan Chaloupka2017-07-261-1/+2
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | Fix etcd conditional check failure (issue #4121)
| * | | | | | | | Fix etcd conditional check failureMarc Rooding2017-07-071-1/+2
| | | | | | | | |
* | | | | | | | | Set TimeoutStartSec=300Scott Dodson2017-07-252-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On nodes with thousands of services it may take a very long time to establish all of the network routing rules. The longest we've seen is about 180s
* | | | | | | | | Revert "set KillMode to process in node service file"Scott Dodson2017-07-252-2/+0
| |_|_|_|/ / / / |/| | | | | | |
* | | | | | | | Merge pull request #4854 from ewolinetz/metrics_remove_no_logScott Dodson2017-07-251-1/+0
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Removing no_log from htpasswd invocation so not to supress errors
| * | | | | | | | Removing nolog from htpasswd invocation so not to supress errorsewolinetz2017-07-241-1/+0
| | |_|_|_|_|_|/ | |/| | | | | |
* | | | | | | | Merge pull request #4839 from tripledes/unexclude_use_abs_pathScott Dodson2017-07-251-2/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Use absolute path when un-excluding
| * | | | | | | | Use absolute path when unexcludingSergi Jimenez2017-07-241-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * As follow-up on #4835, use absolute path also when unexcluding
* | | | | | | | | Merge pull request #4806 from tbielawa/bz1466783_etcd_ips_in_masters_no_proxyScott Dodson2017-07-242-0/+13
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | | Bz1466783 etcd ips in masters no proxy
| * | | | | | | | If proxy in effect, add etcd host IP addresses to NO_PROXY list on mastersTim Bielawa2017-07-212-0/+13
| | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1466783
* | | | | | | | Removed kubernetes.io string from default.Kenny Woodson2017-07-241-2/+2
| | | | | | | |