-
Notifications
You must be signed in to change notification settings - Fork 241
/
Makefile
375 lines (301 loc) · 13.1 KB
/
Makefile
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
.PHONY: all
all: vendor update test build
# In openshift ci (Prow), we need to set $HOME to a writable directory else tests will fail
# because they don't have permissions to create /.local or /.cache directories
# as $HOME is set to "/" by default.
ifeq ($(HOME),/)
export HOME=/tmp/home
endif
# Include the library makefile
include $(addprefix ./vendor/github.com/openshift/build-machinery-go/make/, \
golang.mk \
targets/openshift/yq.mk \
targets/openshift/bindata.mk \
targets/openshift/deps.mk \
targets/openshift/images.mk \
targets/openshift/kustomize.mk \
)
DOCKER_CMD ?= docker
CONTAINER_BUILD_FLAGS ?= --file ./Dockerfile
GOCACHE ?= $(shell C=`go env GOCACHE`; [[ -d $$C ]] && echo $$C)
ifneq ($(GOCACHE),)
GOCACHE_VOL_ARG = --volume "${GOCACHE}:/go/.cache:z"
endif
# Namespace hive-operator will run:
HIVE_OPERATOR_NS ?= hive
# Namespace hive-controllers/hiveadmission/etc will run:
HIVE_NS ?= hive
# Log level that should be used when running hive from source, or with make deploy.
LOG_LEVEL ?= debug
# Image URL to use all building/pushing image targets
IMG ?= hive-controller:latest
GO_PACKAGES :=./...
GO_BUILD_PACKAGES :=./cmd/...
GO_BUILD_BINDIR :=bin
# Exclude e2e tests from unit testing
GO_TEST_PACKAGES :=./pkg/... ./cmd/... ./contrib/...
GO_SUB_MODULES :=./apis
ifeq "$(GO_MOD_FLAGS)" "-mod=vendor"
ifeq "$(GOFLAGS)" ""
GOFLAGS_FOR_GENERATE ?= GOFLAGS=-mod=vendor
else
GOFLAGS_FOR_GENERATE ?= GOFLAGS=-mod=vendor,$(GOFLAGS)
endif
endif
# Look up distro name (e.g. Fedora)
DISTRO ?= $(shell if which lsb_release &> /dev/null; then lsb_release -si; else echo "Unknown"; fi)
# Default fedora to not using sudo since it's not needed
ifeq ($(DISTRO),Fedora)
SUDO_CMD =
else # Other distros like RHEL 7 and CentOS 7 currently need sudo.
SUDO_CMD = sudo
endif
# build-machinery-go adds a versionFromGit to -ldflags that by default constructs the version
# string based on the most recent repository tag in this branch. That doesn't work for us, since
# we don't tag versions. Override using the same versioning we apply to OperatorHub builds:
# v{major}.{minor}.{commitcount}-{sha}
# Note that building against a local commit may result in {major}.{minor} being rendered as
# `UnknownBranch`. However, the {commitcount} and {sha} should still be accurate.
SOURCE_GIT_TAG := $(shell export HOME=$(HOME); python3 -m ensurepip >&2; python3 -mpip --no-cache install --user gitpython pyyaml >&2; hack/version2.py)
BINDATA_INPUTS :=./config/sharded_controllers/... ./config/hiveadmission/... ./config/controllers/... ./config/rbac/... ./config/configmaps/...
$(call add-bindata,operator,$(BINDATA_INPUTS),,assets,pkg/operator/assets/bindata.go)
$(call build-image,hive,$(IMG),./Dockerfile,.)
$(call build-image,hive-fedora-dev-base,hive-fedora-dev-base,./build/fedora-dev/Dockerfile.devbase,.)
$(call build-image,hive-fedora-dev,$(IMG),./build/fedora-dev/Dockerfile.dev,.)
clean:
rm -rf $(GO_BUILD_BINDIR)
.PHONY: vendor
vendor:
go mod tidy
go mod vendor
.PHONY: vendor-submodules
vendor-submodules: $(addprefix vendor-submodules-,$(GO_SUB_MODULES))
vendor: vendor-submodules
.PHONY: $(addprefix vendor-submodules-,$(GO_SUB_MODULES))
$(addprefix vendor-submodules-,$(GO_SUB_MODULES)):
# handle tidy for submodules
(cd $(subst vendor-submodules-,,$@); go mod tidy && go mod vendor)
# Update the manifest directory of artifacts OLM will deploy. Copies files in from
# the locations kubebuilder generates them.
.PHONY: manifests
manifests: crd
# controller-gen is adding a yaml break (---) at the beginning of each file. OLM does not like this break.
# We use yq to strip out the yaml break by having yq replace each file with yq's formatting.
# $1 - CRD file
define strip-yaml-break
@$(YQ) m -i $(1) $(1)
endef
# patch-crd-yq allows using yq to merge patch to the CRDs generated by kubebuilder
# like adding annotations or labels etc. see yq m -h for more info
# $1 - crd file
# $2 - patch file
define patch-crd-yq
$(YQ) m -i -x '$(1)' '$(2)'
endef
CONTROLLER_GEN_SRC := $(shell realpath vendor/sigs.k8s.io/controller-tools/cmd/controller-gen)
CONTROLLER_GEN := $(shell go list -f '{{.Target}}' $(CONTROLLER_GEN_SRC))
$(CONTROLLER_GEN): $(CONTROLLER_GEN_SRC)
go install $(CONTROLLER_GEN_SRC)
# Generate CRD yaml from our api types:
.PHONY: crd
crd: $(CONTROLLER_GEN) ensure-yq
rm -rf ./config/crds
(cd apis; $(CONTROLLER_GEN) crd:crdVersions=v1 paths=./hive/v1 paths=./hiveinternal/v1alpha1 output:dir=../config/crds)
@echo Stripping yaml breaks from CRD files
$(foreach p,$(wildcard ./config/crds/*.yaml),$(call strip-yaml-break,$(p)))
@echo Patching CRD files for additional static information
$(foreach p,$(wildcard ./config/crdspatch/*.yaml),$(call patch-crd-yq,$(subst ./config/crdspatch/,./config/crds/,$(p)),$(p)))
# Patch ClusterProvision CRD to remove the massive PodSpec def we consider an internal implementation detail:
@echo Patching ClusterProvision CRD yaml to remove overly verbose PodSpec details:
$(YQ) d -i config/crds/hive.openshift.io_clusterprovisions.yaml "spec.versions[0].schema.openAPIV3Schema.properties.spec.properties.podSpec"
# This does not appear possible with controller-runtime flags when dealing with an array,
# kubebuilder:validation:EmbeddedResource adds the x-kubernetes-embedded-resource to the array,
# not the elements within it.
@echo Patching SyncSet CRDs to flag resource RawExtensions as embedded resources:
$(YQ) w -i config/crds/hive.openshift.io_syncsets.yaml "spec.versions[0].schema.openAPIV3Schema.properties.spec.properties.resources.items.x-kubernetes-embedded-resource" true
$(YQ) w -i config/crds/hive.openshift.io_syncsets.yaml "spec.versions[0].schema.openAPIV3Schema.properties.spec.properties.resources.items.x-kubernetes-preserve-unknown-fields" true
$(YQ) w -i config/crds/hive.openshift.io_selectorsyncsets.yaml "spec.versions[0].schema.openAPIV3Schema.properties.spec.properties.resources.items.x-kubernetes-embedded-resource" true
$(YQ) w -i config/crds/hive.openshift.io_selectorsyncsets.yaml "spec.versions[0].schema.openAPIV3Schema.properties.spec.properties.resources.items.x-kubernetes-preserve-unknown-fields" true
update: crd
.PHONY: verify-crd
verify-crd: $(CONTROLLER_GEN) ensure-yq
./hack/verify-crd.sh
verify: verify-crd
.PHONY: test-unit-submodules
test-unit-submodules: $(addprefix test-unit-submodules-,$(GO_SUB_MODULES))
test-unit: test-unit-submodules
.PHONY: $(addprefix test-unit-submodules-,$(GO_SUB_MODULES))
$(addprefix test-unit-submodules-,$(GO_SUB_MODULES)):
# hande unit test for submodule
(cd $(subst test-unit-submodules-,,$@); $(GO) test $(GO_MOD_FLAGS) $(GO_TEST_FLAGS) ./...)
.PHONY: test-e2e
test-e2e:
hack/e2e-test.sh
.PHONY: test-e2e-pool
test-e2e-pool:
hack/e2e-pool-test.sh
.PHONY: test-e2e-postdeploy
test-e2e-postdeploy:
go test $(GO_MOD_FLAGS) -v -timeout 0 -count=1 ./test/e2e/postdeploy/...
.PHONY: test-e2e-postinstall
test-e2e-postinstall:
go test $(GO_MOD_FLAGS) -v -timeout 0 -count=1 ./test/e2e/postinstall/...
.PHONY: test-e2e-destroycluster
test-e2e-destroycluster:
go test $(GO_MOD_FLAGS) -v -timeout 0 -count=1 ./test/e2e/destroycluster/...
.PHONY: test-e2e-uninstallhive
test-e2e-uninstallhive:
go test $(GO_MOD_FLAGS) -v -timeout 0 -count=1 ./test/e2e/uninstallhive/...
# Run against the configured cluster in ~/.kube/config
run: build
./bin/manager --log-level=${LOG_LEVEL}
# Run against the configured cluster in ~/.kube/config
run-operator: build
./bin/operator --log-level=${LOG_LEVEL}
# Install CRDs into a cluster
install: crd
oc apply -f config/crds
# Deploy controller in the configured Kubernetes cluster in ~/.kube/config
.PHONY: deploy
deploy: ensure-kustomize install
# Deploy the operator manifests:
oc create namespace ${HIVE_OPERATOR_NS} || true
mkdir -p overlays/deploy
cp overlays/template/kustomization.yaml overlays/deploy
cd overlays/deploy && ../../$(KUSTOMIZE) edit set image registry.ci.openshift.org/openshift/hive-v4.0:hive=${IMG} && ../../$(KUSTOMIZE) edit set namespace ${HIVE_OPERATOR_NS}
$(KUSTOMIZE) build overlays/deploy | sed 's/ - info/ - debug/' | oc apply -f -
rm -rf overlays/deploy
# Create a default basic HiveConfig so the operator will deploy Hive
oc process --local=true -p HIVE_NS=${HIVE_NS} -p LOG_LEVEL=${LOG_LEVEL} -f config/templates/hiveconfig.yaml | oc apply -f -
# NOTE: Keep the paths checked below in sync with those passed to the generators in `hack/update-codegen.sh`
verify-codegen: update-codegen
git diff --exit-code pkg/client
git diff --exit-code apis
verify: verify-codegen
update-codegen:
hack/update-codegen.sh
update: update-codegen
# This needs to come after codegen to copy zz_generated.deepcopy files down into vendor/
.PHONY: verify-vendor
verify-vendor: vendor
git diff --exit-code vendor/
verify: verify-vendor
# Build the template file used for direct (OLM-less) deploy by app-sre
build-app-sre-template: ensure-kustomize
# Sync CRDs into kustomize resources
cd hack/app-sre && ../../$(KUSTOMIZE) edit add resource ../../config/crds/*.yaml
# Generate temporary saas object file
$(KUSTOMIZE) build --load-restrictor=LoadRestrictionsNone hack/app-sre --output hack/app-sre/saas-objects.yaml
# Generate saas template
./hack/app-sre/generate-saas-template.py hack/app-sre/saas-template-stub.yaml hack/app-sre/saas-objects.yaml hack/app-sre/saas-template.yaml
# Remove temporary saas object file
rm hack/app-sre/saas-objects.yaml
# This needs to go after codegen so the CRDs are up to date
verify-app-sre-template: build-app-sre-template
git diff --exit-code hack/app-sre/
verify: verify-app-sre-template
# This needs to go after codegen so the CRDs are up to date
update: build-app-sre-template
# Check import naming
.PHONY: verify-imports
verify-imports: build
@echo "Verifying import naming"
@sh -c \
'for file in $(GOFILES) ; do \
$(BINDIR)/hiveutil verify-imports -c $(VERIFY_IMPORTS_CONFIG) $$file || exit 1 ; \
done'
verify: verify-imports
# Check lint
.PHONY: verify-lint
verify-lint: install-tools
@echo Verifying golint
@sh -c \
'for file in $(GOFILES) ; do \
golint --set_exit_status $$file || exit 1 ; \
done'
verify: verify-lint
.PHONY: verify-govet-submodules
verify-govet-submodules: $(addprefix verify-govet-submodules-,$(GO_SUB_MODULES))
verify-govet: verify-govet-submodules
.PHONY: $(addprefix verify-govet-submodules-,$(GO_SUB_MODULES))
$(addprefix verify-govet-submodules-,$(GO_SUB_MODULES)):
# hande govet for submodule
(cd $(subst verify-govet-submodules-,,$@); $(GO) vet $(GO_MOD_FLAGS) ./...)
# Generate code
.PHONY: generate
generate: install-tools
$(GOFLAGS_FOR_GENERATE) go generate ./pkg/... ./cmd/...
update: generate
.PHONY: generate-submodules
generate-submodules: $(addprefix generate-submodules-,$(GO_SUB_MODULES))
generate: generate-submodules
.PHONY: $(addprefix generate-submodules-,$(GO_SUB_MODULES))
$(addprefix generate-submodules-,$(GO_SUB_MODULES)): $(addprefix vendor-submodules-,$(GO_SUB_MODULES))
# hande go generate for submodule
(cd $(subst generate-submodules-,,$@); $(GOFLAGS_FOR_GENERATE) $(GO) generate ./...)
# Build the image using docker
.PHONY: docker-build
docker-build:
@echo "*** DEPRECATED: Use the image-hive target instead ***"
$(DOCKER_CMD) build $(CONTAINER_BUILD_FLAGS) -t ${IMG} .
# Push the image using docker
.PHONY: docker-push
docker-push:
$(DOCKER_CMD) push ${IMG}
# Build and push the dev image
.PHONY: docker-dev-push
docker-dev-push: build image-hive-fedora-dev docker-push
# Build the dev image using builah
.PHONY: buildah-dev-build
buildah-dev-build:
buildah bud --tag ${IMG} -f ./Dockerfile .
.PHONY: podman-dev-build
podman-dev-build:
podman build --tag ${IMG} $(GOCACHE_VOL_ARG) -f ./Dockerfile .
# Build and push the dev image with buildah
.PHONY: buildah-dev-push
buildah-dev-push: buildah-dev-build
buildah push --tls-verify=false ${IMG}
# Push the image using buildah
.PHONY: buildah-push
buildah-push:
$(SUDO_CMD) buildah pull ${IMG}
$(SUDO_CMD) buildah push ${IMG}
# Run golangci-lint against code
# TODO replace verify (except verify-generated), vet, fmt targets with lint as it covers all of it
.PHONY: lint
lint: install-tools
golangci-lint run -c ./golangci.yml ./pkg/... ./cmd/... ./contrib/...
# Remove the golangci-lint from the verify until a fix is in place for permisions for writing to the /.cache directory.
#verify: lint
# Target to build only hiveadmission
.PHONY: build-hiveadmission
build-hiveadmission:
$(call build-package, ./cmd/hiveadmission)
build: build-hiveadmission
# Target to build only hiveutil. This is used so that on the dual build RHEL8/RHEL9, RHEL8 stage only needs to build hiveutil.
.PHONY: build-hiveutil
build-hiveutil:
$(call build-package, ./contrib/cmd/hiveutil)
# Target to build only manager
.PHONY: build-manager
build-manager:
$(call build-package, ./cmd/manager)
build: build-manager
# Target to build only manager
.PHONY: build-operator
build-operator:
$(call build-package, ./cmd/operator)
build: build-operator
.PHONY: modcheck
modcheck:
go run ./hack/modcheck.go
verify: modcheck
.PHONY: install-tools
install-tools:
go install $(GO_MOD_FLAGS) github.com/golang/mock/mockgen
go install $(GO_MOD_FLAGS) golang.org/x/lint/golint
go install $(GO_MOD_FLAGS) github.com/golangci/golangci-lint/cmd/golangci-lint
.PHONY: coverage
coverage:
hack/codecov.sh