- revert service port change

- change default port to 8080 - add probes
wbstack · Jan 6, 2025 · aae9581 · aae9581
1 parent 9f87303
commit aae9581
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 13 deletions.
diff --git a/charts/tool-querybuilder/templates/deployment.yaml b/charts/tool-querybuilder/templates/deployment.yaml
@@ -39,19 +39,16 @@ spec:
               value: {{ .Values.app.siConversionProperty | quote }}
           ports:
             - name: http
-              containerPort: 80
+              containerPort: 8080
               protocol: TCP
-            - name: https
-              containerPort: 443
-              protocol: TCP
-          # livenessProbe:
-          #   httpGet:
-          #     path: /
-          #     port: http
-          # readinessProbe:
-          #   httpGet:
-          #     path: /
-          #     port: http
+          livenessProbe:
+            httpGet:
+              path: /querybuilder/
+              port: http
+          readinessProbe:
+            httpGet:
+              path: /querybuilder/
+              port: http
           resources:
             {{- toYaml .Values.resources | nindent 12 }}
       {{- with .Values.nodeSelector }}

diff --git a/charts/tool-querybuilder/values.yaml b/charts/tool-querybuilder/values.yaml
@@ -29,7 +29,7 @@ securityContext: {}
 
 service:
   type: ClusterIP
-  port: 8080
+  port: 80
 
 resources: {}
   # We usually recommend not to specify default resources and to leave this as a conscious