summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker/test
Commit message (Collapse)AuthorAgeFilesLines
* registry-console: align image and checkLuke Meyer2017-11-151-2/+2
| | | | | enable option to configure basename in image docker_image_availability check: follow registry-console image options
* Merge pull request #5829 from sosiouxme/20171020-registry-console-bz1497310Scott Dodson2017-11-081-5/+41
|\ | | | | reconcile registry-console and docker_image_availability
| * reconcile registry-console and docker_image_availabilityLuke Meyer2017-11-061-5/+41
| | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1497310 https://bugzilla.redhat.com/show_bug.cgi?id=1497310 The registry console is a special case in more than one way. This adds logic to incorporate the openshift_cockpit_deployer_* variables into determining what its image will be in docker_image_availability. Along the way I noticed the origin and enterprise templates for this were not consistent. Now they are, and the example hosts file is updated.
* | openshift_checks: Add OVS versions for OCP 3.7Miciah Masters2017-11-061-1/+2
|/ | | | | | | | | Update the ovs_version check with the allowed Open vSwitch versions for OCP 3.7. Add OVS 2.8 to the allowed versions for OCP 3.6 as well. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1509163
* disk_availability check: include submount storageLuke Meyer2017-10-201-2/+21
| | | | | | | | Fixes bug 1491566 https://bugzilla.redhat.com/show_bug.cgi?id=1491566 In order to determine how much storage is under a path, include any mounts that are below it in addition to the path itself.
* docker_image_availability: credentials to skopeoMichael Gugino2017-10-062-40/+10
| | | | | | | | | | | | | | | | | | Currently, docker_image_availability health_check does not support authenticated registries. This commit adds the '--creds=' option to skopeo if needed to support authentication credentials. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1316341 Some other fixes to handle docker config better: Should now account properly for blocked registries, insecure registries, multiple additional registries, and oreg_url registry with or without credentials. Output on failure should be clearer about what was tried. Fixed a bug in the action_plugin_test exposed by these changes.
* openshift_checks: lb and nfs do not need dockerLuke Meyer2017-10-041-2/+2
| | | | | fixes bug 1496760 https://bugzilla.redhat.com/show_bug.cgi?id=1496760
* openshift_checks: use oo group names everywhereLuke Meyer2017-10-049-82/+82
|
* Migrate enterprise registry logic to docker roleMichael Gugino2017-09-271-5/+5
| | | | | | | | | | | | | Currently, the enterprise registry to forcefully added in openshift_facts. Recently, the docker role has been modified to consume registry variables directly, bypassing openshift_facts. This commit cleans up unused code in openshift_facts, and migrates enterprise registry logic to the docker role. Fixes: https://github.com/openshift/openshift-ansible/issues/5557
* Merge pull request #5491 from sosiouxme/20170920-diagnostics-checkOpenShift Merge Robot2017-09-222-5/+53
|\ | | | | | | | | | | | | | | | | Automatic merge from submit-queue health checks: add diagnostics check Adds a health check that runs `oc adm diagnostics` with each individual diagnostic. Also, moved `is_first_master` method into superclass for reuse. And look at `oo_first_master` and `ansible_host` instead of `masters` and `ansible_ssh_host`.
| * health checks: add diagnostics checkLuke Meyer2017-09-212-5/+53
| | | | | | | | | | | | Also, moved is_first_master method into superclass for reuse. And look at oo_first_master and ansible_host instead of masters and ansible_ssh_host.
* | Cleanup old deployment typesMichael Gugino2017-09-201-2/+33
|/ | | | | | | | | | | Previously, openshift-ansible supported various types of deployments using the variable "openshift_deployment_type" Currently, openshift-ansible only supports two deployment types, "origin" and "openshift-enterprise". This commit removes all logic and references to deprecated deployment types.
* openshift_checks: enable providing file outputsLuke Meyer2017-09-188-47/+166
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Some refactoring of checks and the action plugin to enable writing files locally about the check operation and results, if the user wants them. This is aimed at enabling persistent and machine-readable results from recurring runs of health checks. Now, rather than trying to build a result hash to return from running each check, checks can just register what they need to as they're going along, and the action plugin processes state when the check is done. Checks can register failures, notes about what they saw, and arbitrary files to be saved into a directory structure where the user specifies. If no directory is specified, no files are written. At this time checks can still return a result hash, but that will likely be refactored away in the next iteration. Multiple failures can be registered without halting check execution. Throwing an exception or returning a hash with "failed" is registered as a failure. execute_module now does a little more with the results. Results are automatically included in notes and written individually as files. "changed" results are propagated. Some json results are decoded. A few of the checks were enhanced to use these features; all get some of the features for free.
* Merge pull request #5365 from sosiouxme/20170908-disconnected-image-checkOpenShift Bot2017-09-125-109/+103
|\ | | | | Merged by openshift-bot
| * openshift_health_check: allow disabling all checksLuke Meyer2017-09-121-2/+7
| | | | | | | | | | | | | | | | Can now set openshift_disable_check=* to disable all checks without needing to know their names. fixes bug 1462106 https://bugzilla.redhat.com/show_bug.cgi?id=1462106
| * docker_image_availability: probe registry connectivityLuke Meyer2017-09-121-99/+82
| | | | | | | | | | | | | | | | | | | | | | | | Probe whether the host has connectivity to the registry before trying to inspect it for images, and remember the result. Also if later inspection fails due to timeout, mark registry as unreachable. Note in failure output if any registries were unreachable. Registry order should match what is configured into docker now as well. Fixes bug 1480195 https://bugzilla.redhat.com/show_bug.cgi?id=1480195
| * openshift_checks: add retries in pythonLuke Meyer2017-09-124-14/+20
| |
* | Skip failure dedup instead of crashingRodolfo Carvalho2017-09-111-0/+15
|/ | | | | | This makes the callback plugin behave better when dedup is not possible: work with the original list of failures instead of raising an unhandled exception and producing confusing output for users.
* openshift_checks aos_version: also check installed under yumLuke Meyer2017-09-061-0/+2
| | | | | | | | | Tweaks to the logic around using yum vs dnf; now uses ansible_pkg_mgr to determine which is in effect for a host. Also, extended the yum logic to check installed packages in addition to available packages in the aos_version module so that disconnected installs and others with weird repo configs need not disable the package_version check.
* Update error message: s/non-unique/duplicateRodolfo Carvalho2017-08-241-1/+1
|
* Rewrite failure summary callback pluginRodolfo Carvalho2017-08-242-0/+71
| | | | | The intent is to deduplicate similar errors that happened in many hosts, making the summary more concise.
* List known checks/tags when check name is invalidRodolfo Carvalho2017-08-241-8/+2
|
* List existing health checks when none is requestedRodolfo Carvalho2017-08-241-1/+2
| | | | | | | | This is a simple mechanism to learn what health checks are available. Note that we defer task_vars verification, so that we can compute requested_checks and resolved_checks earlier, allowing us to list checks even if openshift_facts has not run.
* etc_traffic check: factor away short_versionLuke Meyer2017-08-151-4/+2
|
* Merge pull request #5036 from ↵Scott Dodson2017-08-152-5/+5
|\ | | | | | | | | Miciah/openshift_checks-support-ovs-2.7-on-ocp-3.5-and-3.6 openshift_checks: allow OVS 2.7 on OCP 3.5 and 3.6
| * openshift_checks: allow OVS 2.7 on OCP 3.5 and 3.6Miciah Masters2017-08-112-5/+5
| | | | | | | | | | | | | | | | | | rpm_version: Allow package_list items to specify a list value for version. If a list value is provided for a package, pass the check if any version in that list is found. ovs_version: Specify both 2.6 and 2.7 as allowed versions of OVS for OpenShift versions 3.5 and 3.6.
* | Merge pull request #4944 from sosiouxme/20170728-refactor-ansible-mountsScott Dodson2017-08-113-15/+29
|\ \ | | | | | | openshift_checks: refactor find_ansible_mount
| * | openshift_checks: refactor find_ansible_mountLuke Meyer2017-08-083-15/+29
| |/ | | | | | | Reuse the code for finding the ansible_mounts mount for a path.
* | Merge pull request #4922 from sosiouxme/20170728-improve-get-varsScott Dodson2017-08-091-0/+22
|\ \ | |/ |/| openshift_checks: enable variable conversion
| * openshift_checks: enable variable conversionLuke Meyer2017-08-011-0/+22
| |
* | Merge pull request #4913 from sosiouxme/20170720-refactor-check-resultsRodolfo Carvalho2017-08-088-250/+307
|\ \ | | | | | | openshift_checks: refactor check results
| * | openshift_checks: refactor logging checksLuke Meyer2017-08-027-245/+298
| | | | | | | | | | | | | | | | | | | | | Turn failure messages into exceptions that tests can look for without depending on text meant for humans. Turn logging_namespace property into a method. Get rid of _exec_oc and just use logging.exec_oc.
| * | openshift_checks: add property to track 'changed'Luke Meyer2017-08-021-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduced the 'changed' property for checks that can make changes to track whether they did or not. Rather than the check's own logic having to track this and include it in the result hash, just set the property and have the action plugin insert it in the result hash after running (even if there is an exception). Cleared out a lot of crufty "changed: false" hash entries.
* | | Merge pull request #4960 from ↵OpenShift Bot2017-08-071-5/+64
|\ \ \ | | | | | | | | | | | | | | | | juanvallejo/jvallejo/verify-disk-memory-before-upgrade-no-flake Merged by openshift-bot
| * | | add pre-flight checks to ugrade pathjuanvallejo2017-08-021-5/+64
| | |/ | |/|
* | | Merge pull request #4969 from sosiouxme/20170801-tolerate-ovs-beyond-36OpenShift Bot2017-08-021-36/+26
|\ \ \ | |_|/ |/| | Merged by openshift-bot
| * | package_version check: tolerate release version 3.7Luke Meyer2017-08-021-36/+26
| |/ | | | | | | | | | | | | | | | | | | | | Addresses issue https://github.com/openshift/openshift-ansible/issues/4967 For now, any version >= 3.6 is handled as if it were 3.6. We may want to keep that or fine-tune it later. Also, the ovs_version check is not updated. This is a post-install health check (does not block install/upgrade) with an update already in progress so will be addressed there.
* | add fluentd logging driver config checkjuanvallejo2017-08-014-37/+376
|/
* Revert "Add health checks to upgrade playbook"Rodolfo Carvalho2017-07-281-64/+5
|
* fixes after rebasing with #4485juanvallejo2017-07-261-2/+2
|
* add pre-flight checks to ugrade pathjuanvallejo2017-07-261-5/+64
|
* openshift_checks: refactor to internalize task_varsLuke Meyer2017-07-2519-182/+152
| | | | | | | | | 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-251-16/+2
|
* 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.