Skip to content

feat: add ppm-oidc-client-secret to site secret provisioning#163

Merged
ian-flores merged 1 commit intomainfrom
ppm-oidc-secret
Mar 5, 2026
Merged

feat: add ppm-oidc-client-secret to site secret provisioning#163
ian-flores merged 1 commit intomainfrom
ppm-oidc-secret

Conversation

@ian-flores
Copy link
Contributor

Summary

  • Adds ppm-oidc-client-secret key to SiteSecret struct (Go) and TypedDict (Python)
  • Supports PPM's OpenID Connect integration in the team-operator (PR #112)
  • Field is empty by default — users set it manually after configuring their OIDC provider

Context

The team-operator now supports PPM OIDC authentication and Identity Federation for authenticated package repos (PTDC-222, PTDC-223, PTDC-224). The operator's SecretProviderClass expects a ppm-oidc-client-secret key in the site vault to mount into the PPM container at /etc/rstudio-pm/oidc-client-secret.

This PR adds the key to the PTD secret provisioning so that:

  1. New workloads get the key created during bootstrap (empty, ready to be populated)
  2. Existing workloads get the key added via aws_ensure_secret() merge on next bootstrap run

Test plan

  • Verify just cli builds successfully
  • Verify just check passes

Add the ppm-oidc-client-secret key to the SiteSecret struct (Go) and
TypedDict (Python). This secret holds the OIDC client secret for PPM's
OpenID Connect integration, used by the team-operator to mount the
secret into the PPM container.

The field is left empty by default — users set it manually after
configuring their OIDC provider (e.g., Keycloak). The
aws_ensure_secret() merge logic will add the key to existing vaults
on the next bootstrap run without disrupting other secrets.
@ian-flores ian-flores marked this pull request as ready for review March 5, 2026 18:17
@ian-flores ian-flores requested a review from a team as a code owner March 5, 2026 18:17
@ian-flores ian-flores added this pull request to the merge queue Mar 5, 2026
Merged via the queue into main with commit 92bfe69 Mar 5, 2026
6 checks passed
@ian-flores ian-flores deleted the ppm-oidc-secret branch March 5, 2026 18:54
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