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

Make Keycloak Admin configurable #9490

Merged
3 commits merged into from
Apr 19, 2018
Merged

Make Keycloak Admin configurable #9490

3 commits merged into from
Apr 19, 2018

Conversation

ghost
Copy link

@ghost ghost commented Apr 19, 2018

What does this PR do?

Makes Keycloak admin user name and password configurable

What issues does this PR fix or reference?

#9489

@ghost ghost requested review from benoitf, garagatyi, l0rd and riuvshin as code owners April 19, 2018 07:34
@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/question Questions that haven't been identified as being feature requests or bugs. labels Apr 19, 2018
@@ -24,6 +21,12 @@ cat /scripts/che-realm.json.erb | \
sed -e "s@<%= scope\.lookupvar('che::che_server_url') %>@${PROTOCOL}://che-${NAMESPACE}.${ROUTING_SUFFIX}@" \
> /scripts/che-realm.json

echo "Creating Admin user..."

if [ $KEYCLOAK_USER ] && [ $KEYCLOAK_PASSWORD ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

! -z ?

Copy link
Author

Choose a reason for hiding this comment

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

Basically reused this https://github.com/jboss-dockerfiles/keycloak/blob/3.3.0.CR2-3/server/docker-entrypoint.sh

In fact, even a check isn't really required since ENVS are set in deploymentConfig.

Copy link

@garagatyi garagatyi left a comment

Choose a reason for hiding this comment

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

Looks good, but I wonder why we haven't used command line params instead of those files beforehand.

displayName: Keycloak admin password
description: Default Keycloak admin password. Can be changed after login
value: "admin"
value: http

Choose a reason for hiding this comment

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

Looks like this line is a typo

Copy link
Author

Choose a reason for hiding this comment

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

Fixed. Thanks

@@ -140,10 +140,20 @@ objects:
storage: 1Gi

parameters:

Choose a reason for hiding this comment

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

Do we need this empty line?

Copy link
Author

Choose a reason for hiding this comment

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

Fixed. Thanks

@ghost ghost merged commit b4c9fa9 into master Apr 19, 2018
@ghost ghost deleted the keycloak_admin branch April 19, 2018 11:09
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Apr 19, 2018
@benoitf benoitf added this to the 6.5.0 milestone Apr 19, 2018
hbhargav pushed a commit to hbhargav/che that referenced this pull request Dec 5, 2018
* Make Keycloak Admin configurable

* Make Keycloak Admin configurable

* Fix typos
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/question Questions that haven't been identified as being feature requests or bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants