Skip to content

Commit 9616e7e

Browse files
author
Per Goncalves da Silva
committed
update default chart values to point to pinned opm image
Signed-off-by: Per Goncalves da Silva <[email protected]>
1 parent c233ef0 commit 9616e7e

File tree

3 files changed

+12
-3
lines changed

3 files changed

+12
-3
lines changed

Makefile

+10-2
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ deploy:
163163
--set olm.image.pullPolicy=IfNotPresent \
164164
--set catalog.image.ref=$(OLM_IMAGE) \
165165
--set catalog.image.pullPolicy=IfNotPresent \
166-
--set catalog.commandArgs=--configmapServerImage=$(CONFIGMAP_SERVER_IMAGE)\
166+
--set catalog.commandArgs=--configmapServerImage=$(CONFIGMAP_SERVER_IMAGE) \
167167
--set catalog.opmImageArgs=--opmImage=$(OPERATOR_REGISTRY_IMAGE) \
168168
--set package.image.ref=$(OLM_IMAGE) \
169169
--set package.image.pullPolicy=IfNotPresent \
@@ -227,25 +227,33 @@ verify-manifests: manifests
227227

228228
verify: vendor verify-codegen verify-mockgen verify-manifests
229229

230+
.PHONY: pull-opm
231+
pull-opm:
232+
docker pull $(OPERATOR_REGISTRY_IMAGE)
233+
230234
# before running release, bump the version in OLM_VERSION and push to master,
231235
# then tag those builds in quay with the version in OLM_VERSION
232236
release: ver=v$(shell cat OLM_VERSION)
233-
release: manifests
237+
release: pull-opm manifests
234238
@echo "Generating the $(ver) release"
235239
docker pull $(IMAGE_REPO):$(ver)
236240
$(MAKE) target=upstream ver=$(ver) quickstart=true package
237241

238242
package: olmref=$(shell docker inspect --format='{{index .RepoDigests 0}}' $(IMAGE_REPO):$(ver))
243+
package: opmref=$(shell docker inspect --format='{{index .RepoDigests 0}}' $(OPERATOR_REGISTRY_IMAGE))
239244
package:
240245
ifndef target
241246
$(error target is undefined)
242247
endif
243248
ifndef ver
244249
$(error ver is undefined)
245250
endif
251+
@echo "Getting operator registry image"
252+
docker pull $(OPERATOR_REGISTRY_IMAGE)
246253
$(YQ_INTERNAL) w -i deploy/$(target)/values.yaml olm.image.ref $(olmref)
247254
$(YQ_INTERNAL) w -i deploy/$(target)/values.yaml catalog.image.ref $(olmref)
248255
$(YQ_INTERNAL) w -i deploy/$(target)/values.yaml package.image.ref $(olmref)
256+
$(YQ_INTERNAL) w -i deploy/$(target)/values.yaml -- catalog.opmImageArgs "--opmImage=$(opmref)"
249257
./scripts/package_release.sh $(ver) deploy/$(target)/manifests/$(ver) deploy/$(target)/values.yaml
250258
ln -sfFn ./$(ver) deploy/$(target)/manifests/latest
251259
ifeq ($(quickstart), true)

deploy/chart/values.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ catalog:
4040
setWorkloadUserID: true
4141
replicaCount: 1
4242
commandArgs: --configmapServerImage=quay.io/operator-framework/configmap-operator-registry:latest
43+
opmImageArgs: --opmImage=quay.io/operator-framework/opm:latest
4344
image:
4445
ref: quay.io/operator-framework/olm:master
4546
pullPolicy: Always

e2e.Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# ./pkg/controller/bundle/bundle_unpacker.go requires "/bin/cp"
2-
FROM busybox
2+
FROM fedora:40
33
COPY olm catalog package-server wait cpb copy-content /bin/
44
EXPOSE 8080
55
EXPOSE 5443

0 commit comments

Comments
 (0)