Merge pull request #215 from JimmyYang20/fixbug

Fix building wrong GM/LC arm64 docker images
kubeedge · Nov 2, 2021 · 1ce64fc · 1ce64fc
2 parents a760f17 + e95caf9
commit 1ce64fc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -142,7 +142,7 @@ gmimage lcimage kbimage:
 	docker build --build-arg GO_LDFLAGS=${GO_LDFLAGS} -t ${IMAGE_REPO}/sedna-${@:image=}:${IMAGE_TAG} -f build/${@:image=}/Dockerfile .
 
 
-.PHONY: push push-examples push-all push-multi-platforms
+.PHONY: push push-examples push-all push-multi-platform-images
 push-all: push-multi-platform-images push-examples
 
 # push target pushes sedna-built images
@@ -154,6 +154,7 @@ push: images
 
 push-examples:
 	bash examples/push_image.sh
+
 # push multi-platform images
 push-multi-platform-images:
 	bash hack/make-rules/push.sh

diff --git a/hack/lib/buildx.sh b/hack/lib/buildx.sh
@@ -38,7 +38,7 @@ sedna::buildx::prepare_env() {
 
 sedna::buildx:generate-dockerfile() {
   dockerfile=${1}
-  sed "/AS builder/s/FROM/FROM --platform=\$BUILDPLATFORM/g" ${dockerfile}
+  sed "/AS builder/s/FROM/FROM --platform=\$TARGETPLATFORM/g" ${dockerfile}
 }
 
 sedna::buildx::push-multi-platform-images() {