-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
ceph: add ldap configs for rgw #8750
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's design more explicitly for usability and developer understanding.
Also, I will again ask where the unit tests are for this feature.
We will want community help testing this feature, and it would be most helpful if someone could help us set up an integration test for this so that we can be sure we aren't breaking it in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We also need unit tests for the spec generation, thanks!
// LDAP represents ldap settings | ||
// +optional | ||
// +nullable | ||
LDAP *LDAPSpec `json:"ldap,omitempty"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this needs to be in a higher level Spec. LDAP and OPA for instance could fall under the same category. Perhaps something 'IdentityManagement
?
This pull request has merge conflicts that must be resolved before it can be merged. @thotz please rebase it. https://rook.io/docs/rook/latest/development-flow.html#updating-your-fork |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions. |
6bce2ab
to
8df4b4b
Compare
One of the ideas was from @leseb to merge this with opa feature, I was not intended to test both features and it kinda blocked me from further development. I will refresh this PR by addressing existing commands |
This pull request has merge conflicts that must be resolved before it can be merged. @thotz please rebase it. https://rook.io/docs/rook/latest/development-flow.html#updating-your-fork |
The Ceph Object Gateway supports integrating with LDAP for authenticating and creating users, for that required config options need to be added in cephobjectstore CRD. Resolves# 4315 Signed-off-by: Jiffin Tony Thottan <thottanjiffin@gmail.com>
This pull request has merge conflicts that must be resolved before it can be merged. @thotz please rebase it. https://rook.io/docs/rook/latest/Contributing/development-flow/#updating-your-fork |
I am interested in this as well! |
This pull request has merge conflicts that must be resolved before it can be merged. @thotz please rebase it. https://rook.io/docs/rook/latest/Contributing/development-flow/#updating-your-fork |
Closing in favor of #11091 |
The Ceph Object Gateway supports integrating with LDAP for authenticating
and creating users, for that required config options need to be added in
cephobjectstore CRD.
Resolves #4315
Signed-off-by: Jiffin Tony Thottan thottanjiffin@gmail.com
Checklist:
make codegen
) has been run to update object specifications, if necessary.