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

#711: Update Keycloak to newest version #729

Closed
wants to merge 3 commits into from
Closed

Conversation

anmol797
Copy link

No description provided.

Copy link

welcome bot commented Jun 25, 2024

Thanks so much for opening this pull request and for helping to improve SirixDB 🚀

@JohannesLichtenberger
Copy link
Member

"Waiting for keycloak-1 to become healthy (it's starting)"

Hmm

@anmol797
Copy link
Author

"Waiting for keycloak-1 to become healthy (it's starting)"

Hmm

the java CI gradle build is also getting failed , any idea ?

@anmol797
Copy link
Author

"Waiting for keycloak-1 to become healthy (it's starting)"

Hmm

where did you see that log ?

@JohannesLichtenberger
Copy link
Member

You have to check the docker container log locally, I guess and I still think the realm needs to be imported, as otherwise the admin user and the roles are not available and there's no out of the box way to add startup scripts anymore, I guess

@JohannesLichtenberger
Copy link
Member

BTW I think you should be able to check if the tests for sirix-rest-api run locally.

@anmol797
Copy link
Author

anmol797 commented Jun 25, 2024

BTW I think you should be able to check if the tests for sirix-rest-api run locally.

the container is not coming up for that , trying to use that jar method now
then will run the IT

@JohannesLichtenberger
Copy link
Member

I guess you've had no success so far? Maybe we have to create our own Keycloak image to still be able to run startup scripts...

@anmol797
Copy link
Author

I guess you've had no success so far? Maybe we have to create our own Keycloak image to still be able to run startup scripts...

how to make it ?

@JohannesLichtenberger
Copy link
Member

Going to close this for now

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.

2 participants