release new version of namespace-lister (#5482)

* release new version of namespace-lister * dedicated server for namespace-lister metrics endpoint (konflux-ci/namespace-lister#51) * add correlation ID in logs (konflux-ci/namespace-lister#48) Signed-off-by: Francesco Ilario <[email protected]> * enable metrics in containter args Signed-off-by: Francesco Ilario <[email protected]> * add service to kustomization Signed-off-by: Francesco Ilario <[email protected]> * fix tls Signed-off-by: Francesco Ilario <[email protected]> --------- Signed-off-by: Francesco Ilario <[email protected]>
redhat-appstudio · Feb 6, 2025 · 86b64e2 · 86b64e2
1 parent 98010ee
commit 86b64e2
Show file tree

Hide file tree

Showing 6 changed files with 25 additions and 5 deletions.
diff --git a/components/namespace-lister/base/deployment.yaml b/components/namespace-lister/base/deployment.yaml
@@ -30,6 +30,8 @@ spec:
         - -enable-tls
         - -cert-path=/var/tls/tls.crt
         - -key-path=/var/tls/tls.key
+        - -enable-metrics
+        - -metrics-address=:9100
         image: namespace-lister:foo
         name: namespace-lister
         env:
@@ -47,6 +49,8 @@ spec:
         ports:
         - containerPort: 8080
           name: http
+        - containerPort: 9100
+          name: metrics
         securityContext:
           allowPrivilegeEscalation: false
           readOnlyRootFilesystem: true

diff --git a/components/namespace-lister/base/kustomization.yaml b/components/namespace-lister/base/kustomization.yaml
@@ -11,7 +11,7 @@ namespace: namespace-lister
 images:
 - name: namespace-lister
   newName: quay.io/konflux-ci/namespace-lister
-  newTag: 3a25aa9a18c3f9ddff8cfaf4119d1d8e45ebd5fb
+  newTag: 3f758b0f004adb5c7b94d079ddc485573d9c6823
 patches:
 - path: ./patches/with_cachenamespacelabelselector.yaml
   target:

diff --git a/components/namespace-lister/base/metrics/kustomization.yaml b/components/namespace-lister/base/metrics/kustomization.yaml
@@ -3,5 +3,6 @@ kind: Kustomization
 resources:
 - monitor.yaml
 - service-account.yaml
+- service.yaml
 - rbac.yaml
 - network_policy.yaml
diff --git a/components/namespace-lister/base/metrics/monitor.yaml b/components/namespace-lister/base/metrics/monitor.yaml
@@ -9,7 +9,7 @@ spec:
     - interval: 15s
       scheme: https
       path: /metrics
-      port: http
+      port: metrics
       authorization:
         credentials:
           key: token
@@ -20,7 +20,7 @@ spec:
             key: tls.crt
             name: namespace-lister-tls
             optional: false
-        serverName: namespace-lister.namespace-lister.svc
+        serverName: namespace-lister-metrics.namespace-lister.svc
   selector:
     matchLabels:
-      apps: namespace-lister
+      apps: namespace-lister-metrics
diff --git a/components/namespace-lister/base/metrics/network_policy.yaml b/components/namespace-lister/base/metrics/network_policy.yaml
@@ -16,4 +16,4 @@ spec:
           kubernetes.io/metadata.name: openshift-user-workload-monitoring
     ports:
     - protocol: TCP
-      port: 8080
+      port: metrics
diff --git a/components/namespace-lister/base/metrics/service.yaml b/components/namespace-lister/base/metrics/service.yaml
@@ -0,0 +1,15 @@
+apiVersion: v1
+kind: Service
+metadata:
+  labels:
+    apps: namespace-lister-metrics
+  name: namespace-lister-metrics
+  namespace: namespace-lister
+spec:
+  selector:
+    apps: namespace-lister
+  type: ClusterIP
+  ports:
+  - name: metrics
+    targetPort: metrics
+    port: 9100