summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker/openshift_checks/docker_storage.py
Commit message (Collapse)AuthorAgeFilesLines
* openshift_checks: refactor find_ansible_mountLuke Meyer2017-08-081-21/+1
| | | | Reuse the code for finding the ansible_mounts mount for a path.
* openshift_checks: add property to track 'changed'Luke Meyer2017-08-021-6/+4
| | | | | | | | | | Introduced the 'changed' property for checks that can make changes to track whether they did or not. Rather than the check's own logic having to track this and include it in the result hash, just set the property and have the action plugin insert it in the result hash after running (even if there is an exception). Cleared out a lot of crufty "changed: false" hash entries.
* openshift_checks: refactor to internalize task_varsLuke Meyer2017-07-251-18/+18
| | | | | | | | | Move task_vars into instance variable so we don't have to pass it around everywhere. Also store tmp. Make sure both are filled in on execute_module. In the process, is_active became an instance method, and task_vars is basically never used directly outside of test code.
* openshift_checks/docker_storage: overlay/2 supportLuke Meyer2017-07-191-32/+145
| | | | | | | | fix bug 1469197 https://bugzilla.redhat.com/show_bug.cgi?id=1469197 When Docker is configured with the overlay or overlay2 storage driver, check that it is supported and usage is below threshold.
* Add overlay to supported Docker storage driversRodolfo Carvalho2017-07-111-1/+1
| | | | | | | 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.
* docker_storage check: make vgs return sane outputLuke Meyer2017-06-271-1/+1
| | | | | | | | | | 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.
* pre-install checks: add more during byo installLuke Meyer2017-06-141-2/+2
| | | | | | | | | | | | 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.
* Consider previous value of 'changed' when updatingRodolfo Carvalho2017-06-091-1/+1
| | | This avoids unintentionally overriding the value from `True` to `False`.
* docker checks: finish and refactorLuke Meyer2017-06-071-91/+166
| | | | | | | | | 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.
* add docker storage, docker driver checksjuanvallejo2017-06-011-0/+110