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 Dockerfile.minimal to use alpine 3.4 (and go 1.6) #669

Merged
merged 1 commit into from
Oct 3, 2016

Conversation

aboyett
Copy link
Contributor

@aboyett aboyett commented Sep 22, 2016

The alpine-based minimal docker image has been broken since 0d7ae8e.
That commit removed Godep-based dependency management favor of go
1.5/1.6 vendoring. This commit moves the base alpine image forward to
3.4 which includes go 1.6.3. The resulting image of 0d7ae8e^ grows from
66 MB to 73 MB.

The alpine-based minimal docker image has been broken since 0d7ae8e.
That commit removed Godep-based dependency management favor of go
1.5/1.6 vendoring. This commit moves the base alpine image forward to
3.4 which includes go 1.6.3. The resulting image of 0d7ae8e^ grows from
66 MB to 73 MB.
@aboyett aboyett changed the title update Docker.minimal to use alpine 3.4 (and go 1.6) update Dockerfile.minimal to use alpine 3.4 (and go 1.6) Sep 22, 2016
@aboyett
Copy link
Contributor Author

aboyett commented Sep 22, 2016

whoops, slight typo in the commit and PR title. updated!

@codecov-io
Copy link

codecov-io commented Sep 22, 2016

Current coverage is 58.36% (diff: 100%)

Merging #669 into master will not change coverage

@@             master       #669   diff @@
==========================================
  Files            74         74          
  Lines          6420       6420          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           3747       3747          
  Misses         2313       2313          
  Partials        360        360          

Powered by Codecov. Last update 45eb038...0674cc7

@0xhaven
Copy link
Contributor

0xhaven commented Sep 30, 2016

This would explain why the automated Dockerfile.minimal builds have been failing. 👍 Though, I'm a bit disappointed that alpine:latest still only has Go 1.6.3

@aboyett
Copy link
Contributor Author

aboyett commented Sep 30, 2016

@jacobhaven Alpine Linux has Go 1.7.1 in their development branch (edge.) https://pkgs.alpinelinux.org/packages?name=go

@kisom kisom merged commit 28c9ad5 into cloudflare:master Oct 3, 2016
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.

5 participants