Skip to content

Commit 88dc60e

Browse files
authored
Merge pull request #6443 from chrischdi/pr-makefile-refactoring
🌱 consolidate test/infrastructure/docker/Makefile to Makefile
2 parents bd4fa87 + d4170d7 commit 88dc60e

File tree

13 files changed

+135
-427
lines changed

13 files changed

+135
-427
lines changed

Makefile

Lines changed: 118 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,10 @@ KUBEADM_BOOTSTRAP_CONTROLLER_IMG ?= $(REGISTRY)/$(KUBEADM_BOOTSTRAP_IMAGE_NAME)
153153
KUBEADM_CONTROL_PLANE_IMAGE_NAME ?= kubeadm-control-plane-controller
154154
KUBEADM_CONTROL_PLANE_CONTROLLER_IMG ?= $(REGISTRY)/$(KUBEADM_CONTROL_PLANE_IMAGE_NAME)
155155

156+
# capd
157+
CAPD_IMAGE_NAME ?= capd-manager
158+
CAPD_CONTROLLER_IMG ?= $(REGISTRY)/$(CAPD_IMAGE_NAME)
159+
156160
# clusterctl
157161
CLUSTERCTL_MANIFEST_DIR := cmd/clusterctl/config
158162
CLUSTERCTL_IMAGE_NAME ?= clusterctl
@@ -188,12 +192,11 @@ help: # Display this help
188192

189193
##@ generate:
190194

191-
ALL_GENERATE_MODULES = core kubeadm-bootstrap kubeadm-control-plane
195+
ALL_GENERATE_MODULES = core kubeadm-bootstrap kubeadm-control-plane capd
192196

193197
.PHONY: generate
194198
generate: ## Run all generate-manifests-*, generate-go-deepcopy-*, generate-go-conversions-* and generate-go-openapi targets
195199
$(MAKE) generate-modules generate-manifests generate-go-deepcopy generate-go-conversions generate-go-openapi
196-
$(MAKE) -C $(CAPD_DIR) generate
197200

198201
.PHONY: generate-manifests
199202
generate-manifests: $(addprefix generate-manifests-,$(ALL_GENERATE_MODULES)) ## Run all generate-manifests-* targets
@@ -246,6 +249,19 @@ generate-manifests-kubeadm-control-plane: $(CONTROLLER_GEN) ## Generate manifest
246249
output:webhook:dir=./controlplane/kubeadm/config/webhook \
247250
webhook
248251

252+
.PHONY: generate-manifests-capd
253+
generate-manifests-capd: $(CONTROLLER_GEN) ## Generate manifests e.g. CRD, RBAC etc. for capd provider
254+
cd $(CAPD_DIR); $(CONTROLLER_GEN) \
255+
paths=./api/... \
256+
paths=./$(EXP_DIR)/api/... \
257+
paths=./$(EXP_DIR)/internal/controllers/... \
258+
paths=./internal/controllers/... \
259+
crd:crdVersions=v1 \
260+
rbac:roleName=manager-role \
261+
output:crd:dir=./config/crd/bases \
262+
output:webhook:dir=./config/webhook \
263+
webhook
264+
249265
.PHONY: generate-go-deepcopy
250266
generate-go-deepcopy: ## Run all generate-go-deepcopy-* targets
251267
$(MAKE) $(addprefix generate-go-deepcopy-,$(ALL_GENERATE_MODULES))
@@ -276,6 +292,13 @@ generate-go-deepcopy-kubeadm-control-plane: $(CONTROLLER_GEN) ## Generate deepco
276292
object:headerFile=./hack/boilerplate/boilerplate.generatego.txt \
277293
paths=./controlplane/kubeadm/api/...
278294

