Skip to content

Override Keycloak admin client secret in Helm and deployment pipeline - #190

Merged
oveldman merged 2 commits into
mainfrom
bugfix/override-keycloak-settings
Jul 23, 2026
Merged

Override Keycloak admin client secret in Helm and deployment pipeline#190
oveldman merged 2 commits into
mainfrom
bugfix/override-keycloak-settings

Conversation

@oveldman

Copy link
Copy Markdown
Contributor

Summary

Adds KeyCloak__AuthServerUrl and KeyCloak__AdminClientSecret as configurable environment variables on the API Helm deployment. AuthServerUrl is always derived from the ingress domain; AdminClientSecret is exposed as a Helm value and wired through the production values file, the GitHub Actions deployment pipeline, and the docs.

Also updates the Authentication Server guide to add the view-users role assignment and remove the now-outdated manual production secret instructions (superseded by the pipeline secret).

Changes

  • deployments/helm/byakko/templates/api.yaml — added KeyCloak__AuthServerUrl (derived from ingress domain) and KeyCloak__AdminClientSecret env vars to the API deployment
  • deployments/helm/byakko/Values.yaml — added api.keycloak.adminClientSecret default value
  • deployments/helm/byakko/Values.Production.yaml — added api.keycloak.adminClientSecret override
  • .github/workflows/deploy-production.yml — added KEYCLOAK_ADMIN_CLIENT_SECRET secret to dry-run and deployment steps
  • docs/developer-guides/Pipelines.md — added KEYCLOAK_ADMIN_CLIENT_SECRET to the environment secrets table
  • docs/developer-guides/AuthenticationServer.md — added view-users to the required Keycloak roles; removed outdated manual production secret instructions

How to Test

  • Run the dry-run pipeline on a PR and confirm helm template succeeds with the new --set api.keycloak.adminClientSecret flag
  • Add KEYCLOAK_ADMIN_CLIENT_SECRET under Settings → Environments → vps-production in GitHub and deploy; verify the API pod has the KeyCloak__AdminClientSecret env
    var set correctly
  • Confirm the API can obtain a Keycloak token and delete a user account end-to-end

Checklist

  • Code builds without errors
  • Changes have been tested locally
  • No new warnings introduced

oveldman added 2 commits July 23, 2026 21:41
…pdate Helm values, document secret usage, and adjust deployment pipeline.
…nd remove outdated production secret instructions
@oveldman oveldman self-assigned this Jul 23, 2026
@oveldman oveldman added the bug Something isn't working label Jul 23, 2026
@sonarqubecloud

Copy link
Copy Markdown

@oveldman
oveldman merged commit 5dacab1 into main Jul 23, 2026
10 checks passed
@oveldman
oveldman deleted the bugfix/override-keycloak-settings branch July 23, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant