Skip to content

matrix-rtc: fix services monitors#569

Merged
gaelgatelement merged 6 commits intomainfrom
gaelg/matrix-rtc-metrics
Jun 30, 2025
Merged

matrix-rtc: fix services monitors#569
gaelgatelement merged 6 commits intomainfrom
gaelg/matrix-rtc-metrics

Conversation

@gaelgatelement
Copy link
Copy Markdown
Member

In the room, it was raised that matrix-rtc services monitors were wrong : https://matrix.to/#/!RkXKfjpGXIVObpCqOI:element.io/$DM2GN8exEvvq69WPVVVBMJf-1G_E0_35G7pSB0nkhCw?via=matrix.org&via=element.io&via=k8ekat.dev

  1. This drops the Authoriser Service Monitor as it does not have any metrics
  2. This fixes the port of t he SFU Service Monitor

@gaelgatelement gaelgatelement requested a review from a team as a code owner June 23, 2025 06:42
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 23, 2025

dyff of changes in rendered templates of CI manifests

matrix-rtc-checkov-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

matrix-rtc-exposed-services-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

matrix-rtc-external-livekit-secrets-externally-values.yaml
@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci

matrix-rtc-external-livekit-secrets-in-helm-values.yaml
@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci

matrix-rtc-host-mode-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

matrix-rtc-minimal-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

matrix-rtc-secrets-externally-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

matrix-rtc-secrets-in-helm-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

pytest-matrix-rtc-standalone-values.yaml
@@ metadata @@
# apps/v1/Deployment/ess-ci/release-name-matrix-rtc-authorisation-service
! + one map entry added:
+   annotations:
+     has-no-service-monitor: "true"

@@ spec.template.metadata @@
# apps/v1/Deployment/ess-ci/release-name-matrix-rtc-authorisation-service
! + one map entry added:
+   annotations:
+     has-no-service-monitor: "true"


@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

pytest-matrix-rtc-synapse-wellknown-values.yaml
@@ metadata @@
# apps/v1/Deployment/ess-ci/release-name-matrix-rtc-authorisation-service
! + one map entry added:
+   annotations:
+     has-no-service-monitor: "true"

@@ spec.template.metadata @@
# apps/v1/Deployment/ess-ci/release-name-matrix-rtc-authorisation-service
! + one map entry added:
+   annotations:
+     has-no-service-monitor: "true"


@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-certificates-pg-external-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-certificates-pg-with-helm-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-external-cert-pg-external-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-external-cert-pg-with-helm-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-letsencrypt-pg-external-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-letsencrypt-pg-with-helm-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-wildcard-cert-pg-external-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

quick-setup-wildcard-cert-pg-with-helm-values.yaml
@@ spec.ports @@
# v1/Service/ess-ci/release-name-matrix-rtc-sfu
! + one list entry added:
+   - name: metrics
+     port: 6789
+     targetPort: metrics


@@ servicemonitor-release-name-matrix-rtc-authorisation-service.yaml @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-authorisation-service
! - one file removed
- apiVersion: monitoring.coreos.com/v1
- kind: ServiceMonitor
- metadata:
-   labels:
-     helm.sh/chart: "matrix-stack-25.6.3-dev"
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/part-of: matrix-stack
-     app.kubernetes.io/component: matrix-rtc-authorisation-service
-     app.kubernetes.io/name: matrix-rtc-authorisation-service
-     app.kubernetes.io/instance: release-name-matrix-rtc-authorisation-service
-     app.kubernetes.io/version: "0.2.3"
-   name: release-name-matrix-rtc-authorisation-service
-   namespace: ess-ci


@@ spec.endpoints.0.port @@
# monitoring.coreos.com/v1/ServiceMonitor/ess-ci/release-name-matrix-rtc-sfu
! ± value change
- http
+ metrics

@gaelgatelement gaelgatelement force-pushed the gaelg/matrix-rtc-metrics branch 2 times, most recently from 66ca7d4 to 4dd40dc Compare June 23, 2025 07:42
Copy link
Copy Markdown
Member

@benbz benbz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all looks plausible but there's no automated way of knowing whether this is correct.

How tricky would it be to have an integration test that looks at all ServiceMonitors, looks at all matching Services, poke /metrics on all Endpoints on said Services and assert all have successful responses with Prometheus/OpenTelemetry looking responses

@gaelgatelement
Copy link
Copy Markdown
Member Author

This all looks plausible but there's no automated way of knowing whether this is correct.

How tricky would it be to have an integration test that looks at all ServiceMonitors, looks at all matching Services, poke /metrics on all Endpoints on said Services and assert all have successful responses with Prometheus/OpenTelemetry looking responses

The difficult part is that we have to do some ports-forward tricks as we cant access those endpoints from the host network.

@benbz
Copy link
Copy Markdown
Member

benbz commented Jun 23, 2025

This all looks plausible but there's no automated way of knowing whether this is correct.
How tricky would it be to have an integration test that looks at all ServiceMonitors, looks at all matching Services, poke /metrics on all Endpoints on said Services and assert all have successful responses with Prometheus/OpenTelemetry looking responses

The difficult part is that we have to do some ports-forward tricks as we cant access those endpoints from the host network.

Makes sense. Or kubectl exec a cURL command would also work I suppose

Comment thread tests/integration/test_networking.py Outdated
@gaelgatelement gaelgatelement force-pushed the gaelg/matrix-rtc-metrics branch 3 times, most recently from 104869b to c9aff39 Compare June 30, 2025 11:42
@gaelgatelement gaelgatelement force-pushed the gaelg/matrix-rtc-metrics branch 2 times, most recently from b69c358 to 84b1088 Compare June 30, 2025 11:51
@gaelgatelement gaelgatelement force-pushed the gaelg/matrix-rtc-metrics branch from 84b1088 to 4694f5d Compare June 30, 2025 11:54
Comment thread charts/matrix-stack/configs/matrix-rtc/sfu/config-overrides.yaml.tpl Outdated
@gaelgatelement gaelgatelement force-pushed the gaelg/matrix-rtc-metrics branch from c93b5c0 to 4ab25b2 Compare June 30, 2025 12:35
@gaelgatelement gaelgatelement force-pushed the gaelg/matrix-rtc-metrics branch from c0f9a9f to ca62c63 Compare June 30, 2025 12:50
@gaelgatelement gaelgatelement merged commit bfc0928 into main Jun 30, 2025
67 checks passed
@gaelgatelement gaelgatelement deleted the gaelg/matrix-rtc-metrics branch June 30, 2025 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants