-
Notifications
You must be signed in to change notification settings - Fork 7
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
Adding saml2 support #58
Conversation
Thanks @xtrusia, taking a look. |
@xtrusia can you please rebase your PR with main ? I have unblocked the cephadm CI tests, we should be able to get your PR merged with it. |
@UtkarshBhatthere thanks. I adjusted it. but it seems that still doesn't work yet? |
The CephadmTest passes which implies the change is OK. We still have not fixed the Rook tests yet. |
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.
LGTM
@xtrusia are all the commits in this PR not signed ? Github seems to be complaining about it. |
Signed-off-by: Utkarsh Bhatt <[email protected]>
Fixing below error by adding proper pkg $ ceph dashboard sso show saml2 Error EPERM: Required library not found: `python3-saml` Signed-off-by: Seyeong Kim <[email protected]>
Signed-off-by: Utkarsh Bhatt <[email protected]>
Signed-off-by: Utkarsh Bhatt <[email protected]>
Fixing below error by adding proper pkg
$ ceph dashboard sso show saml2
Error EPERM: Required library not found:
python3-saml
Closes #57