Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Clean up unnecessary quotes | Rodolfo Carvalho | 2017-07-27 | 3 | -3/+3 |
| | |||||
* | Make LoggingCheck.run return the correct type | Rodolfo Carvalho | 2017-07-27 | 1 | -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 playbooks | Rodolfo Carvalho | 2017-07-27 | 2 | -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.6 | Scott Dodson | 2017-07-27 | 1 | -1/+2 |
|\ | | | | | Follow the new naming conventions. | ||||
| * | Follow the new naming conventions. | William Chang | 2017-07-27 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #4882 from sdodson/fix-node-resolv-conf | Scott Dodson | 2017-07-27 | 1 | -11/+18 |
|\ \ | | | | | | | Simplify generation of /etc/origin/node/resolv.conf | ||||
| * | | Simplify generation of /etc/origin/node/resolv.conf | Scott Dodson | 2017-07-26 | 1 | -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-values | Jan Chaloupka | 2017-07-27 | 6 | -7/+16 |
|\ \ \ | |_|/ |/| | | Sync all openshift.common.use_openshift_sdn uses in yaml files | ||||
| * | | Sync all openshift.common.use_openshift_sdn uses in yaml files | Jan Chaloupka | 2017-07-26 | 6 | -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_caCert | Scott Dodson | 2017-07-26 | 1 | -1/+1 |
|\ \ \ | |_|/ |/| | | Fixing route spec caCertificate to be correctly capitalized | ||||
| * | | Fixing route spec caCertificate to be correctly capitalized | ewolinetz | 2017-07-25 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #4881 from abutcher/glusterfs-cluster-hosts | Scott Dodson | 2017-07-26 | 1 | -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 Butcher | 2017-07-26 | 1 | -0/+1 |
| | | | | |||||
* | | | | Merge pull request #4448 from mtnbikenc/vars-syntax | Scott Dodson | 2017-07-26 | 1 | -4/+1 |
|\ \ \ \ | | | | | | | | | | | Remove invalid when: from vars: | ||||
| * | | | | Remove invalid when: from vars: | Russell Teague | 2017-06-14 | 1 | -4/+1 |
| | | | | | |||||
* | | | | | Merge pull request #4738 from sjug/opt_ansible_cfg | Scott Dodson | 2017-07-26 | 1 | -1/+9 |
|\ \ \ \ \ | | | | | | | | | | | | | Additional optimization parameters for ansible.cfg | ||||
| * | | | | | Additional optimization parameters for ansible.cfg | Sebastian Jug | 2017-07-12 | 1 | -1/+9 |
| | | | | | | |||||
* | | | | | | Merge pull request #4844 from ewolinetz/bz1463081 | Scott Dodson | 2017-07-26 | 2 | -6/+14 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Updating how storage type is determined, adding bool filter in opensh… | ||||
| * | | | | | | Updating how storage type is determined, adding bool filter in ↵ | ewolinetz | 2017-07-24 | 2 | -6/+14 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | openshift_logging_elasticsearch | ||||
* | | | | | | | Merge pull request #4816 from abutcher/containerized-lb-image-tag | Scott Dodson | 2017-07-26 | 2 | -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 Butcher | 2017-07-20 | 2 | -0/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | containerized haproxy installation. | ||||
* | | | | | | | | Merge pull request #4485 from sosiouxme/20170616-store-args-in-check-properties | Scott Dodson | 2017-07-26 | 43 | -544/+494 |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | openshift_checks: store args in check properties | ||||
| * | | | | | | | | openshift_checks: refactor to internalize task_vars | Luke Meyer | 2017-07-25 | 39 | -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_executor | Luke Meyer | 2017-07-25 | 4 | -25/+10 |
| | | | | | | | | | |||||
| * | | | | | | | | openshift_checks: improve comments/names | Luke Meyer | 2017-07-25 | 13 | -51/+57 |
| | | | | | | | | | |||||
* | | | | | | | | | Merge pull request #4693 from enoodle/create_router_certificate_by_default | Scott Dodson | 2017-07-26 | 2 | -4/+4 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Router wildcard certificate created by default | ||||
| * | | | | | | | | | add default value for router path in the cert | Erez Freiberger | 2017-07-25 | 1 | -2/+2 |
| | | | | | | | | | | |||||
| * | | | | | | | | | Router wildcard certificate created by default | Erez Freiberger | 2017-07-25 | 2 | -2/+2 |
| | | | | | | | | | | |||||
* | | | | | | | | | | Merge pull request #4876 from ewolinetz/fix_podpreset_perms | Scott Dodson | 2017-07-26 | 1 | -1/+5 |
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / |/| | | | | | | | | | Fixing podpresets perms for service-catalog-controller | ||||
| * | | | | | | | | | Fixing podpresets perms for service-catalog-controller | ewolinetz | 2017-07-26 | 1 | -1/+5 |
|/ / / / / / / / / | |||||
* | | | | | | | | | Merge pull request #4813 from sdodson/log-dumper | Scott Dodson | 2017-07-26 | 2 | -1/+21 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Fix log dumping on service failure | ||||
| * | | | | | | | | | Fix log dumping on service failure | Scott Dodson | 2017-07-20 | 2 | -1/+21 |
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Also, add log dumping to master service startup too | ||||
* | | | | | | | | | Merge pull request #4855 from mwringe/hawkular-namespace-listener-update | Scott Dodson | 2017-07-26 | 3 | -1/+41 |
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | | Metrics: grant hawkular namespace listener role | ||||
| * | | | | | | | | Metrics: grant hawkular namespace listener role | mwringe | 2017-07-24 | 3 | -1/+41 |
| | | | | | | | | | |||||
* | | | | | | | | | Merge pull request #4687 from mrooding/fix-etcd-conditional-check-failure | Jan Chaloupka | 2017-07-26 | 1 | -1/+2 |
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | | Fix etcd conditional check failure (issue #4121) | ||||
| * | | | | | | | | Fix etcd conditional check failure | Marc Rooding | 2017-07-07 | 1 | -1/+2 |
| | | | | | | | | | |||||
* | | | | | | | | | Merge pull request #4866 from sdodson/timeoutstartsec-node | Scott Dodson | 2017-07-25 | 2 | -0/+2 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Set TimeoutStartSec=300 | ||||
| * | | | | | | | | | Set TimeoutStartSec=300 | Scott Dodson | 2017-07-25 | 2 | -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 Dodson | 2017-07-25 | 2 | -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 Dodson | 2017-07-25 | 2 | -2/+0 |
|/ / / / / / / / | |||||
* | | | | | | | | Merge pull request #4854 from ewolinetz/metrics_remove_no_log | Scott Dodson | 2017-07-25 | 1 | -1/+0 |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Removing no_log from htpasswd invocation so not to supress errors | ||||
| * | | | | | | | | Removing nolog from htpasswd invocation so not to supress errors | ewolinetz | 2017-07-24 | 1 | -1/+0 |
| | | | | | | | | | |||||
* | | | | | | | | | Merge pull request #4858 from jarrpa/unsupported-params | Scott Dodson | 2017-07-25 | 5 | -28/+28 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Remove unsupported parameters from example inventory files. | ||||
| * | | | | | | | | | Remove unsupported parameters from example inventory files. | Jose A. Rivera | 2017-07-25 | 5 | -28/+28 |
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jose A. Rivera <jarrpa@redhat.com> | ||||
* | | | | | | | | | Merge pull request #4839 from tripledes/unexclude_use_abs_path | Scott Dodson | 2017-07-25 | 1 | -2/+2 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Use absolute path when un-excluding | ||||
| * | | | | | | | | | Use absolute path when unexcluding | Sergi Jimenez | 2017-07-24 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * As follow-up on #4835, use absolute path also when unexcluding | ||||
* | | | | | | | | | | Merge pull request #4856 from sdodson/fix-lint | Scott Dodson | 2017-07-24 | 1 | -3/+3 |
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | Fix lint errors | ||||
| * | | | | | | | | | | Fix lint errors | Scott Dodson | 2017-07-24 | 1 | -3/+3 |
|/ / / / / / / / / / | |||||
* | | | | | | | | | | Merge pull request #4846 from dcbw/libvirt-fixups | Scott Dodson | 2017-07-24 | 2 | -2/+12 |
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | libvirt fixups | ||||
| * | | | | | | | | | | libvirt: fall back to mkisofs if genisoimage isn't available | Dan Williams | 2017-07-24 | 2 | -2/+6 |
| | | | | | | | | | | |