| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
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
|
| |
|
| |
|
| |
|
|\
| |
| | |
Merged by openshift-bot
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Fix instantiation of action plugin test to support Ansible 2.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.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- 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.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- 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.
|
| | |
|
|/
|
|
|
| |
So that all names it return are directly usable, without checking for
existence.
|
|
|
|
|
| |
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.
|
|
|