-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Uninstallation of selinux packages in container-engine docker reset.yml #11055
Comments
Uninstalling selinux-policy is not expected, maybe it's bug. |
Oh, the packages which start with "container" could not be matched in |
In my case it matched For enforcing containerd, maybe its should be '+' ? |
@liofko Could you please provide some screenshots to verify the existence of this problem? |
Hi, After changing '^containerd*' to '^containerd+' Its deleted only "containerd.io" |
@liofko Good catch! |
/assign |
While installing, seems like the "Remove docker package" task in "container-engine/docker/tasks/reset.yml" is removing all the packages which start with "container" (e.g. container-selinux, containers-common) and not only the ones starting with "containerd".
This is probably due to the python regex with handle "*" as "Zero or more occurrences" in '^containerd*', so the "d" is not a must.
containerd_package: "{{ ansible_facts.packages.keys() | select('search', '^containerd*') }}"
Is it a bug or the intention was to remove all the "container*" packages (if so, its problematic since removing container-selinux is uninstalling other important packages as selinux-policy) ?
Thanks
The text was updated successfully, but these errors were encountered: