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

WIP: Attempt to fix travis builds #38336

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
4 participants
@alexcrichton
Copy link
Member

alexcrichton commented Dec 13, 2016

This is just a shot in the dark, but after reading https://blog.phusion.nl/2015/01/20/docker-and-the-pid-1-zombie-reaping-problem/ and https://engineeringblog.yelp.com/2016/01/dumb-init-an-init-for-docker.html today I got paranoid and started wondering if this is what's causing problems.

Let's see if this fixes anything by taking Yelp's "dumb-init" init program and see if it fixes the issues we're seeing. If this works I'll resubmit with the PR, updating all other images, and with a better description.

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Dec 13, 2016

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton alexcrichton changed the title Attempt to fix travis builds WIP: Attempt to fix travis builds Dec 13, 2016

@alexcrichton alexcrichton assigned alexcrichton and unassigned brson Dec 13, 2016

@sanxiyn

This comment has been minimized.

Copy link
Member

sanxiyn commented Dec 13, 2016

It seems it worked!

@alexcrichton

This comment has been minimized.

Copy link
Member Author

alexcrichton commented Dec 13, 2016

Well who'da thunk. Will reopen with a better description and such.

@alexcrichton alexcrichton deleted the alexcrichton:fix-travis branch Dec 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.