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

Updating ldap objects to user records #270

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

huynconfluent
Copy link
Contributor

This PR is to update the ldap assets in this examples repo to be user objects. Currently without this change, the examples where Control Center is deployed is unable to find appropriate users as they do not appear properly in ldapsearches and can prevent proper rolebindings from being assigned or even login, since they do not share the same userObjectClass as the service accounts that are used by the Confluent Components. This creates a disconnect from user expectation and experience.

The propose change here updates the ldap objects to all be user records and updates all kafka CR to use said update objects.

Copy link

cla-assistant bot commented Nov 21, 2023

CLA assistant check
All committers have signed the CLA.

Copy link

cla-assistant bot commented Nov 21, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@arodoni arodoni requested review from ptyagii and a team November 30, 2023 00:50
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