Skip to content

chore(ci): update uptime tests to include uds-portal#2813

Open
eoghanriley wants to merge 1 commit into
mainfrom
eoghanriley/core-646
Open

chore(ci): update uptime tests to include uds-portal#2813
eoghanriley wants to merge 1 commit into
mainfrom
eoghanriley/core-646

Conversation

@eoghanriley

Copy link
Copy Markdown
Contributor

Description

Added portal healthz endpoint to blackbox targets for e2e uptime tests.

Related Issue

Fixes CORE-646

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Steps to Validate

  • If this PR introduces new functionality to UDS Core or addresses a bug, please document the steps to test the changes.

Checklist before merging

@eoghanriley
eoghanriley requested a review from a team as a code owner July 17, 2026 22:54
@greptile-apps

greptile-apps Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds https://portal.uds.dev/healthz to the blackbox probe targets in the e2e uptime test, so the portal health endpoint is verified alongside the other core services.

  • The addition is backed by the portal chart's existing uptime.checks.paths: [/healthz] configuration (in uds-portal's uds-package.yaml), which causes the UDS operator to create a Prometheus Probe resource for this endpoint.
  • The /healthz path is also used as the portal deployment's liveness and readiness probe, confirming it is a legitimate health indicator.
  • No recording rule (uds:portal_endpoint:up) is added, leaving a minor consistency gap with how grafana and keycloak expose their endpoint uptime metrics — but that is out of scope for this PR.

Confidence Score: 5/5

Safe to merge — single-line test addition with no production code changes.

The change adds one URL to an existing test array. The portal chart already configures the /healthz uptime probe via its UDS Package CRD, so the backing infrastructure is in place. No logic, configuration, or runtime behavior is altered.

No files require special attention.

Important Files Changed

Filename Overview
test/vitest/uptime.spec.ts Adds https://portal.uds.dev/healthz to the blackbox probe target list; one-line change, consistent with existing pattern.

Sequence Diagram

%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
    participant Test as uptime.spec.ts
    participant Prom as Prometheus
    participant BBX as Blackbox Exporter
    participant Portal as portal.uds.dev/healthz

    Test->>Prom: "query probe_success{instance="https://portal.uds.dev/healthz"}"
    Prom-->>Test: metric value (polls up to 200s)
    Note over BBX,Portal: Already configured by UDS Operator via portal UDS Package uptime.checks.paths=[/healthz]
    BBX->>Portal: HTTP GET /healthz (SSO module)
    Portal-->>BBX: 200 OK
    BBX->>Prom: "probe_success=1"
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
    participant Test as uptime.spec.ts
    participant Prom as Prometheus
    participant BBX as Blackbox Exporter
    participant Portal as portal.uds.dev/healthz

    Test->>Prom: "query probe_success{instance="https://portal.uds.dev/healthz"}"
    Prom-->>Test: metric value (polls up to 200s)
    Note over BBX,Portal: Already configured by UDS Operator via portal UDS Package uptime.checks.paths=[/healthz]
    BBX->>Portal: HTTP GET /healthz (SSO module)
    Portal-->>BBX: 200 OK
    BBX->>Prom: "probe_success=1"
Loading

Reviews (1): Last reviewed commit: "chore(ci): update uptime tests to includ..." | Re-trigger Greptile

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.

1 participant