Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Conversation

@cyphar
Copy link
Contributor

@cyphar cyphar commented Jan 20, 2016

In preparation for removing dockerinit upstream, remove the docker info
fields that relate to dockerinit. This is related to moby/moby#19490.

Signed-off-by: Aleksa Sarai asarai@suse.com

In preparation for removing dockerinit upstream, remove the docker info
fields that relate to dockerinit.

Signed-off-by: Aleksa Sarai <asarai@suse.com>
@cyphar
Copy link
Contributor Author

cyphar commented Jan 20, 2016

@cyphar
Copy link
Contributor Author

cyphar commented Jan 21, 2016

The test cases pass on Docker, so it looks like we're all good to remove this. :P

@calavera
Copy link
Contributor

I'd like to wait until we move moby/moby#19490 out of design review before merging this.

@vdemeester
Copy link
Contributor

@calavera @cyphar agreed 😛. We should add a label status/x-pending or something :trollface: .

@cyphar
Copy link
Contributor Author

cyphar commented Jan 21, 2016

@calavera @vdemeester moby/moby#19490 is out of design review.

@calavera
Copy link
Contributor

LGTM

@vdemeester
Copy link
Contributor

LGTM 🐶

@LK4D4
Copy link
Contributor

LK4D4 commented Jan 22, 2016

LGTM

LK4D4 added a commit that referenced this pull request Jan 22, 2016
@LK4D4 LK4D4 merged commit e2f2ec4 into docker-archive-public:master Jan 22, 2016
@cyphar cyphar deleted the remove-dockerinit branch January 23, 2016 00:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants