Skip to content
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.

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

feat(docker-jans): change persistence entry check to read configuration #3617

Closed
iromli opened this issue Jan 13, 2023 · 0 comments · Fixed by #3618
Closed

feat(docker-jans): change persistence entry check to read configuration #3617

iromli opened this issue Jan 13, 2023 · 0 comments · Fixed by #3618
Assignees
Labels
comp-docker-jans-persistence-loader comp-jans-pycloudlib kind-enhancement Issue or PR is an enhancement to an existing functionality kind-feature Issue or PR is a new feature request

Comments

@iromli
Copy link
Contributor

iromli commented Jan 13, 2023

Current wait_for_persistence checks for existing client with specific inum (pulled from configmap role_based_client_id).
In hybrid persistence scenario where there are shared clients used across 2 domain/namespace/cluster, the role_based_client_id configmap in certain namespace/cluster may not match the one in shared clients hence the entrypoint couldn't get the required entry.
Unfortunately this will lead to pod stuck in wait phase as it assume persistence is not pre-populated with data yet.

The wait_for_persistence should check for configuration entry instead (i.e. ou=jans-auth,ou=configuration,o=jans) and this entry should be inserted after any client creation.

@iromli iromli added kind-enhancement Issue or PR is an enhancement to an existing functionality comp-docker-jans-persistence-loader comp-jans-pycloudlib labels Jan 13, 2023
@iromli iromli self-assigned this Jan 13, 2023
@mo-auto mo-auto added the kind-feature Issue or PR is a new feature request label Jan 13, 2023
@iromli iromli changed the title feat(docker-jans): change persistence entry check to support shared clients feat(docker-jans): change persistence entry check to read configuration Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-docker-jans-persistence-loader comp-jans-pycloudlib kind-enhancement Issue or PR is an enhancement to an existing functionality kind-feature Issue or PR is a new feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants