| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
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
|
| | | | |
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | | |
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."
|
| | | |
|
| |/
|/| |
|
| | |
|
|/ |
|
|\
| |
| | |
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.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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.
|
| |
| |
| |
| |
| | |
- Expose only is_active and no other method.
- Move general comment to module docstring.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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.
|
| | |
|
|/ |
|
|\
| |
| | |
Merged by openshift-bot
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
juanvallejo/jvallejo/skip-yum-update-check-on-fedora
Merged by openshift-bot
|
| |/ |
|
|/
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
The intention is to set a starting point and let another team member
work on the code to gain experience with tests.
|
| |
|
|
|
|
|
|
|
|
| |
It is a function/callable, the name should imply action, should be a
verb and not a noun.
Keep supporting the old name while we have PRs in-flight that use the
old name.
|
|
|
|
|
|
|
|
| |
This patch adds a check to ensure that required docker images are
available in at least one of the registries supplied in an installation
host. Images are available if they are either already present locally,
or able to be inspected using Skopeo on one of the configured
registries.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It turns out the code in
roles/openshift_health_checker/openshift_checks/__init__.py had a
mistake in the import statement, and not the case that `add_metaclass`
is not available in the embedded `six` from `ansible.module_utils`.
The fix is to correct the import and do not require python-six to be
installed on target hosts.
Note: now that we have some unit tests in the role, errors like that
would have been caught.
|
| |
|