Skip to content

Replace legacy keycloak operator - #11

Merged
andy-cshub merged 5 commits into
masterfrom
feature/resource_keycloak_operator
Sep 24, 2026
Merged

andy-cshub merged 5 commits into
masterfrom
feature/resource_keycloak_operator

Conversation

@sapak-cshub

Copy link
Copy Markdown
Contributor

Resolves: #8

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The deployment remains on Keycloak 25.0.1, lacks legacy-operator cleanup, and can continue recurring user reconciliation.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Replaces the legacy Keycloak realm operator with Hostzero’s operator and migrates Keycloak resources to its v1beta1 CRDs.

Changes:

  • Installs Hostzero Keycloak Operator 0.12.0.
  • Migrates realms, clients, users, scopes, and credentials.
  • Bumps the umbrella chart version to 3.2.0.
File summaries
File Description
Vagrantfile Installs the replacement operator.
keycloak-user.yaml Migrates user resources.
keycloak-realm.yaml Migrates realm and scope resources.
keycloak-instance.yaml Adds operator connection configuration.
keycloak-grafana-client.yaml Migrates the Grafana client.
keycloak-external.yaml Removes the legacy external resource.
keycloak-crczp-client.yaml Migrates the CRCZP client.
Chart.yaml Bumps the chart version.
Review details
  • Files reviewed: 8/8 changed files
  • Comments generated: 6
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Vagrantfile
Comment thread Vagrantfile
Comment thread helm/crczp-head/charts/crczp-keycloak/templates/keycloak-grafana-client.yaml Outdated
Comment thread helm/crczp-head/charts/crczp-keycloak/templates/keycloak-instance.yaml Outdated
Comment thread helm/crczp-head/charts/crczp-keycloak/templates/keycloak-instance.yaml Outdated
Comment thread helm/crczp-head/charts/crczp-keycloak/templates/keycloak-user.yaml Outdated
sapak-cshub and others added 4 commits September 19, 2026 10:01
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Quote the keycloakAdmin and keycloakPassword values for proper YAML formatting.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Update baseUrl to use release namespace for Keycloak service.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@andy-cshub
andy-cshub merged commit 321e131 into master Sep 24, 2026
4 checks passed
@andy-cshub
andy-cshub deleted the feature/resource_keycloak_operator branch September 24, 2026 14:20
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.

3 participants