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

Reconfigure SSO on keycloak activation #39

Merged
merged 1 commit into from May 31, 2022

Conversation

arso
Copy link
Contributor

@arso arso commented May 19, 2022

This patch makes sure that ovirt oauth sso configuration in
/etc/grafana/grafana.ini gets updated to use ovirt engine internal
keycloak.

This feature originally was only supporting new installations, now it
allows to enable keycloak on existing ovirt engine installations and
setups grafana to make use of ovirt internal sso.

This patch should be merged alongside with:

@arso arso force-pushed the fix_sso_on_keycloak_activation branch 3 times, most recently from bbfa681 to 7576a8d Compare May 30, 2022 13:39
This patch makes sure that ovirt oauth sso configuration in
`/etc/grafana/grafana.ini` gets updated to use ovirt engine internal
keycloak.
This feature originally was only supporting new installations, now it
allows to enable keycloak on existing ovirt engine installations and
setups grafana to make use of ovirt internal sso.
@arso arso force-pushed the fix_sso_on_keycloak_activation branch from 7576a8d to d5ba09b Compare May 30, 2022 14:28
@arso
Copy link
Contributor Author

arso commented May 30, 2022

Please see the description of recent changes that I posted there oVirt/ovirt-engine#379 (comment)

@arso arso requested a review from mwperina May 30, 2022 15:34
Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

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

+1

@mwperina mwperina merged commit 032a517 into oVirt:master May 31, 2022
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.

None yet

3 participants