-
Notifications
You must be signed in to change notification settings - Fork 33
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
Remove docker from the provision
OperatingSystemConfig
#221
base: master
Are you sure you want to change the base?
Remove docker from the provision
OperatingSystemConfig
#221
Conversation
Co-authored-by: Vladimir Nachev <[email protected]> Co-authored-by: Ismail Alidzhikov <[email protected]>
provision
OperatingSystemConfig
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.
/lgtm
We had a sync with @vpnachev and @Kostov6 to discuss if such change might be a breaking one for end users that make use of the To be on the safe side, we decided to hold with the change until end of February when old gardenlinux version which still contain docker such as [email protected] and [email protected] reach EoL. /hold |
How to categorize this PR?
/area os
/kind cleanup
/os garden-linux
What this PR does / why we need it:
This PR removes executions of docker commands in the init OperatingSystemConfig.
From Gardener side, the
docker
binary is no longer used/required on a Shoot Node: gardener/gardener#4673.From gardeninux OS side, [email protected]+ no longer includes the
docker
binary unit in the OS.Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note: