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
GCE Windows: add 20H2; install docker when nodes are started #98284
GCE Windows: add 20H2; install docker when nodes are started #98284
Conversation
/cc @jeremyje |
6e119f9
to
dd26a20
Compare
cluster/gce/util.sh
Outdated
elif [[ "${WINDOWS_NODE_OS_DISTRIBUTION}" == "win1809" ]]; then | ||
WINDOWS_NODE_IMAGE="windows-server-1809-dc-core-for-containers-v20200908" | ||
WINDOWS_NODE_IMAGE="windows-server-1909-dc-core-v20210112" | ||
elif [[ "${WINDOWS_NODE_OS_DISTRIBUTION}" == "win20h2" || "${WINDOWS_NODE_OS_DISTRIBUTION}" == "win20H2" ]]; then |
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.
Use elif [[ "${WINDOWS_NODE_OS_DISTRIBUTION,,}" == "win20h2" ]]; then
https://stackoverflow.com/questions/2264428/how-to-convert-a-string-to-lower-case-in-bash
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.
Actually probably should coerce lowercase to all options here.
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.
Actually probably should coerce lowercase to all options here
I don't understand, can you elaborate?
"${WINDOWS_NODE_OS_DISTRIBUTION,,}"
I find this to be much less readable, but don't really mind either way.
8d4cdfb
to
e45580b
Compare
e45580b
to
21592c2
Compare
Output from startup script: https://gist.github.com/pjh/95bb5571a6928738086832115212958b |
/lgtm |
Looking at the startup script output from a single 2004 node: installing the Containers feature and rebooting adds just under 2 minutes to the startup time. Installing Docker then takes about 1 minute. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jeremyje, pjh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Adds support on GCE for the latest Windows SAC version, 20H2. Removes 1809 which is now obsolete. Uses the latest version of all Windows VM images (January 2021 Windows Updates).
Updates the GCE Windows startup scripts to check for and install the Windows Containers feature and docker, since these dependencies are no longer installed on Windows SAC images.
Tested by running
kube-up.sh
withWINDOWS_NODE_OS_DISTRIBUTION=win1909
andWINDOWS_NODE_OS_DISTRIBUTION=win20h2
./kind cleanup