| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the past, health checks were implemented with ignore_errors: True in
the playbook level, requiring us to store all failures, ignored or not,
so that we could report on all failed checks.
Now checks are run from a single action plugin entry point, without
ignoring errors (all errors are aggregated via the action plugin).
Since the integration of the openshift_health_checker role with the
install playbook, failure summaries are part of the output of a lot more
calls to ansible-playbook. We shall report only failures that caused the
execution to stop, as ignored failures in the summary only serve to
confuse users.
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
| |
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.
|
|\
| |
| | |
Merged by openshift-bot
|
| |
| |
| |
| | |
fixes bug 1466622 - docker_image_availability check on etcd host failed for 'openshift_image_tag' is undefined
|
|\ \
| | |
| | | |
Merged by openshift-bot
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
fix bug 1464974 https://bugzilla.redhat.com/show_bug.cgi?id=1464974
Specify --units on vgs call. In my testing with lvm 2.0.2.171(2) on
RHEL Atomic Host 7.4, this turned a response of "<4.07g" into "4.07g"
which should resolve the issue. I haven't found what the "<" is for
in the first place but I'm thinking this should at least be a safe
change.
|
|\ \
| | |
| | | |
Merged by openshift-bot
|
| |/
| |
| |
| |
| |
| |
| | |
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).
|
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
|
| | |
|
| |
| |
| |
| | |
During install, those paths are used and require some free space.
|
|/
|
|
| |
Prepare the check to support verifying multiple paths, not only /var.
|
|
|
|
|
|
|
| |
Some registries are not configured with valid certificates and thus the
check fails with 'http: server gave HTTP response to HTTPS client'.
Since this is not fetching images, but only checking for existence,
trade security for convenience.
|
|
|
|
|
|
|
|
| |
The package name has changed.
See
https://bugzilla.redhat.com/show_bug.cgi?id=1461689
https://bugzilla.redhat.com/show_bug.cgi?id=1419718
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|\
| |
| | |
Merged by openshift-bot
|
| |
| |
| | |
This avoids unintentionally overriding the value from `True` to `False`.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | |
|
|\ \
| | |
| | | |
Merged by openshift-bot
|
| | | |
|
|\ \ \
| | | |
| | | | |
Merged by openshift-bot
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Capturing the ImportError is a common idiom in Ansible modules, and it
is not specific to tox.
|
| | | | |
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | | |
juanvallejo/jvallejo/add-retroactive-ovs-version-check
Merged by openshift-bot
|
| | |/
| |/| |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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."
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Added indicator to check result for why that check was skipped.
Note that currently the user will only see it with ansible-playbook -vv
|
| | |
| | |
| | |
| | |
| | | |
Customized the error summary to depend on the intent of the playbook run.
Ensured output makes sense when failures are unrelated to running checks.
|
| | | |
|
| | | |
|
| |/
|/|
| |
| |
| |
| |
| |
| |
| | |
Example usage:
$ ansible-playbook -i hosts playbooks/byo/config.yml -e
openshift_disable_check=memory_availability,disk_availability
Or add the variable to the inventory / hosts file.
|
| | |
|
|/ |
|
|\
| |
| | |
Merged by openshift-bot
|
| | |
|
| |\
| | |
| | |
| | | |
github.com:juanvallejo/openshift-ansible into jvallejo/add-etcd-volume-check
|
| | |
| | |
| | | |
It was agreed to name role variables as `r_ROLE_NAME_VARIABLE_NAME`. Giving it a try.
|
| | | |
|
| | | |
|
| |/ |
|
| | |
|
| |
| |
| | |
We don't need to convert to int and then to float. Read it as float from the start.
|
| |
| |
| | |
It has been agreed that we don't use it any longer.
|
| | |
|
|\ \
| | |
| | | |
Merged by openshift-bot
|