-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Update coreos node e2e image to a version that uses cgroupfs #28661
Update coreos node e2e image to a version that uses cgroupfs #28661
Conversation
GCE e2e build/test passed for commit 7524b49ea95175b7c8ebea37224484395eeaf7c4. |
7524b49
to
b38eb05
Compare
if [ $# -ne 4 ]; then | ||
print_usage | ||
exit 1 | ||
fi |
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.
nit: can you assign the positional arguments to named variables? I find it much more readable that way
e.g. FROM_IMAGE=$1
, etc.
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.
SG.
/cc @euank |
GCE e2e build/test passed for commit b38eb0573bc18a2f4b383bf1fa4db077a6443e4d. |
b38eb05
to
fb7f18f
Compare
@timstclair nit addressed. PTAL. |
GCE e2e build/test passed for commit fb7f18f. |
@yifan-gu Can you add LGTM? |
Bumping up priority since this PR is blocking other feature PRs. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit fb7f18f. |
Automatic merge from submit-queue |
Temporary fix for #28192. This PR updates coreos node e2e image to a version that uses cgroupfs.
cc @vishh @yifan-gu