295+
.PHONY: generate-go-deepcopy-capd
296+
generate-go-deepcopy-capd: $(CONTROLLER_GEN) ## Generate deepcopy go code for capd
297+
cd $(CAPD_DIR); $(CONTROLLER_GEN) \
298+
object:headerFile=../../../hack/boilerplate/boilerplate.generatego.txt \
299+
paths=./api/... \
300+
paths=./$(EXP_DIR)/api/...
301+
279302
.PHONY: generate-go-conversions
280303
generate-go-conversions: ## Run all generate-go-conversions-* targets
281304
$(MAKE) $(addprefix generate-go-conversions-,$(ALL_GENERATE_MODULES))
@@ -353,6 +376,19 @@ generate-go-conversions-kubeadm-control-plane: $(CONVERSION_GEN) ## Generate con
353376
--output-file-base=zz_generated.conversion $(CONVERSION_GEN_OUTPUT_BASE) \
354377
--go-header-file=./hack/boilerplate/boilerplate.generatego.txt
355378

379+
.PHONY: generate-go-conversions-capd
380+
generate-go-conversions-capd: $(CONVERSION_GEN) ## Generate conversions go code for capd
381+
cd $(CAPD_DIR); $(CONVERSION_GEN) \
382+
--input-dirs=./api/v1alpha3 \
383+
--input-dirs=./api/v1alpha4 \
384+
--input-dirs=./$(EXP_DIR)/api/v1alpha3 \
385+
--input-dirs=./$(EXP_DIR)/api/v1alpha4 \
386+
--build-tag=ignore_autogenerated_capd \
387+
--extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1alpha3 \
388+
--extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1alpha4 \
389+
--output-file-base=zz_generated.conversion $(CONVERSION_GEN_OUTPUT_BASE) \
390+
--go-header-file=../../../hack/boilerplate/boilerplate.generatego.txt
391+
356392
.PHONY: generate-go-openapi
357393
generate-go-openapi: $(OPENAPI_GEN) $(CONTROLLER_GEN) ## Generate openapi go code for runtime SDK
358394
@for pkg in "api/v1beta1" "$(EXP_DIR)/runtime/hooks/api/v1alpha1"; do \
@@ -401,7 +437,7 @@ APIDIFF_OLD_COMMIT ?= $(shell git rev-parse origin/main)
401437
apidiff: $(GO_APIDIFF) ## Check for API differences
402438
$(GO_APIDIFF) $(APIDIFF_OLD_COMMIT) --print-compatible
403439

404-
ALL_VERIFY_CHECKS = doctoc boilerplate shellcheck tiltfile modules gen conversions docker-provider
440+
ALL_VERIFY_CHECKS = doctoc boilerplate shellcheck tiltfile modules gen conversions
405441

406442
.PHONY: verify
407443
verify: $(addprefix verify-,$(ALL_VERIFY_CHECKS)) ## Run all verify-* targets
@@ -444,11 +480,6 @@ verify-shellcheck: ## Verify shell files
444480
verify-tiltfile: ## Verify Tiltfile format
445481
./hack/verify-starlark.sh
446482

447-
.PHONY: verify-docker-provider
448-
verify-docker-provider:
449-
@echo "Verifying CAPD"
450-
cd $(CAPD_DIR); $(MAKE) verify
451-
452483
## --------------------------------------
453484
## Binaries
454485
## --------------------------------------
@@ -459,7 +490,7 @@ verify-docker-provider:
459490
clusterctl: ## Build the clusterctl binary
460491
go build -trimpath -ldflags "$(LDFLAGS)" -o $(BIN_DIR)/clusterctl sigs.k8s.io/cluster-api/cmd/clusterctl
461492

462-
ALL_MANAGERS = core kubeadm-bootstrap kubeadm-control-plane
493+
ALL_MANAGERS = core kubeadm-bootstrap kubeadm-control-plane capd
463494

464495
.PHONY: managers
465496
managers: $(addprefix manager-,$(ALL_MANAGERS)) ## Run all manager-* targets
@@ -476,6 +507,10 @@ manager-kubeadm-bootstrap: ## Build the kubeadm bootstrap manager binary into th
476507
manager-kubeadm-control-plane: ## Build the kubeadm control plane manager binary into the ./bin folder
477508
go build -trimpath -ldflags "$(LDFLAGS)" -o $(BIN_DIR)/kubeadm-control-plane-manager sigs.k8s.io/cluster-api/controlplane/kubeadm
478509

