charts icon indicating copy to clipboard operation
charts copied to clipboard

Published 20 hours ago • verified publisher icon bitnami

[bitnami/keycloak] Allow to use DB_VENDOR: H2 for e.g. integration tests

Open jxsl13 opened this issue 2 years ago • 6 comments

Name and Version

bitnami/keycloak 9.3.4

What is the problem this feature will solve?

Using the chart for integration tests without having to spin up a postgres database, as one does not need to persist the integration test data anyway.

What is the feature you are proposing to solve the problem?

When disabling the database and not setting any external database parameters I do get an error message. It would be nice to fallback to the DB_VENDOR: H2 in this case.

What alternatives have you considered?

the codecentric keycloak chart but that's pretty bad to configure. One also might try to not use the bitnami keycloak docker image but a different one but that is going to cause the same error when trying to disable the database.

The other alternative is to spin up keycloak with the database and still run the tests.

jxsl13 avatar Jul 01 '22 07:07 jxsl13

another good addition for integration tests would be to allow the modification of the postgresql helm chart to some extent e.g.) disabling persistence.

jxsl13 avatar Jul 01 '22 10:07 jxsl13

Hi,
Would you like to send a PR implementing this ? We will be glad to review and merge it.

rafariossaa avatar Jul 04 '22 08:07 rafariossaa

This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback.

github-actions[bot] avatar Jul 20 '22 01:07 github-actions[bot]

Let's wait until it gets reviewed and merged.

rafariossaa avatar Jul 20 '22 07:07 rafariossaa

This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback.

github-actions[bot] avatar Aug 05 '22 01:08 github-actions[bot]

I am sorry, but my previous comment was wrong. There is not PR to add this.

rafariossaa avatar Aug 10 '22 08:08 rafariossaa

This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback.

bitnami-bot avatar Aug 27 '22 01:08 bitnami-bot

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Issue. Do not hesitate to reopen it later if necessary.

bitnami-bot avatar Sep 02 '22 01:09 bitnami-bot

Hi there. I'd like this as well for ease of use in small CI test environments.

trevor-vaughan avatar Jan 06 '23 23:01 trevor-vaughan

FYI: Using chart version 15.1.2, one can work around this limitation by providing values to "trick" the chart into not complaining.

The following config works:

keycloak:
  postgresql:
    enabled: false
  externalDatabase:
    host: "none"     # any non-empty value here works
  extraEnvVars:
    - name: KEYCLOAK_DATABASE_VENDOR
      value: "dev-file"

Of course, a proper solution would be ideal.

orendain avatar May 24 '23 15:05 orendain