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

packaging: setup: Add keycloak_utils #669

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

didib
Copy link
Member

@didib didib commented Sep 21, 2022

with a single function, for now, keycloak_env_from_engine_conf.

Change-Id: I8d03c2ce9ec8fdad8afd796317ff1e69ea55ecbe
Signed-off-by: Yedidyah Bar David didi@redhat.com

@didib didib force-pushed the add-setup-keycloak_utils branch 3 times, most recently from 124aacb to 8625bf7 Compare September 22, 2022 13:56
with a single function, for now, keycloak_env_from_engine_conf.

Change-Id: I8d03c2ce9ec8fdad8afd796317ff1e69ea55ecbe
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
didib added a commit to didib/ovirt-dwh that referenced this pull request Oct 18, 2022
With this flow:
1. Setup an engine with keycloak, no dwh/grafana
2. engine-setup --reconfigure-optional-components, enable dwh+grafana

keycloak is already configured, but we do want to configure grafana to
use keycloak. So remove the condition that we only do this if keycloak
is not configured yet.

Also support configuring grafana on a separate machine when keycloak on
the engine machine is configured.

Requires [1]. spec updated to require 4.5.4 engine-setup to get it.

[1] oVirt/ovirt-engine#669

Change-Id: I019335c60f686d139da0e9368147e035db75ba37
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
didib added a commit to didib/ovirt-dwh that referenced this pull request Oct 24, 2022
With this flow:
1. Setup an engine with keycloak, no dwh/grafana
2. engine-setup --reconfigure-optional-components, enable dwh+grafana

keycloak is already configured, but we do want to configure grafana to
use keycloak. So remove the condition that we only do this if keycloak
is not configured yet.

Also support configuring grafana on a separate machine when keycloak on
the engine machine is configured.

Requires [1]. spec updated to require 4.5.4 engine-setup to get it.

[1] oVirt/ovirt-engine#669

Change-Id: I019335c60f686d139da0e9368147e035db75ba37
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
Copy link
Member

@sandrobonazzola sandrobonazzola left a comment

Choose a reason for hiding this comment

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

LGTM

@sandrobonazzola sandrobonazzola merged commit f62a440 into oVirt:master Nov 27, 2023
sandrobonazzola pushed a commit to didib/ovirt-dwh that referenced this pull request Nov 28, 2023
With this flow:
1. Setup an engine with keycloak, no dwh/grafana
2. engine-setup --reconfigure-optional-components, enable dwh+grafana

keycloak is already configured, but we do want to configure grafana to
use keycloak. So remove the condition that we only do this if keycloak
is not configured yet.

Also support configuring grafana on a separate machine when keycloak on
the engine machine is configured.

Requires [1]. spec updated to require 4.5.4 engine-setup to get it.

[1] oVirt/ovirt-engine#669

Change-Id: I019335c60f686d139da0e9368147e035db75ba37
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
sandrobonazzola pushed a commit to didib/ovirt-dwh that referenced this pull request Jul 12, 2024
With this flow:
1. Setup an engine with keycloak, no dwh/grafana
2. engine-setup --reconfigure-optional-components, enable dwh+grafana

keycloak is already configured, but we do want to configure grafana to
use keycloak. So remove the condition that we only do this if keycloak
is not configured yet.

Also support configuring grafana on a separate machine when keycloak on
the engine machine is configured.

Requires [1]. spec updated to require 4.5.4 engine-setup to get it.

[1] oVirt/ovirt-engine#669

Change-Id: I019335c60f686d139da0e9368147e035db75ba37
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
sandrobonazzola pushed a commit to didib/ovirt-dwh that referenced this pull request Jul 12, 2024
With this flow:
1. Setup an engine with keycloak, no dwh/grafana
2. engine-setup --reconfigure-optional-components, enable dwh+grafana

keycloak is already configured, but we do want to configure grafana to
use keycloak. So remove the condition that we only do this if keycloak
is not configured yet.

Also support configuring grafana on a separate machine when keycloak on
the engine machine is configured.

Requires [1]. spec updated to require 4.5.4 engine-setup to get it.

[1] oVirt/ovirt-engine#669

Change-Id: I019335c60f686d139da0e9368147e035db75ba37
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
sandrobonazzola pushed a commit to oVirt/ovirt-dwh that referenced this pull request Jul 12, 2024
With this flow:
1. Setup an engine with keycloak, no dwh/grafana
2. engine-setup --reconfigure-optional-components, enable dwh+grafana

keycloak is already configured, but we do want to configure grafana to
use keycloak. So remove the condition that we only do this if keycloak
is not configured yet.

Also support configuring grafana on a separate machine when keycloak on
the engine machine is configured.

Requires [1]. spec updated to require 4.5.4 engine-setup to get it.

[1] oVirt/ovirt-engine#669

Change-Id: I019335c60f686d139da0e9368147e035db75ba37
Signed-off-by: Yedidyah Bar David <didi@redhat.com>
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

2 participants