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

Adding saml2 support #58

Closed
wants to merge 7 commits into from
Closed

Adding saml2 support #58

wants to merge 7 commits into from

Conversation

xtrusia
Copy link
Contributor

@xtrusia xtrusia commented Jul 22, 2024

Fixing below error by adding proper pkg

$ ceph dashboard sso show saml2
Error EPERM: Required library not found: python3-saml

Closes #57

@UtkarshBhatthere
Copy link
Contributor

Thanks @xtrusia, taking a look.

@UtkarshBhatthere
Copy link
Contributor

@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.

@xtrusia
Copy link
Contributor Author

xtrusia commented Aug 6, 2024

@UtkarshBhatthere thanks. I adjusted it. but it seems that still doesn't work yet?

@UtkarshBhatthere
Copy link
Contributor

The CephadmTest passes which implies the change is OK. We still have not fixed the Rook tests yet.

Copy link
Contributor

@UtkarshBhatthere UtkarshBhatthere left a comment

Choose a reason for hiding this comment

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

LGTM

@UtkarshBhatthere
Copy link
Contributor

@xtrusia are all the commits in this PR not signed ? Github seems to be complaining about it.

UtkarshBhatthere and others added 6 commits August 8, 2024 10:06
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]>
@xtrusia xtrusia mentioned this pull request Aug 21, 2024
@xtrusia xtrusia closed this Aug 21, 2024
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.

Error EPERM: Required library not found: python3-saml
3 participants