ADORSYS-GIS / ADORSYS-GIS/ai-helm

[Ticket]: ai-helm-values GitLab issuer list + AuthConfig resolve branch (values-repo-first)

Offen
#590 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
ticket
Vorherrschende Sprache
Go Template
Sterne
3
Forks
1
Ø Merge
19 Std. 24 Min.
Gemergte PRs (30 T.)
80

Beschreibung

**Type:** Operational task

## Summary

We need to **add the GitLab issuer/JWKS list and the Authorino resolve branch to `ai-helm-values`** (values-repo-first), because the multi-issuer AuthConfig (ADR-0078) reads its trusted-issuer list and CEL from the values repo, not the chart (ADR-0056). Epic ADORSYS-GIS/ai-helm#588.

Expected result:
> `environments/prod/values/security-policies.yaml` carries the trusted GitLab issuers (gitlab.com + self-hosted) and the resolve/stamp branch, rendering a working multi-forge AuthConfig.

## Intent

Land the deployed configuration for GitLab trust, sequenced so it never breaks the live gateway AuthConfig.

## Source of truth (links)

- Epic ADORSYS-GIS/ai-helm#588; ADR-0078; ADR-0056 (values-repo-first), ADR-0055.

## Current Behavior

`security-policies.yaml` (the ~405-line AuthConfig) trusts Keycloak + GitHub only. GitLab issuers absent.

## Expected Behavior

- Add the GitLab issuer entries (issuer URL + JWKS) and the resolve/stamp branch, matching the chart template from the Authorino ticket.
- Per-instance egress/CA trust prerequisites captured (self-hosted JWKS reachability).

## Acceptance Criteria

- [ ] `render-check` (YAML + `helm template` of the OCI charts) is green with the GitLab issuers present.
- [ ] Change merges **only after** the chart template support (Authorino ticket) and `lightbridge-repo-auth` GitLab resolve exist — else the gateway AuthConfig breaks (ADR-0056). Explicit dependency.
- [ ] Live: a GitLab CI token → correct `x-account-id`; GitHub/Keycloak unchanged.

## Out of Scope

Chart template logic (Authorino ticket). `lightbridge-repo-auth` (separate ticket). Docs.

## Technical Context

`ai-helm-values` `environments/prod/values/security-policies.yaml`; `render-check.yml`. **Sequencing gate: lands last.**

## Risks

Mis-sequenced merge drops the gateway AuthConfig (high impact). Mitigate: values-repo-first, explicit dependency, render-check.

## Test Plan

`render-check`; live header verification for GitLab / GitHub / Keycloak.

## Verification evidence

_To be filled: render-check output; live header checks._

## Human accountable owner

@stephane-segning

## AI Usage Declaration

Drafting the ticket, Proposing implementation. Verified by owner.

## Human verification completed

- [ ] I understood the intent
- [ ] I checked the source of truth
- [ ] I verified the implementation manually
- [ ] I documented remaining risks
- [x] I am the accountable owner and accept responsibility for this ticket.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Start with environments/prod/values/security-policies.yaml and ADR-0056/ADR-0078, comparing the existing Keycloak and GitHub entries with the linked Authorino chart-template work. Confirm the lightbridge-repo-auth dependency, then run render-check.yml. Done means the values render successfully with GitLab issuers and live GitLab, GitHub, and Keycloak tokens produce the expected account headers.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
helm, kubernetes, yaml
Bereich
authentication, devops, infrastructure, security
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.