Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

Commit e147b98

Browse files
authored
Merge pull request #97 from christopherhein/bug/control-plane-manifests
🐛 control plane manifests weren't being pushed
2 parents ea129c6 + 7296d9f commit e147b98

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

Makefile

+10
Original file line numberDiff line numberDiff line change
@@ -266,6 +266,7 @@ docker-controlplane-build-%:
266266
.PHONY: docker-push-all ## Push all the architecture docker images
267267
docker-push-all: $(addprefix docker-infrastructure-push-,$(ALL_ARCH)) $(addprefix docker-controlplane-push-,$(ALL_ARCH))
268268
$(MAKE) docker-push-core-manifest
269+
$(MAKE) docker-push-nested-control-plane-manifest
269270

270271
.PHONY: docker-push
271272
docker-push:
@@ -289,6 +290,15 @@ docker-push-core-manifest: ## Push the fat manifest docker image for the core im
289290
$(MAKE) set-manifest-image MANIFEST_IMG=$(CONTROLLER_IMG) MANIFEST_TAG=$(TAG) TARGET_RESOURCE="./config/default/manager_image_patch.yaml"
290291
$(MAKE) set-manifest-pull-policy TARGET_RESOURCE="./config/default/manager_pull_policy.yaml"
291292

293+
.PHONY: docker-push-nested-control-plane-manifest
294+
docker-push-nested-control-plane-manifest: ## Push the fat manifest docker image for the nested control plane image.
295+
## Minimum docker version 18.06.0 is required for creating and pushing manifest images.
296+
docker manifest create --amend $(CONTROLPLANE_CONTROLLER_IMG):$(TAG) $(shell echo $(ALL_ARCH) | sed -e "s~[^ ]*~$(CONTROLPLANE_CONTROLLER_IMG)\-&:$(TAG)~g")
297+
@for arch in $(ALL_ARCH); do docker manifest annotate --arch $${arch} ${CONTROLPLANE_CONTROLLER_IMG}:${TAG} ${CONTROLPLANE_CONTROLLER_IMG}-$${arch}:${TAG}; done
298+
docker manifest push --purge $(CONTROLPLANE_CONTROLLER_IMG):$(TAG)
299+
$(MAKE) set-manifest-image MANIFEST_IMG=$(CONTROLPLANE_CONTROLLER_IMG) MANIFEST_TAG=$(TAG) TARGET_RESOURCE="./controlplane/nested/config/default/manager_image_patch.yaml"
300+
$(MAKE) set-manifest-pull-policy TARGET_RESOURCE="./controlplane/nested/config/default/manager_pull_policy.yaml"
301+
292302
.PHONY: set-manifest-pull-policy
293303
set-manifest-pull-policy:
294304
$(info Updating kustomize pull policy file for manager resources)

0 commit comments

Comments
 (0)