Skip to content

Keycloak migration from bitnamilegacy to official operator & chart | pr2 #1124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
sandeepbh5 wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Keycloak migration from bitnamilegacy to official operator & chart | pr2 #1124

sandeepbh5 wants to merge 12 commits into from

Conversation

@sandeepbh5
Copy link
Contributor

@sandeepbh5 sandeepbh5 commented Nov 21, 2025 •
edited
Loading

Description

This PR contains the changed for migration of keycloak (from bitnami to official keycloak).
Changes:-
Fixes # ITEP-78841

  • replaced the bitnami keycloak with official keycloak
  • Overridden the OIDC URL for respective services
  • Added alias service (for backward compatibility), so that keycloak is accessible on old URL as well.
  • Updated keycloak NS and Secrets creation
  • Added copy job application (following the pattern) to copy secrets to NS

Fixes # (issue) Cloud installer issue

  • Added the new namespace (keycloak-system) to the list for ns auto creation.
  • Corrected the logic to support DB config. (local & aurora) for keycloak.

Previous PR which got reverted due to cloudInstaller issue:-
#948

Any Newly Introduced Dependencies

NA

How Has This Been Tested?

Coder and CI

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

krishnajs
krishnajs previously approved these changes Nov 21, 2025
View reviewed changes
@hwindlas hwindlas enabled auto-merge November 21, 2025 16:51
scottmbaker
scottmbaker previously approved these changes Nov 21, 2025
View reviewed changes
@hwindlas hwindlas disabled auto-merge November 21, 2025 16:51
soniabha-intc
soniabha-intc previously approved these changes Nov 21, 2025
View reviewed changes
@sandeepbh5 sandeepbh5 changed the title fix: Added missing entry for new namespace creation fix: Added missing entry for new namespace creation & external DB support Nov 21, 2025
@sandeepbh5 sandeepbh5 force-pushed the fix-new-ns-creation-in-installer-workflow branch from 97d1fe6 to 598e5e0 Compare November 22, 2025 07:22
@sandeepbh5 sandeepbh5 changed the title fix: Added missing entry for new namespace creation & external DB support Keycloak migration from bitnamilegacy to official operator & chart Nov 22, 2025
@sandeepbh5 sandeepbh5 changed the title Keycloak migration from bitnamilegacy to official operator & chart Keycloak migration from bitnamilegacy to official operator & chart | pr2 Nov 22, 2025
@sandeepbh5 sandeepbh5 marked this pull request as draft November 25, 2025 10:06
@github-actions
Copy link

github-actions bot commented Jan 2, 2026

This pull request is stale because it has been open 30 days with no activity. Make a comment or update the PR to avoid closing PR after 15 days.

@github-actions github-actions bot added the stale label Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@scottmbaker scottmbaker scottmbaker left review comments

@krishnajs krishnajs krishnajs left review comments

@soniabha-intc soniabha-intc soniabha-intc left review comments

@johnoloughlin johnoloughlin Awaiting requested review from johnoloughlin johnoloughlin is a code owner

@garyloug garyloug Awaiting requested review from garyloug garyloug is a code owner

@palade palade Awaiting requested review from palade palade is a code owner

@rranjan3 rranjan3 Awaiting requested review from rranjan3 rranjan3 is a code owner

@hyunsun hyunsun Awaiting requested review from hyunsun hyunsun is a code owner

@damiankopyto damiankopyto Awaiting requested review from damiankopyto damiankopyto is a code owner

@cjnolan cjnolan Awaiting requested review from cjnolan cjnolan is a code owner

@SushilLakra SushilLakra Awaiting requested review from SushilLakra SushilLakra is a code owner

@guptagunjan guptagunjan Awaiting requested review from guptagunjan guptagunjan is a code owner

@sys-orch-approve sys-orch-approve Awaiting requested review from sys-orch-approve sys-orch-approve is a code owner

@Ram-srini Ram-srini Awaiting requested review from Ram-srini Ram-srini is a code owner

@shankarsrinivas1 shankarsrinivas1 Awaiting requested review from shankarsrinivas1 shankarsrinivas1 is a code owner

@adimoft adimoft Awaiting requested review from adimoft adimoft is a code owner

@daveroge daveroge Awaiting requested review from daveroge daveroge is a code owner

@shanedonohue shanedonohue Awaiting requested review from shanedonohue shanedonohue is a code owner

@manilk1x manilk1x Awaiting requested review from manilk1x manilk1x is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

DO NOT MERGE stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@sandeepbh5 @scottmbaker @krishnajs @soniabha-intc @hwindlas @johnoloughlin