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
podman + latest c-storage: thinpool throws a warning #22473
Comments
see containers/storage@137ba2c I guess the default storage.conf in the containers-common rpm has to get updated to remove the section. |
I would humbly suggest that a better solution would be to fix storage.conf first, then wait for it to distribute everywhere that podman can possibly run, and then settle, and then remove thinpool from podman. |
FYI: thinpool was never part of podman (well not for years) |
Sorry, I misspoke due to haste. As a podman user, I am not likely to know or care what thinpool is. If I upgrade podman, maybe on some distro that is not 100% careful about bumping c-common, I will see the above warning and I will waste time worrying about it. Multiply by hundreds or thousands of users. I would like to encourage our team to take steps such that no user ever sees this warning. (Possible exception for someone with a custom |
Rawhide should have it removed now. We could tone down containers/storage verifying its storage.conf file, but that could lead to people adding bad settings and not knowing they have typos in storage.conf files. |
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Fixes: containers/podman#22473 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
With latest c-storage:
The text was updated successfully, but these errors were encountered: