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

[bitnami/keycloak] Wildfly based configuration for Keycloak Quarkus edition #40881

Closed
ice09 opened this issue Jul 12, 2023 · 5 comments · Fixed by #40948 or #42163
Closed

[bitnami/keycloak] Wildfly based configuration for Keycloak Quarkus edition #40881

ice09 opened this issue Jul 12, 2023 · 5 comments · Fixed by #40948 or #42163
Assignees
Labels
keycloak solved tech-issues The user has a technical issue about an application

Comments

@ice09
Copy link

ice09 commented Jul 12, 2023

Name and Version

bitname/keycloak

What architecture are you using?

None

What steps will reproduce the bug?

  1. see https://github.com/bitnami/containers/tree/main/bitnami/keycloak#configuration
  2. there are several Wildfly-based configuration parameters like KEYCLOAK_MANAGEMENT_USER

What is the expected behavior?

As Keycloak is Quarkus based since several versions, Wildfly is not used anymore and the configuration should not work.

What do you see instead?

Wildfly-based configuration parameters.

Additional information

No response

@ice09 ice09 added the tech-issues The user has a technical issue about an application label Jul 12, 2023
@github-actions github-actions bot added the triage Triage is needed label Jul 12, 2023
@javsalgar javsalgar changed the title Wildfly based configuration for Keycloak Quarkus edition [bitnami/keycloak] Wildfly based configuration for Keycloak Quarkus edition Jul 13, 2023
@carrodher
Copy link
Member

Thanks for creating this issue and the associated PR. The team will review it and provide feedback. Once merged the PR, this issue will be automatically closed.

@ice09
Copy link
Author

ice09 commented Jul 21, 2023

Please check the "Full Configuration" section, as far as I understand the mentioned standalone.xml configuration is not valid any more.

@github-actions github-actions bot removed the solved label Jul 21, 2023
@carrodher
Copy link
Member

Thanks for reporting this issue. Would you like to contribute by creating a PR to solve the issue? The Bitnami team will be happy to review it and provide feedback. Here you can find the contributing guidelines.

@ice09
Copy link
Author

ice09 commented Jul 22, 2023

Actually, I can't - I don't know what you want to do about this section. Personally, I would delete it as I think there is no such possibility of a full configuration with the Quarkus edition, but I am not sure.

@carrodher carrodher reopened this Jul 24, 2023
@github-actions github-actions bot added in-progress and removed triage Triage is needed labels Jul 24, 2023
@bitnami-bot bitnami-bot assigned juan131 and unassigned carrodher Jul 24, 2023
@juan131
Copy link
Contributor

juan131 commented Jul 25, 2023

Hi @ice09

As you can check in the piece of code's below, the init logic is actually copying any configuration file included at the mounted directory indicated at KEYCLOAK_MOUNTED_CONF_DIR to the Keycloak's configuration directory.

However, the README's is incorrect since it's mentioning the old Wildfly-based configuration file as an example. I created a PR to address this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keycloak solved tech-issues The user has a technical issue about an application
Projects
None yet
4 participants