Skip to content

Commit b835e3d

Browse files
authored
Merge pull request #1987 from rabbitmq/update-dockerfile
Make Docker registry and container tool configurable
2 parents 2c46b1b + f12d20b commit b835e3d

File tree

2 files changed

+12
-7
lines changed

2 files changed

+12
-7
lines changed

Dockerfile

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
ARG GO_TAG=1.24
2+
ARG DOCKER_REGISTRY=docker.io
23
# Build the manager binary
3-
FROM --platform=$BUILDPLATFORM golang:${GO_TAG} AS builder
4+
FROM --platform=$BUILDPLATFORM ${DOCKER_REGISTRY}/library/golang:${GO_TAG} AS builder
45

56
WORKDIR /workspace
67

@@ -29,7 +30,9 @@ ENV GOFIPS140=$FIPS_MODE
2930
RUN CGO_ENABLED=0 GO111MODULE=on go build -a -tags timetzdata -o manager main.go
3031

3132
# ---------------------------------------
32-
FROM alpine:latest AS etc-builder
33+
ARG DOCKER_REGISTRY=docker.io
34+
FROM ${DOCKER_REGISTRY}/library/alpine:latest AS etc-builder
35+
3336

3437
RUN echo "rabbitmq-cluster-operator:x:1000:" > /etc/group && \
3538
echo "rabbitmq-cluster-operator:x:1000:1000::/home/rabbitmq-cluster-operator:/usr/sbin/nologin" > /etc/passwd

Makefile

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -177,11 +177,13 @@ deploy-dev::deploy-namespace-rbac
177177
deploy-dev::docker-registry-secret
178178
deploy-dev::deploy-manager-dev
179179

180+
CONTAINER ?= docker
181+
180182
GIT_COMMIT := $(shell git rev-parse --short HEAD)
181183
deploy-kind: manifests deploy-namespace-rbac ## Load operator image and deploy operator into current KinD cluster
182184
@$(call check_defined, OPERATOR_IMAGE, path to the Operator image within the registry e.g. rabbitmq/cluster-operator)
183185
@$(call check_defined, DOCKER_REGISTRY_SERVER, URL of docker registry containing the Operator image e.g. registry.my-company.com)
184-
docker buildx build --build-arg=GIT_COMMIT=$(GIT_COMMIT) -t $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT) .
186+
$(CONTAINER) buildx build --build-arg=DOCKER_REGISTRY=$(DOCKER_REGISTRY_SERVER) --build-arg=GIT_COMMIT=$(GIT_COMMIT) -t $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT) .
185187
kind load docker-image $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT)
186188
kustomize build config/crd | kubectl apply -f -
187189
kustomize build config/default/overlays/kind | sed 's@((operator_docker_image))@"$(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT)"@' | kubectl apply -f -
@@ -205,18 +207,18 @@ generate-installation-manifest: | $(YTT)
205207
docker-build: ## Build the docker image with tag `latest`
206208
@$(call check_defined, OPERATOR_IMAGE, path to the Operator image within the registry e.g. rabbitmq/cluster-operator)
207209
@$(call check_defined, DOCKER_REGISTRY_SERVER, URL of docker registry containing the Operator image e.g. registry.my-company.com)
208-
docker buildx build --build-arg=GIT_COMMIT=$(GIT_COMMIT) -t $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):latest .
210+
$(CONTAINER) buildx build --build-arg=DOCKER_REGISTRY=$(DOCKER_REGISTRY_SERVER) --build-arg=GIT_COMMIT=$(GIT_COMMIT) -t $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):latest .
209211

210212
docker-push: ## Push the docker image with tag `latest`
211213
@$(call check_defined, OPERATOR_IMAGE, path to the Operator image within the registry e.g. rabbitmq/cluster-operator)
212214
@$(call check_defined, DOCKER_REGISTRY_SERVER, URL of docker registry containing the Operator image e.g. registry.my-company.com)
213-
docker push $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):latest
215+
$(CONTAINER) push $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):latest
214216

215217
docker-build-dev:
216218
@$(call check_defined, OPERATOR_IMAGE, path to the Operator image within the registry e.g. rabbitmq/cluster-operator)
217219
@$(call check_defined, DOCKER_REGISTRY_SERVER, URL of docker registry containing the Operator image e.g. registry.my-company.com)
218-
docker buildx build --build-arg=GIT_COMMIT=$(GIT_COMMIT) -t $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT) .
219-
docker push $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT)
220+
$(CONTAINER) buildx build --build-arg=DOCKER_REGISTRY=$(DOCKER_REGISTRY_SERVER) --build-arg=GIT_COMMIT=$(GIT_COMMIT) -t $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT) .
221+
$(CONTAINER) push $(DOCKER_REGISTRY_SERVER)/$(OPERATOR_IMAGE):$(GIT_COMMIT)
220222

221223
# https://github.com/cert-manager/cmctl/releases
222224
# Cert Manager now publishes CMCTL independently from cert-manager

0 commit comments

Comments
 (0)