Skip to content

fix(local): align keycloak realm import names#54

Draft
FrankSpitulski wants to merge 1 commit into
mainfrom
codex/nvbug-6202208-keycloak-realm
Draft

fix(local): align keycloak realm import names#54
FrankSpitulski wants to merge 1 commit into
mainfrom
codex/nvbug-6202208-keycloak-realm

Conversation

@FrankSpitulski
Copy link
Copy Markdown
Collaborator

Summary

  • Rename the local eval Keycloak import file from realm-csc.json to realm-event-bus.json.
  • Update setup-keycloak.sh and local infra docs to use the matching event-bus realm file name.
  • Rename the standalone auth-callout DevSpace inline realm key to realm-auth-callout.json so it also matches its realm.

Root Cause

The local Keycloak setup created keycloak-realm-import with --from-file, so the ConfigMap key inherited the source filename. The file was named realm-csc.json while the imported Keycloak realm was event-bus, making the key suggest the wrong realm.

Validation

  • Reproduced that local/infra/keycloak/realm-event-bus.json did not exist while realm-csc.json contained realm event-bus.
  • jq empty local/infra/keycloak/realm-event-bus.json.
  • bash -n local/infra/scripts/setup-keycloak.sh.
  • kubectl create configmap keycloak-realm-import --from-file=local/infra/keycloak/realm-event-bus.json --dry-run=client -o json produced key realm-event-bus.json.
  • yq parsed auth-callout/devspace.yaml.
  • rg found no remaining realm-csc references.
  • make check-license-headers.
  • git diff --check.
  • Adversarial subagent review; no blockers.

E2E

  • make -C local setup-keycloak was attempted outside the sandbox, but this machine currently has no Kind clusters, so it failed before deployment with: No kind clusters found / CSC cluster not found.

Signed-off-by: Frank Spitulski <fspitulski@nvidia.com>
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 29, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

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