| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| |
| | |
Automatic merge from submit-queue.
disk_availability check: include submount storage
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.
|
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|\ \
| | |
| | | |
1504593 Refactor health check playbooks
|
| |/
| |
| |
| |
| |
| | |
- Standardize play/tasks naming
- Move install checks to separate playbook with checkpointing
- Correct 'docker_storage' tags
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, docker_image_availability.py plugin check is
using the raw strings for variables from task_vars.
This results in any variables that utilized within the
plugin to be un-templated. For instance, if variable
"x" is set to "{{ y }}" and y is set to "2", one
would expect that x == 2 inside the plugin. Currently,
the plugin will use the string "{{ y }}" for the value
of x instead of templating the variable.
This commit ensures skopeo registry auth credentials
are templated properly.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1500698
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
fixes bug 1496760
https://bugzilla.redhat.com/show_bug.cgi?id=1496760
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
docker_image_availability casted openshift_docker_additional_registries
to a list using the list() function. If a string was returned (IE: only
a single registry added) the result would be the string split up by
component characters. This change forces a string result from get_var to
be placed inside a list. If the result is anything BUT a string the
original list() function is called on the result.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1497274
Signed-off-by: Steve Milner <smilner@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
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`.
|
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|\
| |
| | |
Merged by openshift-bot
|
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| |
| | |
An image in the docker index may be tagged by name or by registry plus
name. In order to find the image correctly locally and prevent looking
for it externally, make sure all possible variations are searched.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
|
|/
|
|
|
|
| |
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.
|
|\
| |
| | |
Merged by openshift-bot
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/ |
|
|\
| |
| |
| |
| | |
Miciah/openshift_checks-ignore-hidden-files-in-checks-directory
openshift_checks: ignore hidden files in checks dir
|
| |
| |
| |
| | |
load_checks: Ignore hidden files when scanning the directory for checks.
|
|\ \
| | |
| | | |
disk_availability: fix bug where msg is overwritten
|
| | | |
|
|\ \ \
| | | |
| | | | |
docker_image_availability: timeout skopeo inspect
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Set a 10 second timeout when using skopeo to inspect remote registries,
so that it does not wait for a tcp timeout to fail if they are unreachable.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
And beautify the code a bit.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This serves two purposes:
- Gracefully omit the summary if there was an error computing it, no
confusion to the regular end user.
- Provide a stacktrace of the error when running verbose, giving
developers or users reporting bugs a better insight of what went wrong,
as opposed to Ansible's opaque handling of errors in callback plugins.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
The intent is to deduplicate similar errors that happened in many hosts,
making the summary more concise.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This prevents an exception in one check from interfering with other
checks. Skips checks that raise an exception in their is_active method.
Whenever capturing a broad exception in the `is_action` or `run`
methods, include traceback information that can be useful in bug
reports.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is useful on its own, and also aids in developing/testing new
checks that are not part of any playbook.
Since the intent when running this playbook is to execute checks, opt
for a less verbose explanation on the error summary.
|
|\ \ \
| |/ /
|/| | |
Add dnf support
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Miller <maxamillion@fedoraproject.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Miller <maxamillion@fedoraproject.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Adam Miller <maxamillion@fedoraproject.org>
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | | |
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
|
| |/
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|\ \
| | |
| | | |
Removing openshift_repo dependencies. Moving to beginning of openshift_cluster.
|
| | |
| | |
| | |
| | | |
cluster build.
|
|\ \ \
| |/ /
|/| | |
openshift_checks: refactor find_ansible_mount
|
| |/
| |
| |
| | |
Reuse the code for finding the ansible_mounts mount for a path.
|
|\ \
| | |
| | | |
openshift_checks: enable variable conversion
|