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

Now using golang:1.6.2 as a base for docker images #206

Merged
merged 1 commit into from
May 23, 2016

Conversation

mvallerie
Copy link
Contributor

Hi there.

As stated in #200 , Go 1.6 may fix some issues. This PR changes the base image in Dockerfiles to switch to golang:1.6.2.

Please note that golang image is based on jessie-scm, so your existing Dockerfiles were mainly compatible with it. GOPATH is /go on golang image, so i had to change this, and i also removed some (apparently ?) unused ppa repositories which would have not been compatible with jessie-scm anyway.

Disclaimer : i tested Dockerfile on our production cluster, works great. But i wasn't able to test Dockerfile-deb. Only a few minor changes, so i assume it won't break anything, but i would feel it safer if someone who uses it everyday could confirm that.

Thanks in adv. !

@swagatha-christie
Copy link

By analyzing the blame information on this pull request, we identified @bateau84, @sttts and @rthomas to be potential reviewers

@coveralls
Copy link

Coverage Status

Coverage increased (+3.3%) to 39.154% when pulling 23f2741 on mvallerie:master into 01da91b on QubitProducts:master.

@j1n6
Copy link
Contributor

j1n6 commented May 16, 2016

LGTM.

Sent from my iPhone

On 13 May 2016, at 04:33, Coveralls notifications@github.com wrote:

Coverage increased (+3.3%) to 39.154% when pulling 23f2741 on mvallerie:master into 01da91b on QubitProducts:master.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

@j1n6 j1n6 closed this May 23, 2016
@j1n6 j1n6 reopened this May 23, 2016
@j1n6 j1n6 merged commit cbac0cd into QubitProducts:master May 23, 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.

None yet

4 participants