Add GOLANG_VERSION arg support to proxy-docker make target + Dockerfile (#1430)

* Add GOLANG_VERSION arg to proxy-docker

* Set default GOLANG_VERSION to 1.13 in Makefile

* Add note in proxy Dockerfile re: GOLANG_VERSION

* Add GOLANG_VERSION to docker-compose dev target
This commit is contained in:
David Farrell
2019-11-06 11:32:59 -08:00
committed by Aaron Schlesinger
parent ace301e4ec
commit 000d2b92a1
3 changed files with 11 additions and 4 deletions
+3 -2
View File
@@ -2,7 +2,7 @@ VERSION = "unset"
DATE=$(shell date -u +%Y-%m-%d-%H:%M:%S-%Z)
ifndef GOLANG_VERSION
override GOLANG_VERSION = 1.12
override GOLANG_VERSION = 1.13
endif
.PHONY: build
@@ -24,6 +24,7 @@ run: ## run the athens proxy with dev configs
.PHONY: run-docker
run-docker:
docker-compose -p athensdockerdev build --build-arg GOLANG_VERSION=${GOLANG_VERSION} dev
docker-compose -p athensdockerdev up -d dev
.PHONY: run-docker-teardown
@@ -71,7 +72,7 @@ docker: proxy-docker
.PHONY: proxy-docker
proxy-docker:
docker build -t gomods/athens -f cmd/proxy/Dockerfile .
docker build -t gomods/athens -f cmd/proxy/Dockerfile --build-arg GOLANG_VERSION=${GOLANG_VERSION} .
.PHONY: docker-push
docker-push:
+6 -2
View File
@@ -1,8 +1,12 @@
# Links to compare against to ensure we have all VCS's setup in this build
# https://github.com/docker-library/buildpack-deps/blob/1845b3f918f69b4c97912b0d4d68a5658458e84f/stretch/scm/Dockerfile
# https://github.com/golang/go/blob/f082dbfd4f23b0c95ee1de5c2b091dad2ff6d930/src/cmd/go/internal/get/vcs.go#L90
FROM golang:1.13-alpine AS builder
#
# You can override the Go version used to build the image.
# See project Makefile if using make.
# See docker --build-arg if building directly.
ARG GOLANG_VERSION=1.13
FROM golang:${GOLANG_VERSION}-alpine AS builder
WORKDIR $GOPATH/src/github.com/gomods/athens
+2
View File
@@ -4,6 +4,8 @@ services:
build:
context: .
dockerfile: cmd/proxy/Dockerfile
args:
GOLANG_VERSION: 1.13
environment:
- ATHENS_MONGO_STORAGE_URL=mongodb://mongo:27017
- TIMEOUT=20 # in case the mongo dependency takes longer to start up