Merge pull request #23697 from luxas/pause_multi_arch

Automatic merge from submit-queue Update the pause image to build for arm64 and ppc64le Fixes #22683
kubernetes · Apr 14, 2016 · e363a26 · e363a26 · k8s-teamcity-mesosphere · Apr 14, 2016
2 parents 1965c3c + c214c76
commit e363a26
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 37 deletions.
diff --git a/build/pause/Makefile b/build/pause/Makefile
@@ -14,18 +14,40 @@
 
 .PHONY:	build push
 
-TAG = 2.0
+TAG=2.0
+REGISTRY?="gcr.io/google_containers"
 
-ARCH = amd64
-IMAGE = pause
-# ARCH = arm
-# IMAGE = pause-arm
+# Architectures supported: amd64, arm, arm64 and ppc64le
+ARCH?=amd64
+GOLANG_VERSION=1.6
+TEMP_DIR:=$(shell mktemp -d)
+
+all: push-legacy-too
 
 build:
-	./prepare.sh ${ARCH}
-	docker build -t gcr.io/google_containers/$(IMAGE):$(TAG) .
+	cp pause.go Dockerfile $(TEMP_DIR)
+
+	docker run -it -v $(TEMP_DIR):/build golang:$(GOLANG_VERSION) /bin/bash -c \
+		"cd /build && CGO_ENABLED=0 GOARM=6 GOARCH=$(ARCH) go build -a -installsuffix cgo -ldflags '-w' ./pause.go"
+
+ifeq ($(ARCH),$(filter $(ARCH),amd64 arm))
+# Run goupx for amd64 and arm. The condition above is equal to 'if [[ $ARCH == "amd64" || $ARCH == "arm" ]]' in bash
+	docker run -it -v $(TEMP_DIR):/build golang:$(GOLANG_VERSION) /bin/bash -c \
+		"cd /build && apt-get update && apt-get install -y upx \
+		&& go get github.com/pwaller/goupx && goupx pause"
+endif
+
+	# And build the image
+	docker build -t $(REGISTRY)/pause-$(ARCH):$(TAG) $(TEMP_DIR)
+
+ifeq ($(ARCH),amd64)
+	docker tag -f $(REGISTRY)/pause-$(ARCH):$(TAG) $(REGISTRY)/pause:$(TAG)
+endif
 
-push:	build
-	gcloud docker --server=gcr.io push gcr.io/google_containers/$(IMAGE):$(TAG)
+push: build
+	gcloud docker --server=gcr.io push $(REGISTRY)/pause-$(ARCH):$(TAG)
 
-all:	push
+push-legacy-too: push
+ifeq ($(ARCH),amd64)
+	gcloud docker push $(REGISTRY)/pause:$(TAG)
+endif
diff --git a/build/pause/prepare.sh b/build/pause/prepare.sh