summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker/openshift_checks/docker_storage.py
Commit message (Collapse)AuthorAgeFilesLines
* 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