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

keycloak-template.yaml Invokes command that causes Keycloak to use H2 #11620

Closed
BarryDrez opened this issue Oct 15, 2018 · 12 comments
Closed

keycloak-template.yaml Invokes command that causes Keycloak to use H2 #11620

BarryDrez opened this issue Oct 15, 2018 · 12 comments
Labels
kind/bug Outline of a bug - must adhere to the bug report template. status/info-needed More information is needed before the issue can move into the “analyzing” state for engineering.

Comments

@BarryDrez
Copy link

Description

Attempting to create and start a pod for Keycloak within OpenShift for multi-user che. Using the file: che/deploy/openshift/templates/multi/keycloak-template.yaml.

The file invokes the command: /scripts/kc_realm_user.sh. When tracing through the steps of the command, we can see that the standalone.xml file for Keycloak is never modified and defaults to allow Keycloak to use the H2 database.

Reproduction Steps

Create a project in OpenShift.
Create a pod for Postgresql using postgres-template.yaml (not really necessary)
Create a pod for Keycloak using keycloak-template.yaml
Note that the Postgresql db is not used and that H2 is.

OS and version:
Any OS, Latest Version

@ghost ghost added the kind/bug Outline of a bug - must adhere to the bug report template. label Oct 15, 2018
@ghost ghost self-assigned this Oct 15, 2018
@BarryDrez
Copy link
Author

There is something strange when I create a pod from the image: jboss/keycloak-openshift:3.4.3.Final and run the command:
/usr/bin/start-keycloak.sh -Dkeycloak.migration.action=import -Dkeycloak.migration.provider=dir -Dkeycloak.migration.strategy=IGNORE_EXISTING -Dkeycloak.migration.dir=/opt/jboss/keycloak/realms/ -Djboss.bind.address=0.0.0.0
I get the error:
java.lang.IllegalStateException: Directory /opt/jboss/keycloak/realms doesn't exists

It works when running che standalone in multiuser mode.

Wondering if this is related.

@BarryDrez
Copy link
Author

@eivantsov Wondering if you were able to reproduce this.

@BarryDrez
Copy link
Author

@eivantsov Can I please get an update on this?

@ghost
Copy link

ghost commented Nov 5, 2018

There's a PR in works #11761

@BarryDrez
Copy link
Author

Thank you

@BarryDrez BarryDrez changed the title keycloak-template.yaml Invokes command that causes Keycloak to us H2 keycloak-template.yaml Invokes command that causes Keycloak to use H2 Nov 5, 2018
@skabashnyuk
Copy link
Contributor

@l0rd @slemeur can you triage this issue, I think it might be important to fix for 7.0. WDYT?

@slemeur
Copy link
Contributor

slemeur commented Jul 17, 2019

Before adding it to the scope, I think it's important to review this issue and check if this is still accurate. The deployment on openshift is now handle through the operator.
I'm labelling it for platform team.

@slemeur slemeur added status/info-needed More information is needed before the issue can move into the “analyzing” state for engineering. team/platform labels Jul 17, 2019
@skabashnyuk
Copy link
Contributor

@mkuznyetsov could you take a look?

@mkuznyetsov
Copy link
Contributor

yes, on Operator it works with correct database, although it uses different keycloak image and it's entrypoint

@skabashnyuk
Copy link
Contributor

@slemeur so, basically this issue is linked to the installation way that is no longer relevant.

@l0rd
Copy link
Contributor

l0rd commented Jul 18, 2019

@mkuznyetsov @skabashnyuk so we close right?

@skabashnyuk
Copy link
Contributor

@mkuznyetsov @skabashnyuk so we close right?

+0
Maybe it supposes to happen at the same time with #13888

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. status/info-needed More information is needed before the issue can move into the “analyzing” state for engineering.
Projects
None yet
Development

No branches or pull requests

5 participants