-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release-3.6] openshift_checks/docker_storage: overlay/2 support #4791
[release-3.6] openshift_checks/docker_storage: overlay/2 support #4791
Conversation
Document the option so that those who want to run on OverlayFS can find it. Fix the task so that setting it to False isn't interpreted as true.
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.
aos-ci-test |
[merge] |
should have been severity blocker anyway, but: |
openshift-eng/aos-cd-jobs#443 to update the bot |
[merge][severity:blocker] |
if that fails we'll just merge manually |
Evaluated for openshift ansible merge up to 2b1c749 |
continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/721/) (Base Commit: 6e2cc5d) (PR Branch Commit: 2b1c749) (Extended Tests: blocker) |
@sdodson Still |
checking with CD team on whether or not something needs to be done to update the jobs |
fix https://bugzilla.redhat.com/show_bug.cgi?id=1469197
When Docker is configured with the
overlay
oroverlay2
storage driver, check that it is supported and usage is below threshold.Also, document and fix
openshift_docker_selinux_enabled
option so that running the check need not result in the docker role reconfiguring and restarting docker with--selinux-enabled
(which will prevent docker from running with these drivers at all).