Skip to content

Commit c7f4815

Browse files
committed
Fix environment variables in Helm chart
1 parent 88f530b commit c7f4815

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

Makefile

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ OPERATOR_SDK_VERSION := v0.18.0
5252
OPERATOR_SDK = $(CURRDIR)/etc/sdk/$(UNAME_S)-$(UNAME_M)/operator-sdk
5353
OP_CHMOD := $(shell chmod +x $(OPERATOR_SDK))
5454

55-
# The image prefix to use for Coherence images
56-
COHERENCE_IMAGE_PREFIX ?= oraclecoherence/
5755
# The Coherence image name to inject into the Helm chart
5856
HELM_COHERENCE_IMAGE ?= oraclecoherence/coherence-ce:14.1.1-0-1
5957

@@ -1053,7 +1051,6 @@ build-all: build-mvn build-operator
10531051
# ---------------------------------------------------------------------------
10541052
.PHONY: run
10551053
run: export OPERATOR_IMAGE := $(OPERATOR_IMAGE)
1056-
run: export HELM_COHERENCE_IMAGE := $(HELM_COHERENCE_IMAGE)
10571054
run: export UTILS_IMAGE := $(UTILS_IMAGE)
10581055
run: export VERSION_FULL := $(VERSION_FULL)
10591056
run: export HELM_COHERENCE_IMAGE := $(HELM_COHERENCE_IMAGE)
@@ -1083,7 +1080,6 @@ run-clean: reset-namespace run
10831080
# ---------------------------------------------------------------------------
10841081
.PHONY: run-debug
10851082
run-debug: export OPERATOR_IMAGE := $(OPERATOR_IMAGE)
1086-
run-debug: export HELM_COHERENCE_IMAGE := $(HELM_COHERENCE_IMAGE)
10871083
run-debug: export UTILS_IMAGE := $(UTILS_IMAGE)
10881084
run-debug: export VERSION_FULL := $(VERSION_FULL)
10891085
run-debug: export HELM_COHERENCE_IMAGE := $(HELM_COHERENCE_IMAGE)

helm-charts/coherence-operator/templates/deployment.yaml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,6 @@ spec:
5656
- name: {{ .Chart.Name }}
5757
image: {{ .Values.coherenceOperator.image }}
5858
imagePullPolicy: {{ .Values.coherenceOperator.imagePullPolicy | default "IfNotPresent" }}
59-
env:
60-
- name: HELM_COHERENCE_IMAGE
61-
value: {{ .Values.coherenceOperator.defaultCoherenceImage | quote }}
62-
- name: UTILS_IMAGE
63-
value: {{ .Values.coherenceOperator.defaultCoherenceUtilsImage | quote }}
6459
ports:
6560
- name: "rest"
6661
containerPort: 8000
@@ -72,6 +67,10 @@ spec:
7267
# environment variables
7368
# ---------------------------------------------------------------------------
7469
env:
70+
- name: HELM_COHERENCE_IMAGE
71+
value: {{ .Values.coherenceOperator.defaultCoherenceImage | quote }}
72+
- name: UTILS_IMAGE
73+
value: {{ .Values.coherenceOperator.defaultCoherenceUtilsImage | quote }}
7574
- name: WATCH_NAMESPACE
7675
valueFrom:
7776
fieldRef:

0 commit comments

Comments
 (0)