Skip to content

Commit e47ce9a

Browse files
committed
fix missing variable when building the image on cloudbuild
1 parent 909ccdc commit e47ce9a

File tree

2 files changed

+9
-15
lines changed

2 files changed

+9
-15
lines changed

Dockerfile

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,12 @@
1313
# limitations under the License.
1414

1515
FROM golang:1.13.15 as builder
16+
17+
ARG STAGINGVERSION
18+
1619
WORKDIR /go/src/sigs.k8s.io/gcp-compute-persistent-disk-csi-driver
1720
ADD . .
18-
RUN make gce-pd-driver
21+
RUN GCE_PD_CSI_STAGING_VERSION=${STAGINGVERSION} make gce-pd-driver
1922

2023
# MAD HACKS: Build a version first so we can take the scsi_id bin and put it somewhere else in our real build
2124
FROM k8s.gcr.io/build-image/debian-base-amd64:buster-v1.5.0 as mad-hack

Makefile

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,8 @@
1515
# Args:
1616
# GCE_PD_CSI_STAGING_IMAGE: Staging image repository
1717
REV=$(shell git describe --long --tags --match='v*' --dirty 2>/dev/null || git rev-list -n1 HEAD)
18-
ifdef GCE_PD_CSI_STAGING_VERSION
19-
STAGINGVERSION=${GCE_PD_CSI_STAGING_VERSION}
20-
else
21-
STAGINGVERSION=${REV}
22-
endif
23-
24-
GCFLAGS=""
25-
ifdef GCE_PD_CSI_DEBUG
26-
GCFLAGS="all=-N -l"
27-
endif
28-
18+
GCE_PD_CSI_STAGING_VERSION ?= ${REV}
19+
STAGINGVERSION=${GCE_PD_CSI_STAGING_VERSION}
2920
STAGINGIMAGE=${GCE_PD_CSI_STAGING_IMAGE}
3021
DRIVERBINARY=gce-pd-csi-driver
3122
DRIVERWINDOWSBINARY=${DRIVERBINARY}.exe
@@ -51,7 +42,7 @@ gce-pd-driver-windows:
5142
GOOS=windows go build -mod=vendor -ldflags -X=main.version=$(STAGINGVERSION) -o bin/${DRIVERWINDOWSBINARY} ./cmd/gce-pd-csi-driver/
5243

5344
build-container: require-GCE_PD_CSI_STAGING_IMAGE
54-
$(DOCKER) build --build-arg TAG=$(STAGINGVERSION) -t $(STAGINGIMAGE):$(STAGINGVERSION) .
45+
$(DOCKER) build --build-arg STAGINGVERSION=$(STAGINGVERSION) -t $(STAGINGIMAGE):$(STAGINGVERSION) .
5546

5647
build-and-push-windows-container-ltsc2019: require-GCE_PD_CSI_STAGING_IMAGE init-buildx
5748
$(DOCKER) buildx build --file=Dockerfile.Windows --platform=windows \
@@ -93,12 +84,12 @@ push-container: build-container
9384
build-and-push-container-linux: require-GCE_PD_CSI_STAGING_IMAGE init-buildx
9485
$(DOCKER) buildx build --platform=linux/amd64,linux/arm64 \
9586
-t $(STAGINGIMAGE):$(STAGINGVERSION)_linux \
96-
--build-arg TAG=$(STAGINGVERSION) --push .
87+
--build-arg STAGINGVERSION=$(STAGINGVERSION) --push .
9788

9889
build-and-push-container-linux-debug: require-GCE_PD_CSI_STAGING_IMAGE init-buildx
9990
$(DOCKER) buildx build --file=Dockerfile.debug --platform=linux \
10091
-t $(STAGINGIMAGE):$(STAGINGVERSION)_linux \
101-
--build-arg TAG=$(STAGINGVERSION) --push .
92+
--build-arg STAGINGVERSION=$(STAGINGVERSION) --push .
10293

10394
test-sanity: gce-pd-driver
10495
go test -mod=vendor --v -timeout 30s sigs.k8s.io/gcp-compute-persistent-disk-csi-driver/test/sanity -run ^TestSanity$

0 commit comments

Comments
 (0)