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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add secondary "simple" Dockerfile #11122

Merged
merged 1 commit into from Mar 6, 2015
Merged

Conversation

tianon
Copy link
Member

@tianon tianon commented Mar 2, 2015

This is the absolute bare minimum necessary to compile and test Docker -- this is going to be especially useful for testing and verifying assumptions.

With this, we can setup a Jenkins job that tests to ensure that all the work we do to make sure our build scripts and tests don't contain assumptions is not effort spent in vain.

This is important because this is the kind of bare-bones stock environment our packagers build in. Additionally, this verifies that our scripts will work reasonably on other platforms (such as Darwin and Windows) as well.

Assumptions existing tests make that currently fail:

This is the absolute bare minimum necessary to compile and test Docker -- this is going to be especially useful for testing and verifying assumptions.

With this, we can setup a Jenkins job that tests to ensure that all the work we do to make sure our build scripts and tests don't contain assumptions is not effort spent in vain.

This is important because this is the kind of bare-bones stock environment our packagers build in.  Additionally, this verifies that our scripts will work reasonably on other platforms (such as Darwin and Windows) as well.

Assumptions existing tests make that currently fail:

- `registry-v2` exists as a binary in `$PATH` (FIXED IN moby#11005 馃帀)
- `unprivilegeduser` exists as a user in `/etc/passwd`

Signed-off-by: Andrew "Tianon" Page <admwiggin@gmail.com>
@tianon
Copy link
Member Author

tianon commented Mar 4, 2015

@jfrazelle this one's especially for you 馃槈

@jessfraz
Copy link
Contributor

jessfraz commented Mar 4, 2015

LGTM

1 similar comment
@estesp
Copy link
Contributor

estesp commented Mar 6, 2015

LGTM

estesp added a commit that referenced this pull request Mar 6, 2015
Add secondary "simple" Dockerfile
@estesp estesp merged commit dd37469 into moby:master Mar 6, 2015
@tianon tianon deleted the simple-dockerfile branch March 6, 2015 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants