summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* Clean up openshift-checks playbooksRodolfo Carvalho2017-07-272-8/+2
| | | | | | | | | - The referenced issue was closed, seems like a wontfix. Just use the old syntax and make no noise about it. - Remove inconsistent use of tags: if tags are passed in the command line, there is no point in evaluating groups if the checks won't be run anyway. - Remove temporal comment, easy to get out of sync.
* Merge pull request #4884 from warmchang/version-3.6Scott Dodson2017-07-271-1/+2
|\ | | | | Follow the new naming conventions.
| * Follow the new naming conventions.William Chang2017-07-271-1/+2
| |
* | 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 #4881 from abutcher/glusterfs-cluster-hostsScott Dodson2017-07-261-0/+1
|\ \ \ | | | | | | | | Add glusterfs hosts to oo_all_hosts so that hosts set initial facts.
| * | | Add glusterfs hosts to oo_all_hosts so that hosts set initial facts.Andrew Butcher2017-07-261-0/+1
| | | |
* | | | Merge pull request #4448 from mtnbikenc/vars-syntaxScott Dodson2017-07-261-4/+1
|\ \ \ \ | | | | | | | | | | Remove invalid when: from vars:
| * | | | Remove invalid when: from vars:Russell Teague2017-06-141-4/+1
| | | | |
* | | | | Merge pull request #4738 from sjug/opt_ansible_cfgScott Dodson2017-07-261-1/+9
|\ \ \ \ \ | | | | | | | | | | | | Additional optimization parameters for ansible.cfg
| * | | | | Additional optimization parameters for ansible.cfgSebastian Jug2017-07-121-1/+9
| | | | | |
* | | | | | 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-262-0/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | 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-202-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| | | | | | | | | |
* | | | | | | | | | Merge pull request #4876 from ewolinetz/fix_podpreset_permsScott Dodson2017-07-261-1/+5
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / |/| | | | | | | | | Fixing podpresets perms for service-catalog-controller
| * | | | | | | | | 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
| | | | | | | | |
* | | | | | | | | Merge pull request #4866 from sdodson/timeoutstartsec-nodeScott Dodson2017-07-252-0/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Set TimeoutStartSec=300
| * | | | | | | | | 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
* | | | | | | | | Merge pull request #4863 from ↵Scott Dodson2017-07-252-2/+0
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | openshift/revert-4755-set-killmode-to-process-in-node-service-file Revert "set KillMode to process in node service file"
| * | | | | | | | 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 #4858 from jarrpa/unsupported-paramsScott Dodson2017-07-255-28/+28
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove unsupported parameters from example inventory files.
| * | | | | | | | | Remove unsupported parameters from example inventory files.Jose A. Rivera2017-07-255-28/+28
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* | | | | | | | | 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 #4856 from sdodson/fix-lintScott Dodson2017-07-241-3/+3
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix lint errors
| * | | | | | | | | | Fix lint errorsScott Dodson2017-07-241-3/+3
|/ / / / / / / / / /
* | | | | | | | | | Merge pull request #4846 from dcbw/libvirt-fixupsScott Dodson2017-07-242-2/+12
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | libvirt fixups
| * | | | | | | | | | libvirt: fall back to mkisofs if genisoimage isn't availableDan Williams2017-07-242-2/+6
| | | | | | | | | | |
| * | | | | | | | | | libvirt: add documentation about SSH keypair requirementsDan Williams2017-07-241-0/+6
| | |_|_|_|/ / / / / | |/| | | | | | | |