Skip to content
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

Merged

Conversation

dubstack
Copy link

@dubstack dubstack commented Jul 8, 2016

Temporary fix for #28192. This PR updates coreos node e2e image to a version that uses cgroupfs.

cc @vishh @yifan-gu

@dubstack dubstack mentioned this pull request Jul 8, 2016
@k8s-github-robot k8s-github-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. release-note-label-needed labels Jul 8, 2016
@k8s-bot
Copy link

k8s-bot commented Jul 8, 2016

GCE e2e build/test passed for commit 7524b49ea95175b7c8ebea37224484395eeaf7c4.

@dubstack dubstack force-pushed the dubstack-update-coreos branch from 7524b49 to b38eb05 Compare July 8, 2016 17:13
@dubstack dubstack added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-label-needed labels Jul 8, 2016
@dubstack dubstack assigned yifan-gu and unassigned timstclair Jul 8, 2016
if [ $# -ne 4 ]; then
print_usage
exit 1
fi

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.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SG.

@timstclair
Copy link

/cc @euank

@k8s-bot
Copy link

k8s-bot commented Jul 8, 2016

GCE e2e build/test passed for commit b38eb0573bc18a2f4b383bf1fa4db077a6443e4d.

@dubstack dubstack force-pushed the dubstack-update-coreos branch from b38eb05 to fb7f18f Compare July 8, 2016 18:10
@dubstack
Copy link
Author

dubstack commented Jul 8, 2016

@timstclair nit addressed. PTAL.

@k8s-github-robot k8s-github-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 8, 2016
@k8s-bot
Copy link

k8s-bot commented Jul 8, 2016

GCE e2e build/test passed for commit fb7f18f.

@dubstack
Copy link
Author

dubstack commented Jul 8, 2016

@yifan-gu Can you add LGTM?

@vishh vishh added lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/backlog Higher priority than priority/awaiting-more-evidence. labels Jul 8, 2016
@vishh
Copy link
Contributor

vishh commented Jul 8, 2016

Bumping up priority since this PR is blocking other feature PRs.

@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-bot
Copy link

k8s-bot commented Jul 8, 2016

GCE e2e build/test passed for commit fb7f18f.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 66d580b into kubernetes:master Jul 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/backlog Higher priority than priority/awaiting-more-evidence. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants