Fix wrong legacy config mount (apache#141)

mrproliu · Nov 17, 2023 · c281699 · c281699
1 parent e651c38
commit c281699
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 9 deletions.
diff --git a/chart/skywalking/templates/oap-init.job.yaml b/chart/skywalking/templates/oap-init.job.yaml
@@ -83,16 +83,23 @@ spec:
         - name: {{ $key }}
           value: {{ $value | quote }}
         {{- end }}
+
         volumeMounts:
-          {{- if (.Files.Glob "files/conf.d/oap/**") }}
-            {{ range $path, $bytes := .Files.Glob "files/conf.d/oap/**" }}
-            - name: skywalking-oap-override
-              mountPath: {{ print "/skywalking/config/" ($path | replace "files/conf.d/oap/" "") }}
-              subPath: {{ $path | replace "files/conf.d/oap/" "" | b64enc | replace "=" "-" }}
-            {{- end }}
-            {{- end }}
+          {{- range $path, $config := .Values.oap.config }}
+          {{- if typeIs "string" $config }}
+          - name: skywalking-oap-override
+            mountPath: /skywalking/config/{{ $path }}
+            subPath: {{ $path }}
+          {{- else }}
+          {{- range $subpath, $oalContent := $config }}
+          - name: skywalking-oap-override
+            mountPath: /skywalking/config/{{ $path }}/{{ $subpath }}
+            subPath: {{ print $path "-" $subpath }}
+          {{- end }}
+          {{- end }}
+          {{- end }}
       volumes:
-        {{- if (.Files.Glob "files/conf.d/oap/**") }}
+        {{- if .Values.oap.config }}
         - name: skywalking-oap-override
           configMap:
             name: {{ template "skywalking.fullname" . }}-oap-cm-override

diff --git a/chart/skywalking/templates/satellite-deployment.yaml b/chart/skywalking/templates/satellite-deployment.yaml
@@ -136,7 +136,7 @@ spec:
           {{- end }}
 
       volumes:
-        {{- if (.Files.Glob "files/conf.d/satellite/**") }}
+        {{- if .Values.satellite.config }}
         - name: skywalking-satellite-override
           configMap:
             name: {{ template "skywalking.fullname" . }}-satellite-cm-override