510+
.PHONY: manager-capd
511+
manager-capd: ## Build the kubeadm control plane manager binary into the ./bin folder
512+
cd $(CAPD_DIR); go build -trimpath -ldflags "$(LDFLAGS)" -o ../../../$(BIN_DIR)/capd-manager sigs.k8s.io/cluster-api/test/infrastructure/docker
513+
479514
.PHONY: docker-pull-prerequisites
480515
docker-pull-prerequisites:
481516
docker pull docker.io/docker/dockerfile:1.1-experimental
@@ -516,6 +551,9 @@ docker-build-kubeadm-control-plane: ## Build the docker image for kubeadm contro
516551
docker-build-clusterctl: ## Build the docker image for clusterctl with output binary name as 'clusterctl'
517552
DOCKER_BUILDKIT=1 docker build --build-arg builder_image=$(GO_CONTAINER_IMAGE) --build-arg goproxy=$(GOPROXY) --build-arg ARCH=$(ARCH) --build-arg package=./cmd/clusterctl --build-arg ldflags="$(LDFLAGS)" -f ./cmd/clusterctl/Dockerfile . -t $(CLUSTERCTL_IMG)-$(ARCH):$(TAG)
518553

554+
.PHONY: docker-capd-build-all
555+
docker-capd-build-all: $(addprefix docker-capd-build-,$(ALL_ARCH)) ## Build capd docker images for all architectures
556+
519557
.PHONY: e2e-framework
520558
e2e-framework: ## Builds the CAPI e2e framework
521559
cd $(E2E_FRAMEWORK_DIR); go build ./...
@@ -528,6 +566,15 @@ build-book: ## Build the book
528566
serve-book: ## Build and serve the book (with live-reload)
529567
$(MAKE) -C docs/book serve
530568

569+
docker-capd-build-%:
570+
$(MAKE) ARCH=$* docker-capd-build
571+
572+
.PHONY: docker-capd-build
573+
docker-capd-build: ## Build the docker image for capd
574+
cd $(CAPD_DIR); DOCKER_BUILDKIT=1 docker build --build-arg builder_image=$(GO_CONTAINER_IMAGE) --build-arg goproxy=$(GOPROXY) --build-arg ARCH=$(ARCH) --build-arg ldflags="$(LDFLAGS)" ../../.. -t $(CAPD_CONTROLLER_IMG)-$(ARCH):$(TAG) --file Dockerfile
575+
$(MAKE) set-manifest-image MANIFEST_IMG=$(CAPD_CONTROLLER_IMG)-$(ARCH) MANIFEST_TAG=$(TAG) TARGET_RESOURCE="$(CAPD_DIR)/config/default/manager_image_patch.yaml"
576+
$(MAKE) set-manifest-pull-policy TARGET_RESOURCE="$(CAPD_DIR)/config/default/manager_pull_policy.yaml"
577+
531578
## --------------------------------------
532579
## Testing
533580
## --------------------------------------
@@ -566,14 +613,28 @@ test-cover: ## Run unit and integration tests and generate a coverage report
566613
test-e2e: ## Run e2e tests
567614
$(MAKE) -C $(TEST_DIR)/e2e run
568615

616+
.PHONY: test-capd
617+
test-capd: $(SETUP_ENVTEST) ## Run unit and integration tests for capd
618+
cd $(CAPD_DIR); KUBEBUILDER_ASSETS="$(KUBEBUILDER_ASSETS)" go test ./... $(TEST_ARGS)
619+
620+
.PHONY: test-capd-verbose
621+
test-capd-verbose: ## Run unit and integration tests with verbose flag
622+
$(MAKE) test-capd TEST_ARGS="$(TEST_ARGS) -v"
623+
624+
.PHONY: test-capd-junit
625+
test-capd-junit: $(SETUP_ENVTEST) $(GOTESTSUM) ## Run unit and integration tests and generate a junit report for capd
626+
cd $(CAPD_DIR); set +o errexit; (KUBEBUILDER_ASSETS="$(KUBEBUILDER_ASSETS)" go test -json ./... $(TEST_ARGS); echo $$? > $(ARTIFACTS)/junit.infra_docker.exitcode) | tee $(ARTIFACTS)/junit.infra_docker.stdout
627+
$(GOTESTSUM) --junitfile $(ARTIFACTS)/junit.infra_docker.xml --raw-command cat $(ARTIFACTS)/junit.infra_docker.stdout
628+
exit $$(cat $(ARTIFACTS)/junit.infra_docker.exitcode)
629+
569630
.PHONY: kind-cluster
570631
kind-cluster: ## Create a new kind cluster designed for development with Tilt
571632
hack/kind-install-for-capd.sh
572633

