adutra commented on code in PR #626:
URL: https://github.com/apache/polaris/pull/626#discussion_r1930699048


##########
.github/workflows/helm.yml:
##########
@@ -99,7 +99,7 @@ jobs:
         if: steps.list-changed.outputs.changed == 'true'
         run: |
           eval $(minikube -p minikube docker-env)
-          ./gradlew :polaris-quarkus-server:assemble \
+          ./gradlew :polaris-quarkus-server:assemble 
:polaris-quarkus-admin:assemble \
             -Dquarkus.container-image.build=true \
             -PeclipseLinkDeps=com.h2database:h2:2.3.232

Review Comment:
   > is probably hitting in-memory persistency for h2database i think?
   
   Yup it was using H2 in-memory.
   
   > So maybe good to introduce those changes now?
   
   I just pushed a commit that switches our tests to postgres. It's working 
well in CI. 
   
   When doing local testing though, if you install the chart more than once, 
you need to either kill the postgres pod or `--set bootstrap.enabled=false` to 
avoid an error when bootstrapping the realm, since it was already bootstrapped 
previously.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to