Skip to content

Commit 9e48d1a

Browse files
committed
fix: helm chart sgconfig serialization issue
1 parent 827401b commit 9e48d1a

3 files changed

Lines changed: 15 additions & 17 deletions

File tree

stackgres-k8s/e2e/spec/operator-demo

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,11 @@ test_yamls() {
4949
| sed "${OPERATOR_JVM_IMAGE_VERSION_LN}s/value: .*$/value: '${IMAGE_TAG%-jvm}-jvm'/" \
5050
| sed "${OPERATOR_NATIVE_IMAGE_VERSION_LN}s/value: .*$/value: '${IMAGE_TAG%-jvm}'/" \
5151
| sed "s|imagePullPolicy:.*[a-zA-Z].*$|imagePullPolicy: $E2E_OPERATOR_PULL_POLICY|" \
52-
| sed "${OPERATOR_IMAGE_TAG_LN}s/tag: .*$/tag: '$IMAGE_TAG'/" \
53-
| sed "${RESTAPI_IMAGE_TAG_LN}s/tag: .*$/tag: '$IMAGE_TAG'/" \
54-
| sed "${JOBS_IMAGE_TAG_LN}s/tag: .*$/tag: '$IMAGE_TAG'/" \
55-
| sed "${ADMINUI_IMAGE_TAG_LN}s/tag: .*$/tag: '$ADMINUI_IMAGE_TAG'/" \
56-
| sed "s|pullPolicy:.*[a-zA-Z].*$|pullPolicy: $E2E_OPERATOR_PULL_POLICY|" \
52+
| sed "s/{\(\"name\":\"stackgres\/operator\",[^}]\+\)\"tag\":\"[^\"]\+\"}/{\1\"tag\":\"$IMAGE_TAG\"}/" \
53+
| sed "s/{\(\"name\":\"stackgres\/restapi\",[^}]\+\)\"tag\":\"[^\"]\+\"}/{\1\"tag\":\"$IMAGE_TAG\"}/" \
54+
| sed "s/{\(\"name\":\"stackgres\/jobs\",[^}]\+\)\"tag\":\"[^\"]\+\"}/{\1\"tag\":\"$IMAGE_TAG\"}/" \
55+
| sed "s/{\(\"name\":\"stackgres\/admin-ui\",[^}]\+\)\"tag\":\"[^\"]\+\"}/{\1\"tag\":\"$ADMINUI_IMAGE_TAG\"}/" \
56+
| sed "s/\"pullPolicy\":\"[^\"]\+\"/\"pullPolicy\":\"$E2E_OPERATOR_PULL_POLICY\"/g" \
5757
| sed '/image: \([^/]\+\/\)\?ongres\/.*$/{$!{N;s/image: \([^/]\+\/\)\?\(ongres\/.*\)\n\( *\)imagePullPolicy: .*$/image: \2\n\3imagePullPolicy: IfNotPresent/;ty;P;D;:y}}' \
5858
| if grep -q 'http%3A%2F%2Fstackgres-operator-extensions-cache\.stackgres%3FsetHttpScheme%3Dtrue' "${STACKGRES_YAMLS}/stackgres-operator-demo.yml"
5959
then
@@ -75,12 +75,12 @@ test_yamls() {
7575
grep -qF "imagePullPolicy: $E2E_OPERATOR_PULL_POLICY" "${LOG_PATH}/stackgres-operator-demo.yml"
7676
if [ "$IMAGE_TAG" != "$ADMINUI_IMAGE_TAG" ]
7777
then
78-
grep -c "tag: '$IMAGE_TAG'$" "${LOG_PATH}/stackgres-operator-demo.yml" | grep -qxF 3
79-
grep -c "tag: '$ADMINUI_IMAGE_TAG'$" "${LOG_PATH}/stackgres-operator-demo.yml" | grep -qxF 1
78+
grep -o -i "\"tag\":\"$IMAGE_TAG\"" "${LOG_PATH}/stackgres-operator-demo.yml" | wc -l | grep -qxF 3
79+
grep -o -i "\"tag\":\"$ADMINUI_IMAGE_TAG\"" "${LOG_PATH}/stackgres-operator-demo.yml" | wc -l | grep -qxF 1
8080
else
81-
grep -c "tag: '$IMAGE_TAG'$" "${LOG_PATH}/stackgres-operator-demo.yml" | grep -qxF 4
81+
grep -o -i "\"tag\":\"$IMAGE_TAG\"" "${LOG_PATH}/stackgres-operator-demo.yml" | wc -l | grep -qxF 4
8282
fi
83-
grep -qF "pullPolicy: $E2E_OPERATOR_PULL_POLICY" "${LOG_PATH}/stackgres-operator-demo.yml"
83+
grep -qF "\"pullPolicy\":\"$E2E_OPERATOR_PULL_POLICY\"" "${LOG_PATH}/stackgres-operator-demo.yml"
8484
if grep -q 'http%3A%2F%2Fstackgres-operator-extensions-cache\.stackgres%3FsetHttpScheme%3Dtrue' "${STACKGRES_YAMLS}/stackgres-operator-demo.yml"
8585
then
8686
grep -qF "value: \"$EXTENSIONS_REPOSITORY_URL$(

stackgres-k8s/install/helm/stackgres-operator/templates/operator-deployment.yaml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,12 +90,11 @@ spec:
9090
metadata:
9191
name: {{ .Release.Name }}
9292
namespace: {{ .Values.sgConfigNamespace | default .Release.Namespace }}
93-
spec:
9493
{{- $spec := dict }}
9594
{{- range .Values.specFields }}
9695
{{- $spec := set $spec . (index $.Values .) }}
9796
{{- end }}
98-
{{ regexReplaceAll "[^ ]+: null" (toYaml $spec | nindent 16) "" }}
97+
spec: {{ toJson $spec }}
9998
{{- end }}
10099

101100
- name: INSTALL_CRDS

stackgres-k8s/install/helm/stackgres-operator/templates/sgconfig.yaml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,18 +22,17 @@ metadata:
2222
"helm.sh/hook-delete-policy": before-hook-creation,hook-succeeded
2323
"helm.sh/hook-weight": "9"
2424
data:
25-
sgconfig.yaml: |
25+
sgconfig.json: |
2626
apiVersion: stackgres.io/v1
2727
kind: SGConfig
2828
metadata:
2929
name: {{ .Release.Name }}
3030
namespace: {{ .Values.sgConfigNamespace | default .Release.Namespace }}
31-
spec:
3231
{{- $spec := dict }}
3332
{{- range .Values.specFields }}
3433
{{- $spec := set $spec . (index $.Values .) }}
3534
{{- end }}
36-
{{ regexReplaceAll "[^ ]+: null" (toYaml $spec | nindent 6) "" }}
35+
spec: {{ toJson $spec }}
3736
---
3837
apiVersion: batch/v1
3938
kind: Job
@@ -89,7 +88,7 @@ spec:
8988
RETRY=10
9089
while true
9190
do
92-
if kubectl apply -f /sgconfig.yaml
91+
if kubectl apply -f /sgconfig.json
9392
then
9493
break
9594
else
@@ -106,8 +105,8 @@ spec:
106105
mountPath: /SGConfig.yaml
107106
subPath: SGConfig.yaml
108107
- name: sgconfig
109-
mountPath: /sgconfig.yaml
110-
subPath: sgconfig.yaml
108+
mountPath: /sgconfig.json
109+
subPath: sgconfig.json
111110
volumes:
112111
- name: sgconfig-crd
113112
configMap:

0 commit comments

Comments
 (0)