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

Check path kcadm.sh in keycloak #238

Open
flo-kn opened this issue May 7, 2024 · 0 comments · May be fixed by #239
Open

Check path kcadm.sh in keycloak #238

flo-kn opened this issue May 7, 2024 · 0 comments · May be fixed by #239

Comments

@flo-kn
Copy link

flo-kn commented May 7, 2024

At the very beginning the examples/README.md refers to the directory docker/keycloak to spin up keycloak and run the demos. Following that path one can see the keycloak container image including image tag from qay.io registry that is used for the examples is here in this compose.yml:

image: quay.io/keycloak/keycloak:23.0.5

Going further in the guide, one will run into an error trying to navigate to the script kcadm.sh. The directory in the guide is not correct for the current image version of keycloak referenced above (Currently 23.0.5):

cd /opt/jboss/keycloak

As one can see, when logging into the container based on the image version used in the keycloak/compose.yaml the path to the script should rather be /opt/keycloak/

docker ps
CONTAINER ID   IMAGE                              COMMAND                  CREATED        STATUS                         PORTS                                            NAMES
17512408edb1   quay.io/keycloak/keycloak:23.0.5   "/opt/keycloak/bin/k…"   14 hours ago   Up 14 hours                    0.0.0.0:8080->8080/tcp, 0.0.0.0:8443->8443/tcp   docker-keycloak-1
docker exec -ti docker-keycloak-1 /bin/sh
ls /opt/keycloak/bin/
client  federation-sssd-setup.sh  kcadm.bat  kcadm.sh  kc.bat  kcreg.bat  kcreg.sh  kc.sh

Not 100% sure where the difference between the guide and the actual situation in the container comes from (maybe a change that came with version upgrade of the imagequay.io/keycloak/keycloak, but has never been adjusted in the readme), but from perspective the path to the script kcadm.sh needs to be adjusted to...:

cd /opt/keycloak/

...in order to be consistent with the rest of the example.

@flo-kn flo-kn linked a pull request May 7, 2024 that will close this issue
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 a pull request may close this issue.

1 participant