-
Notifications
You must be signed in to change notification settings - Fork 31
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
docs(readme): Add cgroups v2 configuration instructions to README #790
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find this text a bit confusing. Podman and Docker are separate container runtimes, although Podman (and all other OCI tooling) is heavily inspired by Docker. I don't think it should be necessary to have Docker installed alongside Podman, and the Note:
at the bottom is talking about configuring Podman's runc
version but links to Docker documentation?
When I attempted to build/run cryostatio/cryostat-reports#7 my Docker installation complained after enabling |
Sounds good. FWIW I don't have Docker installed at all and haven't in quite some time - probably removed since F30/cgroups v1, if not earlier. |
Makes sense, I only installed it since the OperatorHub.io test suite was refusing to work with Podman. |
Fixes #789