summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker
Commit message (Collapse)AuthorAgeFilesLines
* memory check: use GiB/MiB and adjust memtotalLuke Meyer2017-05-292-20/+36
| | | | | | | | | 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."
* memory health check: adjust threshold for etcdLuke Meyer2017-05-232-4/+10
|
* health checks: specify check skip reasonLuke Meyer2017-05-232-9/+24
| | | | | Added indicator to check result for why that check was skipped. Note that currently the user will only see it with ansible-playbook -vv
* health checks: configure failure output in playbooksLuke Meyer2017-05-232-45/+68
| | | | | Customized the error summary to depend on the intent of the playbook run. Ensured output makes sense when failures are unrelated to running checks.
* disk/memory checks: make threshold configurableLuke Meyer2017-05-234-12/+65
|
* Show help on how to disable checks after failureRodolfo Carvalho2017-05-231-0/+22
|
* Allow disabling checks via Ansible variableRodolfo Carvalho2017-05-231-1/+8
| | | | | | | | | 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.
* remove skopeo dependency on docker-pyjuanvallejo2017-05-192-152/+143
|
* improve error handling for missing varsjuanvallejo2017-05-193-66/+226
|
* Merge pull request #3630 from juanvallejo/jvallejo/add-etcd-volume-checkOpenShift Bot2017-05-192-0/+207
|\ | | | | Merged by openshift-bot
| * revert role-specific var namejuanvallejo2017-05-151-1/+1
| |
| * Merge branch 'jvallejo/add-etcd-volume-check' of ↵juanvallejo2017-05-121-5/+7
| |\ | | | | | | | | | github.com:juanvallejo/openshift-ansible into jvallejo/add-etcd-volume-check
| | * Update variable name to standardRodolfo Carvalho2017-05-111-1/+1
| | | | | | | | | It was agreed to name role variables as `r_ROLE_NAME_VARIABLE_NAME`. Giving it a try.
| | * Make class attribute name shorterRodolfo Carvalho2017-05-111-4/+4
| | |
| | * Add module docstringRodolfo Carvalho2017-05-111-0/+2
| | |
| * | check if hostname is in list of etcd hostsjuanvallejo2017-05-121-3/+4
| |/
| * Update checkRodolfo Carvalho2017-05-102-48/+46
| |
| * int -> floatRodolfo Carvalho2017-05-101-3/+3
| | | | | | We don't need to convert to int and then to float. Read it as float from the start.
| * Remove vim lineRodolfo Carvalho2017-05-101-1/+0
| | | | | | It has been agreed that we don't use it any longer.
| * add etcd volume checkjuanvallejo2017-05-092-0/+207
| |
* | Merge pull request #3660 from juanvallejo/jvallejo/add-etcd-clustersize-checkOpenShift Bot2017-05-183-0/+534
|\ \ | | | | | | Merged by openshift-bot
| * | add etcd cluster size checkjuanvallejo2017-05-173-0/+534
| | |
* | | Remove vim configuration from Python filesRodolfo Carvalho2017-05-093-3/+0
|/ / | | | | | | | | | | | | In a project where contributors are free to use whatever editor they want and we have linting tools that verify the proper formatting of Python files, it should not be required to have a vim-specific line in Python files.
* | Merge pull request #3919 from rhcarvalho/fix-action-plugin-test-ansible2.3Rodolfo Carvalho2017-04-201-1/+3
|\ \ | | | | | | Fix instantiation of action plugin test to support Ansible 2.3
| * | Fix instantiation of action plugin in test fixtureRodolfo Carvalho2017-04-131-1/+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.
* | | Simplify memory availability check, review testsRodolfo Carvalho2017-04-173-74/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - 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.
* | | Simplify mixin classRodolfo Carvalho2017-04-171-10/+7
| | | | | | | | | | | | | | | - Expose only is_active and no other method. - Move general comment to module docstring.
* | | Simplify disk availability check, review testsRodolfo Carvalho2017-04-172-124/+165
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - 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.
* | | add disk and memory availability check testsjuanvallejo2017-04-173-1/+194
| | |
* | | add ram and storage preflight checkjuanvallejo2017-04-172-0/+108
| |/ |/|
* | Merge pull request #3887 from rhcarvalho/pkg-version-only-master-or-nodeOpenShift Bot2017-04-132-0/+29
|\ \ | |/ |/| Merged by openshift-bot
| * Do not check package version on non-master/nodeRodolfo Carvalho2017-04-102-0/+29
| |
* | Merge pull request #3757 from ↵OpenShift Bot2017-04-122-0/+18
|\ \ | | | | | | | | | | | | juanvallejo/jvallejo/skip-yum-update-check-on-fedora Merged by openshift-bot
| * | skip PackageAvailability check if not yumjuanvallejo2017-04-112-0/+18
| | |
* | | preflight checks: refactor and fix aos_versionLuke Meyer2017-04-086-64/+295
| |/ |/| | | | | | | | | | | | | | | | | | | 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.
* | Add tests for action pluginRodolfo Carvalho2017-04-041-15/+168
| |
* | Remove unnecessary codeRodolfo Carvalho2017-04-041-5/+1
| |
* | Make resolve_checks more strictRodolfo Carvalho2017-04-043-23/+114
|/ | | | | So that all names it return are directly usable, without checking for existence.
* Add test scaffold for docker_image_availability.pyRodolfo Carvalho2017-03-282-0/+39
| | | | | The intention is to set a starting point and let another team member work on the code to gain experience with tests.
* Add unit tests for package_version.pyRodolfo Carvalho2017-03-281-0/+21
|
* Add unit tests for package_update.pyRodolfo Carvalho2017-03-281-0/+16
|
* Add unit tests for package_availability.pyRodolfo Carvalho2017-03-281-0/+49
|
* Add unit tests for mixins.pyRodolfo Carvalho2017-03-281-0/+23
|
* Test recursively finding subclassesRodolfo Carvalho2017-03-281-0/+17
|
* Test OpenShift health check loaderRodolfo Carvalho2017-03-283-14/+26
|
* Rename module_executor -> execute_moduleRodolfo Carvalho2017-03-286-7/+47
| | | | | | | | 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.
* vendor patched upstream docker_container module.juanvallejo2017-03-231-0/+2036
| | | | | | | | | | | | | | | | Due to the use of a restricted name in the core `docker_container` module's result, any standard output of a docker container captured in the module's response was stripped out by ansible. Because of this, we are forced to vendor a patched version of this module, until a new version of ansible is released containing the patched module. This file should be removed once we begin requiring a release of ansible containing the patched `docker_container` module. This patch was taken directly from upstream, with no further changes: 20bf02f6b96356ab5fe68578a3af9462b4ca42a5
* add docker_image_availability checkjuanvallejo2017-03-233-0/+193
| | | | | | | | 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.
* Merge pull request #3710 from sosiouxme/20170320-fix-output-and-exclusionsOpenShift Bot2017-03-203-10/+26
|\ | | | | Merged by openshift-bot
| * preflight checks: improve user output from checksLuke Meyer2017-03-201-10/+24
| |
generated by cgit v1.2.1 (git 2.18.0) at 2024-05-22 07:47:29 +0000