Skip to content

Keycloak - Directory Sync Integration - #2402

Merged
golanglemonade merged 36 commits into
theopenlane:mainfrom
manab-pr:feature/keycloak
Jul 8, 2026
Merged

Keycloak - Directory Sync Integration#2402
golanglemonade merged 36 commits into
theopenlane:mainfrom
manab-pr:feature/keycloak

Conversation

@manab-pr

@manab-pr manab-pr commented May 15, 2026

Copy link
Copy Markdown
Contributor

Closes : #2397
WIP : directory sync | keycloak integration

@manab-pr
manab-pr requested a review from a team as a code owner May 15, 2026 03:15
@manab-pr
manab-pr marked this pull request as draft May 15, 2026 03:15
@github-actions github-actions Bot added the enhancement New feature or request label May 15, 2026
@manab-pr
manab-pr marked this pull request as ready for review May 28, 2026 06:21
@manab-pr

manab-pr commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

@golanglemonade please review it

golanglemonade
golanglemonade reviewed Jun 1, 2026
View reviewed changes

@golanglemonade golanglemonade left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

realized I left these comments pending and didn't submit; didn't get through the whole thing yet but heres the comments so far

Comment thread docker/pg-init-scripts/init.sql Outdated
Comment thread internal/integrations/definitions/keycloak/mappings.go Outdated
Comment thread internal/integrations/definitions/keycloak/mappings.go
@github-actions github-actions Bot added the common label Jun 4, 2026
@manab-pr
manab-pr requested a review from golanglemonade June 4, 2026 12:38
@golanglemonade golanglemonade added the run-ci used for forks for the CI to run label Jun 11, 2026
@github-actions github-actions Bot removed the common label Jun 13, 2026
@manab-pr

manab-pr commented Jun 14, 2026

Copy link
Copy Markdown
Contributor Author

@golanglemonade cleaned .

@github-actions github-actions Bot removed the common label Jun 17, 2026
@manab-pr
manab-pr requested a review from golanglemonade June 17, 2026 16:32
@github-actions github-actions Bot removed the cli label Jun 20, 2026
golanglemonade
golanglemonade reviewed Jun 23, 2026
View reviewed changes
Comment thread docker/docker-compose-keycloak.yml Outdated
golanglemonade
golanglemonade reviewed Jun 23, 2026
View reviewed changes
Comment thread docker/pg-init-scripts/init_keycloak.sql Outdated
golanglemonade
golanglemonade reviewed Jul 6, 2026
View reviewed changes
Comment thread internal/integrations/definitions/keycloak/operation_health.go
golanglemonade
golanglemonade reviewed Jul 6, 2026
View reviewed changes
Comment thread internal/integrations/definitions/keycloak/builder.go Outdated
golanglemonade
golanglemonade reviewed Jul 6, 2026
View reviewed changes
Comment thread internal/integrations/definitions/keycloak/operation_directory_sync.go
golanglemonade
golanglemonade reviewed Jul 6, 2026
View reviewed changes
Comment thread internal/integrations/definitions/keycloak/types.go Outdated
manab-pr and others added 6 commits July 7, 2026 08:08
@manab-pr
Merge branch 'theopenlane:main' into feature/keycloak
b4fde76
@manab-pr
Merge branch 'main' of https://github.com/manab-pr/core into feature/…
146667b 
…keycloak
@manab-pr
Merge branch 'feature/keycloak' of https://github.com/manab-pr/core i…
2adf037 
…nto feature/keycloak
@manab-pr @golanglemonade
Update internal/integrations/definitions/keycloak/builder.go
c9d7239 
sure

Co-authored-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: MANAB   PRATIM MAITY <79935184+manab-pr@users.noreply.github.com>
@manab-pr @golanglemonade
Update internal/integrations/definitions/keycloak/types.go
1cb3224 
Yes , this one is valid.

Co-authored-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: MANAB   PRATIM MAITY <79935184+manab-pr@users.noreply.github.com>
@manab-pr
fix : added missing logs, and fixed the type
95e7756
@sonarqubecloud

sonarqubecloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
6.9% Coverage on New Code (required ≥ 20%)

See analysis details on SonarQube Cloud

@manab-pr
manab-pr requested a review from golanglemonade July 7, 2026 04:20
@manab-pr

manab-pr commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

@golanglemonade it's good to go now.

@golanglemonade
golanglemonade merged commit 4119deb into theopenlane:main Jul 8, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@golanglemonade golanglemonade golanglemonade approved these changes
@adelowo adelowo Awaiting requested review from adelowo adelowo is a code owner automatically assigned from theopenlane/blacksmiths
@matoszz matoszz Awaiting requested review from matoszz matoszz is a code owner automatically assigned from theopenlane/blacksmiths
@LemonAid711 LemonAid711 Awaiting requested review from LemonAid711 LemonAid711 is a code owner automatically assigned from theopenlane/blacksmiths

Assignees

No one assigned

Labels

enhancement New feature or request local-development run-ci used for forks for the CI to run

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Integration] Keycloak - Directory Sync

2 participants

@manab-pr @golanglemonade