573634
.PHONY: docker-build-e2e
574635
docker-build-e2e: ## Rebuild all Cluster API provider images to be used in the e2e tests
575-
make docker-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPresent
576-
$(MAKE) -C $(CAPD_DIR) docker-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPresent
636+
$(MAKE) docker-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPresent
637+
$(MAKE) docker-capd-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPresent
577638

578639
## --------------------------------------
579640
## Release
@@ -626,9 +687,13 @@ manifest-modification: # Set the manifest images to the staging/production bucke
626687
$(MAKE) set-manifest-image \
627688
MANIFEST_IMG=$(REGISTRY)/$(KUBEADM_CONTROL_PLANE_IMAGE_NAME) MANIFEST_TAG=$(RELEASE_TAG) \
628689
TARGET_RESOURCE="./controlplane/kubeadm/config/default/manager_image_patch.yaml"
690+
$(MAKE) set-manifest-image \
691+
MANIFEST_IMG=$(REGISTRY)/$(CAPD_IMAGE_NAME) MANIFEST_TAG=$(RELEASE_TAG) \
692+
TARGET_RESOURCE="$(CAPD_DIR)/config/default/manager_image_patch.yaml"
629693
$(MAKE) set-manifest-pull-policy PULL_POLICY=IfNotPresent TARGET_RESOURCE="./config/default/manager_pull_policy.yaml"
630694
$(MAKE) set-manifest-pull-policy PULL_POLICY=IfNotPresent TARGET_RESOURCE="./bootstrap/kubeadm/config/default/manager_pull_policy.yaml"
631695
$(MAKE) set-manifest-pull-policy PULL_POLICY=IfNotPresent TARGET_RESOURCE="./controlplane/kubeadm/config/default/manager_pull_policy.yaml"
696+
$(MAKE) set-manifest-pull-policy PULL_POLICY=IfNotPresent TARGET_RESOURCE="$(CAPD_DIR)/config/default/manager_pull_policy.yaml"
632697

633698
.PHONY: release-manifests
634699
release-manifests: $(RELEASE_DIR) $(KUSTOMIZE) $(RUNTIME_OPENAPI_GEN) ## Build the manifests to publish with a release
@@ -652,11 +717,8 @@ release-manifests: $(RELEASE_DIR) $(KUSTOMIZE) $(RUNTIME_OPENAPI_GEN) ## Build t
652717
$(RUNTIME_OPENAPI_GEN) --version $(RELEASE_TAG) --output-file $(RELEASE_DIR)/runtime-sdk-openapi.yaml
653718

