summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker/test
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge pull request #4682 from juanvallejo/jvallejo/verify-logging-index-timeRodolfo Carvalho2017-07-242-1/+183
|\ | | | | verify sane log times in logging stack
| * verify sane log times in logging stackjuanvallejo2017-07-202-1/+183
| | | | | | | | | | This patch verifies that logs sent from logging pods can be queried on the Elasticsearch pod within a reasonable amount of time.
* | Merge pull request #4316 from ↵Rodolfo Carvalho2017-07-202-0/+237
|\ \ | | | | | | | | | | | | juanvallejo/jvallejo/add-increased-etcd-traffic-check add check to detect increased etcd traffic
| * | add etcd increased-traffic checkjuanvallejo2017-07-192-0/+237
| | |
* | | openshift_checks/docker_storage: overlay/2 supportLuke Meyer2017-07-191-10/+97
| | | | | | | | | | | | | | | | | | | | | | | | fix bug 1469197 https://bugzilla.redhat.com/show_bug.cgi?id=1469197 When Docker is configured with the overlay or overlay2 storage driver, check that it is supported and usage is below threshold.
* | | Allow OVS 2.7 in latest OpenShift releasesRodolfo Carvalho2017-07-171-30/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change the package_version check to tolerate either Open vSwitch 2.6 or 2.7. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1465882 This commit removes a unit test that adds no coverage and tests data instead of logic. This coupling makes every change to supported versions require the same changes to the tests.
* | | Make aos_version module handle multiple versionsRodolfo Carvalho2017-07-171-16/+73
| | | | | | | | | | | | | | | | | | | | | | | | Some packages are supported at more than one major.minor version at the same time. Support is added keeping backward compatibility: the 'version' key can be either a string (single version) or a list of versions.
* | | Split positive and negative unit testsRodolfo Carvalho2017-07-171-50/+40
| | | | | | | | | | | | | | | | | | | | | Split positive and negative tests into their own functions. This means less lines of code, clearer purpose, easier to understand what each test does or doesn't and to add new test cases.
* | | add scheduled pods checkjuanvallejo2017-07-111-0/+30
| | |
* | | Add overlay to supported Docker storage driversRodolfo Carvalho2017-07-111-2/+2
| | | | | | | | | | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1467809 As a next step, we can refine under which conditions the overlay driver is supported.
* | | openshift_checks: fix execute_module paramsLuke Meyer2017-07-112-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Fix where execute_module was being passed task_vars in place of tmp param. Most modules don't seem to use either and so this doesn't fail; but under some conditions (perhaps different per version of ansible?) it tried to treat the dict as a string and came back with a python stack trace.
* | | Merge pull request #4655 from sosiouxme/20170630-atomic-etcd-bz1466622OpenShift Bot2017-06-301-0/+14
|\ \ \ | | | | | | | | Merged by openshift-bot
| * | | docker_image_availability: fix containerized etcdLuke Meyer2017-06-301-0/+14
| | |/ | |/| | | | | | | fixes bug 1466622 - docker_image_availability check on etcd host failed for 'openshift_image_tag' is undefined
* | | Merge pull request #4565 from rhcarvalho/handle-incorrect-check-namesOpenShift Bot2017-06-301-1/+11
|\ \ \ | | | | | | | | Merged by openshift-bot
| * | | Capture exceptions when resolving available checksRodolfo Carvalho2017-06-231-1/+11
| |/ / | | | | | | | | | | | | | | | | | | Calling the action plugin (e.g. when running a playbook) with an incorrect check name was raising an unhandled exception, leading to poor output in Ansible (requiring a higher verbosity level to see what is going wrong).
* | | Enable disk check on containerized installsRodolfo Carvalho2017-06-221-13/+10
| | | | | | | | | | | | | | | | | | | | | According to the docs the disk requirements should be similar to non-containerized installs. https://docs.openshift.org/latest/install_config/install/rpm_vs_containerized.html#containerized-storage-requirements
* | | Require at least 1GB in /usr/bin/local and tempdirRodolfo Carvalho2017-06-221-1/+1
| | | | | | | | | | | | During install, those paths are used and require some free space.
* | | Refactor DiskAvailability for arbitrary pathsRodolfo Carvalho2017-06-221-1/+1
|/ / | | | | | | Prepare the check to support verifying multiple paths, not only /var.
* / pre-install checks: add more during byo installLuke Meyer2017-06-142-13/+90
|/ | | | | | | | | | | | Add the docker and RPM checks to the list that run at install time. They can be disabled the same as the existing ones. Removed cockpit-kubernetes RPM requirement as it no longer is. Fixed up docker_image_availability to handle oreg_url and other nuances. Switched to using the openshift_image_tag that's set by openshift_version for both component and infrastructure images. Fixed a bug where execute_module was being called with incorrect positional arg "tmp" as a dict which caused errors down the call stack.
* Merge pull request #3787 from juanvallejo/jvallejo/docker-storage-checkOpenShift Bot2017-06-092-9/+239
|\ | | | | Merged by openshift-bot
| * docker checks: finish and refactorLuke Meyer2017-06-073-283/+189
| | | | | | | | | | | | | | | | | | Incorporated docker_storage_driver into docker_storage as both need driver info. Corrected storage calculation to include VG free space, not just the current amount in the LV pool. Now makes no assumptions about pool name. Improved user messaging. Factored out some methods that can be shared with docker_image_availability.
| * add docker storage, docker driver checksjuanvallejo2017-06-012-0/+324
| |
* | Merge pull request #3643 from juanvallejo/jvallejo/elastic-search-checkOpenShift Bot2017-06-065-0/+712
|\ \ | | | | | | Merged by openshift-bot
| * | add elasticseatch, fluentd, kibana checkjuanvallejo2017-06-025-0/+712
| | |
* | | Merge pull request #4064 from juanvallejo/jvallejo/add-ovs-version-checkOpenShift Bot2017-06-052-38/+175
|\ \ \ | | | | | | | | Merged by openshift-bot
| * | | update aos_version module to support generic pkgs and versionsjuanvallejo2017-05-242-38/+175
| | | |
* | | | Merge pull request #4157 from ↵OpenShift Bot2017-06-052-0/+171
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | juanvallejo/jvallejo/add-retroactive-ovs-version-check Merged by openshift-bot
| * | | add existing_ovs_version checkjuanvallejo2017-05-192-0/+171
| | |/ | |/|
* | | memory check: use GiB/MiB and adjust memtotalLuke Meyer2017-05-291-10/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | fixes https://bugzilla.redhat.com/show_bug.cgi?id=1455884 Various things reserve memory such that memtotal is quite lower than the actual physical RAM of the system. It's larger as RAM increases but it's not really proportional so I just added a flat 1GiB adjustment in the comparison. This ought to "pass when it's close enough."
* | | memory health check: adjust threshold for etcdLuke Meyer2017-05-231-3/+9
| | |
* | | health checks: specify check skip reasonLuke Meyer2017-05-231-1/+15
| | | | | | | | | | | | | | | Added indicator to check result for why that check was skipped. Note that currently the user will only see it with ansible-playbook -vv
* | | disk/memory checks: make threshold configurableLuke Meyer2017-05-232-9/+58
| |/ |/|
* | remove skopeo dependency on docker-pyjuanvallejo2017-05-191-74/+73
| |
* | improve error handling for missing varsjuanvallejo2017-05-191-18/+169
|/
* Merge pull request #3630 from juanvallejo/jvallejo/add-etcd-volume-checkOpenShift Bot2017-05-191-0/+149
|\ | | | | Merged by openshift-bot
| * Update checkRodolfo Carvalho2017-05-101-13/+13
| |
| * add etcd volume checkjuanvallejo2017-05-091-0/+149
| |
* | add etcd cluster size checkjuanvallejo2017-05-171-0/+328
| |
* | Merge pull request #3919 from rhcarvalho/fix-action-plugin-test-ansible2.3Rodolfo Carvalho2017-04-201-1/+3
|\ \ | | | | | | Fix instantiation of action plugin test to support Ansible 2.3
| * | Fix instantiation of action plugin in test fixtureRodolfo Carvalho2017-04-131-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | In Ansible 2.3+, the base action plugin class' run method depends on accessing attributes (check_mode) of its play_context. In older versions play_context was not involved in run, and thus None was passed in.
* | | Simplify memory availability check, review testsRodolfo Carvalho2017-04-172-52/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Fix required memory for etcd hosts (10 -> 20 GB), as per documentation. - Some changes to make the code more similar to the similar DiskAvailability check. - Do not raise exception for hosts that do not have a recommended memory value (those are ignored anyway through `is_active`, so that was essentially dead code). - Test that the required memory is the max of the recommended memories for all groups assigned to a host. E.g. if a host is master and node, we should check that it has enough memory to be a master, because the memory requirement for a master is higher than for a node.
* | | Simplify disk availability check, review testsRodolfo Carvalho2017-04-171-78/+127
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - only support a fixed list of recommended values for now, no overwriting via Ansible variables (keep it simple, add features as needed). - implement is_active: run this check only for hosts that have a recommended disk space. - test priority of mount paths / and /var.
* | | add disk and memory availability check testsjuanvallejo2017-04-172-0/+190
| |/ |/|
* | Merge pull request #3887 from rhcarvalho/pkg-version-only-master-or-nodeOpenShift Bot2017-04-131-0/+22
|\ \ | |/ |/| Merged by openshift-bot
| * Do not check package version on non-master/nodeRodolfo Carvalho2017-04-101-0/+22
| |
* | Merge pull request #3757 from ↵OpenShift Bot2017-04-121-0/+14
|\ \ | | | | | | | | | | | | juanvallejo/jvallejo/skip-yum-update-check-on-fedora Merged by openshift-bot
| * | skip PackageAvailability check if not yumjuanvallejo2017-04-111-0/+14
| | |
* | | preflight checks: refactor and fix aos_versionLuke Meyer2017-04-083-6/+130
| |/ |/| | | | | | | | | | | | | | | | | | | Bring in openshift_repos to provide Origin repo before checks on Origin. For OCP we want the check to fail if both version 3.3 and version 3.4 are available - they shouldn't have both channels enabled. For Origin everything is in one repo so it's not surprising to find 1.4 and 1.5 versions available. Added unit tests as well.
* | Add tests for action pluginRodolfo Carvalho2017-04-041-15/+168
| |
* | Make resolve_checks more strictRodolfo Carvalho2017-04-042-2/+81
|/ | | | | So that all names it return are directly usable, without checking for existence.