654719
.PHONY: release-manifests-dev
655-
release-manifests-dev: ## Build the development manifests and copies them in the release folder
656-
# Release CAPD components and add them to the release dir
657-
$(MAKE) -C $(CAPD_DIR) release
658-
cp $(CAPD_DIR)/out/infrastructure-components.yaml $(RELEASE_DIR)/infrastructure-components-development.yaml
659-
# Adds CAPD templates
720+
release-manifests-dev: $(RELEASE_DIR) $(KUSTOMIZE) ## Build the development manifests and copies them in the release folder
721+
cd $(CAPD_DIR); $(KUSTOMIZE) build config/default > ../../../$(RELEASE_DIR)/infrastructure-components-development.yaml
660722
cp $(CAPD_DIR)/templates/* $(RELEASE_DIR)/
661723

662724
release-binaries: ## Build the binaries to publish with a release
@@ -701,6 +763,26 @@ release-alias-tag: ## Add the release alias tag to the last build tag
701763
gcloud container images add-tag $(KUBEADM_CONTROL_PLANE_CONTROLLER_IMG):$(TAG) $(KUBEADM_CONTROL_PLANE_CONTROLLER_IMG):$(RELEASE_ALIAS_TAG)
702764
gcloud container images add-tag $(CLUSTERCTL_IMG):$(TAG) $(CLUSTERCTL_IMG):$(RELEASE_ALIAS_TAG)
703765

766+
.PHONY: release-capd-staging
767+
release-capd-staging: ## Build and push container images to the staging bucket
768+
REGISTRY=$(STAGING_REGISTRY) $(MAKE) docker-capd-build-all docker-capd-push-all release-capd-alias-tag
769+
770+
.PHONY: release-capd-staging-nightly
771+
release-capd-staging-nightly: ## Tag and push container images to the staging bucket. Example image tag: cluster-api-controller:nightly_main_20210121
772+
$(eval NEW_RELEASE_ALIAS_TAG := nightly_$(RELEASE_ALIAS_TAG)_$(shell date +'%Y%m%d'))
773+
echo $(NEW_RELEASE_ALIAS_TAG)
774+
$(MAKE) release-capd-alias-tag TAG=$(RELEASE_ALIAS_TAG) RELEASE_ALIAS_TAG=$(NEW_RELEASE_ALIAS_TAG)
775+
# Set the manifest image to the staging bucket.
776+
$(MAKE) manifest-modification REGISTRY=$(STAGING_REGISTRY) RELEASE_TAG=$(NEW_RELEASE_ALIAS_TAG)
777+
## Build the manifests
778+
$(MAKE) release-manifests-dev
779+
# Example manifest location: artifacts.k8s-staging-cluster-api.appspot.com/components/nightly_main_20210121/infrastructure-components.yaml
780+
gsutil cp $(RELEASE_DIR)/* gs://$(STAGING_BUCKET)/components/$(NEW_RELEASE_ALIAS_TAG)
781+
782+
.PHONY: release-capd-alias-tag
783+
release-capd-alias-tag: ## Add the release alias tag to the last build tag
784+
gcloud container images add-tag $(CAPD_CONTROLLER_IMG):$(TAG) $(CAPD_CONTROLLER_IMG):$(RELEASE_ALIAS_TAG)
785+
704786
.PHONY: release-notes
705787
release-notes: $(RELEASE_NOTES_DIR) $(RELEASE_NOTES)
706788
if [ -n "${PRE_RELEASE}" ]; then \
@@ -734,6 +816,17 @@ docker-push-all: $(addprefix docker-push-,$(ALL_ARCH)) ## Push all the architec
734816
docker-push-%:
735817
$(MAKE) ARCH=$* docker-push
736818

819+
.PHONY: docker-capd-push
820+
docker-capd-push: ## Push the capd docker image
821+
docker push $(CAPD_CONTROLLER_IMG)-$(ARCH):$(TAG)
822+
823+
.PHONY: docker-capd-push-all
824+
docker-capd-push-all: $(addprefix docker-capd-push-,$(ALL_ARCH)) ## Push all the architecture docker images
825+
$(MAKE) docker-capd-push-manifest
826+
827+
docker-capd-push-%:
828+
$(MAKE) ARCH=$* docker-capd-push
829+
737830
.PHONY: docker-push-manifest-core
738831
docker-push-manifest-core: ## Push the multiarch manifest for the core docker images
739832
## Minimum docker version 18.06.0 is required for creating and pushing manifest images.
@@ -761,6 +854,15 @@ docker-push-manifest-kubeadm-control-plane: ## Push the multiarch manifest for t
761854
$(MAKE) set-manifest-image MANIFEST_IMG=$(KUBEADM_CONTROL_PLANE_CONTROLLER_IMG) MANIFEST_TAG=$(TAG) TARGET_RESOURCE="./controlplane/kubeadm/config/default/manager_image_patch.yaml"
762855
$(MAKE) set-manifest-pull-policy TARGET_RESOURCE="./controlplane/kubeadm/config/default/manager_pull_policy.yaml"
763856

857+
.PHONY: docker-push-manifest-capd
858+
docker-capd-push-manifest: ## Push the multiarch manifest for the the capd docker images
859+
## Minimum docker version 18.06.0 is required for creating and pushing manifest images.
860+
docker manifest create --amend $(CAPD_CONTROLLER_IMG):$(TAG) $(shell echo $(ALL_ARCH) | sed -e "s~[^ ]*~$(CAPD_CONTROLLER_IMG)\-&:$(TAG)~g")
861+
@for arch in $(ALL_ARCH); do docker manifest annotate --arch $${arch} ${CAPD_CONTROLLER_IMG}:${TAG} ${CAPD_CONTROLLER_IMG}-$${arch}:${TAG}; done
862+
docker manifest push --purge $(CAPD_CONTROLLER_IMG):$(TAG)
863+
$(MAKE) set-manifest-image MANIFEST_IMG=$(CAPD_CONTROLLER_IMG) MANIFEST_TAG=$(TAG) TARGET_RESOURCE="$(CAPD_DIR)/config/default/manager_image_patch.yaml"
864+
$(MAKE) set-manifest-pull-policy TARGET_RESOURCE="$(CAPD_DIR)/config/default/manager_pull_policy.yaml"
865+
764866
.PHONY: docker-push-clusterctl
765867
docker-push-clusterctl: ## Push the clusterctl images
766868
## Minimum docker version 18.06.0 is required for creating and pushing manifest images.

cloudbuild-nightly.yaml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,14 @@ steps:
1414
args:
1515
- release-staging-nightly
1616
- name: 'gcr.io/k8s-staging-test-infra/gcb-docker-gcloud:v20220609-2e4c91eb7e'
17-
dir: 'test/infrastructure/docker'
18-
entrypoint: make
17+
=======
1918
env:
2019
- DOCKER_CLI_EXPERIMENTAL=enabled
2120
- TAG=$_GIT_TAG
2221
- PULL_BASE_REF=$_PULL_BASE_REF
2322
- DOCKER_BUILDKIT=1
2423
args:
25-
- release-staging-nightly
24+
- release-capd-staging-nightly
2625
substitutions:
2726
# _GIT_TAG will be filled with a git-based tag for the image, of the form vYYYYMMDD-hash, and
2827
# can be used as a substitution

cloudbuild.yaml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,14 @@ steps:
1414
args:
1515
- release-staging
1616
- name: 'gcr.io/k8s-staging-test-infra/gcb-docker-gcloud:v20220609-2e4c91eb7e'
17-
dir: 'test/infrastructure/docker'
18-
entrypoint: make
17+
=======
1918
env:
2019
- DOCKER_CLI_EXPERIMENTAL=enabled
2120
- TAG=$_GIT_TAG
2221
- PULL_BASE_REF=$_PULL_BASE_REF
2322
- DOCKER_BUILDKIT=1
2423
args:
25-
- release-staging
24+
- release-capd-staging
2625
substitutions:
2726
# _GIT_TAG will be filled with a git-based tag for the image, of the form vYYYYMMDD-hash, and
2827
# can be used as a substitution

docs/book/src/clusterctl/developers.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ make docker-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPrese
4242
In order to build docker provider artifacts
4343

4444
```
45-
make -C test/infrastructure/docker docker-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPresent
45+
make docker-capd-build REGISTRY=gcr.io/k8s-staging-cluster-api PULL_POLICY=IfNotPresent
4646
```
4747

4848
### Create a clusterctl-settings.json file

docs/book/src/developer/guide.md

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ You'll need to build two docker images, one for Cluster API itself and one for t
113113

114114
```
115115
make docker-build
116-
make -C test/infrastructure/docker docker-build
116+
make docker-capd-build
117117
```
118118

119119
### Push both images
@@ -125,19 +125,17 @@ The push refers to repository [gcr.io/cluster-api-242700/cluster-api-controller-
125125
90a39583ad5f: Layer already exists
126126
932da5156413: Layer already exists
127127
dev: digest: sha256:263262cfbabd3d1add68172a5a1d141f6481a2bc443672ce80778dc122ee6234 size: 739
128-
$ make -C test/infrastructure/docker docker-push
129-
make: Entering directory '/home/liz/src/sigs.k8s.io/cluster-api/test/infrastructure/docker'
130-
docker push gcr.io/cluster-api-242700/manager:dev
131-
The push refers to repository [gcr.io/cluster-api-242700/manager]
128+
$ make docker-capd-push
129+
docker push gcr.io/cluster-api-242700/capd-manager-amd64:dev
130+
The push refers to repository [gcr.io/cluster-api-242700/capd-manager-amd64]
132131
5b1e744b2bae: Pushed
133132
932da5156413: Layer already exists
134133
dev: digest: sha256:35670a049372ae063dad910c267a4450758a139c4deb248c04c3198865589ab2 size: 739
135-
make: Leaving directory '/home/liz/src/sigs.k8s.io/cluster-api/test/infrastructure/docker'
136134
```
137135

138136
Make a note of the URLs and the digests. You'll need them for the next step. In this case, they're...
139137

140-
`gcr.io/cluster-api-242700/manager@sha256:35670a049372ae063dad910c267a4450758a139c4deb248c04c3198865589ab2`
138+
`gcr.io/cluster-api-242700/capd-manager-amd64@sha256:35670a049372ae063dad910c267a4450758a139c4deb248c04c3198865589ab2`
141139

142140
and
143141

@@ -162,7 +160,7 @@ spec:
162160
template:
163161
spec:
164162
containers:
165-
- image: gcr.io/cluster-api-242700/manager@sha256:35670a049372ae063dad910c267a4450758a139c4deb248c04c3198865589ab2`
163+
- image: gcr.io/cluster-api-242700/capd-manager-amd64@sha256:35670a049372ae063dad910c267a4450758a139c4deb248c04c3198865589ab2`
166164
name: manager
167165
```
168166

docs/book/src/reference/jobs.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ GitHub Presubmit Workflows:
3636
### Postsubmits
3737

3838
Prow Postsubmits:
39-
* [post-cluster-api-push-images] Google Cloud Build: `make release-staging`, `make -C test/infrastructure/docker release-staging`
39+
* [post-cluster-api-push-images] Google Cloud Build: `make release-staging`, `make release-capd-staging`
4040

4141
### Periodics
4242

@@ -62,7 +62,7 @@ Prow Periodics:
6262
* GINKGO_FOCUS: `[K8s-Upgrade]`
6363
* [periodic-cluster-api-e2e-workload-upgrade-1-23-latest-main] `./scripts/ci-e2e.sh` FROM: `stable-1.23` TO: `ci/latest-1.24`
6464
* GINKGO_FOCUS: `[K8s-Upgrade]`
65-
* [cluster-api-push-images-nightly] Google Cloud Build: `make release-staging-nightly`, `make -C test/infrastructure/docker release-staging-nightly`
65+
* [cluster-api-push-images-nightly] Google Cloud Build: `make release-staging-nightly`, `make release-capd-staging-nightly`
6666

6767
## Builds and Tests running on releases
6868

scripts/ci-build.sh

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,5 @@ set -o pipefail
2020

2121
REPO_ROOT=$(dirname "${BASH_SOURCE[0]}")/..
2222

23-
# Core Cluster API
23+
# Core Cluster API and Docker provider
2424
cd "${REPO_ROOT}" && make managers clusterctl e2e-framework build-book
25-
26-
echo "*** Building Cluster API Provider Docker ***"
27-
# Docker provider
28-
cd "${REPO_ROOT}/test/infrastructure/docker" && make manager

scripts/ci-e2e-lib.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ capi:buildDockerImages () {
3535
## Build CAPD provider images, if missing
3636
if [[ "$(docker images -q "$REGISTRY/capd-manager-$ARCH:$TAG" 2> /dev/null)" == "" ]]; then
3737
echo "+ Building CAPD images"
38-
make -C test/infrastructure/docker docker-build
38+
make docker-capd-build
3939
else
4040
echo "+ CAPD images already present in the system, skipping make"
4141
fi

scripts/ci-test.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,4 @@ make test-junit
2929

3030
echo -e "\n*** Testing Cluster API Provider Docker ***\n"
3131
# Docker provider
32-
cd test/infrastructure/docker
33-
make test-junit
32+
make test-capd-junit

0 commit comments

Comments
 (0)