You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "fabio-rizzo-01 (via GitHub)" <gi...@apache.org> on 2025/09/04 13:46:21 UTC
[PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
fabio-rizzo-01 opened a new pull request, #2505:
URL: https://github.com/apache/polaris/pull/2505
Added missing port property in Keycloack profile for integration testing
<!--
Possible security vulnerabilities: STOP here and contact security@apache.org instead!
Please update the title of the PR with a meaningful message - do not leave it "empty" or "generated"
Please update this summary field:
The summary should cover these topics, if applicable:
* the motivation for the change
* a description of the status quo, for example the current behavior
* the desired behavior
* etc
PR checklist:
- Do a self-review of your code before opening a pull request
- Make sure that there's good test coverage for the changes included in this PR
- Run tests locally before pushing a PR (./gradlew check)
- Code should have comments where applicable. Particularly hard-to-understand
areas deserve good in-line documentation.
- Include changes and enhancements to the documentation (in site/content/in-dev/unreleased)
- For Work In Progress Pull Requests, please use the Draft PR feature.
Make sure to add the information BELOW this comment.
Everything in this comment will NOT be added to the PR description.
-->
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2505:
URL: https://github.com/apache/polaris/pull/2505#issuecomment-3254136245
It seems it depends of the Docker "machine" you are using. The problem only occurs with Docker Desktop (I bet related to the Docker Desktop network stack). If you are using "native" Docker daemon, or podman, etc, there's no issue.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "fabio-rizzo-01 (via GitHub)" <gi...@apache.org>.
fabio-rizzo-01 commented on code in PR #2505:
URL: https://github.com/apache/polaris/pull/2505#discussion_r2322430215
##########
runtime/test-common/src/main/java/org/apache/polaris/test/commons/keycloak/KeycloakProfile.java:
##########
@@ -27,6 +27,8 @@ public class KeycloakProfile implements QuarkusTestProfile {
@Override
public Map<String, String> getConfigOverrides() {
return Map.of(
+ "quarkus.http.test-port",
Review Comment:
I know the property is in there but when I debugged the test that property is gone, so I assumed the override was removing it, even though it shouldn't be the case. Adding it back fixed the test on my machine.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on PR #2505:
URL: https://github.com/apache/polaris/pull/2505#issuecomment-3254326610
Now that i'm running
./gradlew --no-build-cache :polaris-runtime-service:intTest --tests "RestCatalogKeycloakFileIT.*" --info
I still get failures, and on this branch as well. The same 403 error I posted above.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on PR #2505:
URL: https://github.com/apache/polaris/pull/2505#issuecomment-3254262845
See my notes on the issue, I now can pass without this change *or* any others. I just ran the full suite on the RC candidate and everything passed.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "fabio-rizzo-01 (via GitHub)" <gi...@apache.org>.
fabio-rizzo-01 commented on PR #2505:
URL: https://github.com/apache/polaris/pull/2505#issuecomment-3254119734
> On my machine this at least gets us to the real exception
>
> ```
> RestCatalogKeycloakFileIT > testDropTableWithPurge() FAILED
> java.lang.RuntimeException: java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager
> at io.quarkus.test.junit.QuarkusIntegrationTestExtension.throwBootFailureException(QuarkusIntegrationTestExtension.java:346)
> at io.quarkus.test.junit.QuarkusIntegrationTestExtension.beforeEach(QuarkusIntegrationTestExtension.java:114)
> at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
> at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
>
> Caused by:
> java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager
> at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:315)
> at java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:320)
> at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1807)
> at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
> at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
> at java.base/java.lang.Thread.run(Thread.java:1583)
>
> Caused by:
> java.lang.RuntimeException: Unable to start Quarkus test resource class org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager
> at io.quarkus.test.common.TestResourceManager$TestResourceRunnable.run(TestResourceManager.java:690)
> at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804)
> ... 3 more
>
> Caused by:
> jakarta.ws.rs.ProcessingException: jakarta.ws.rs.ForbiddenException: HTTP 403 Forbidden
> at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.filterRequest(ClientInvocation.java:652)
> at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.invoke(ClientInvocation.java:424)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invokeSync(ClientInvoker.java:134)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invoke(ClientInvoker.java:103)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientProxy.invoke(ClientProxy.java:102)
> at app/jdk.proxy3/jdk.proxy3.$Proxy111.create(Unknown Source)
> at app//org.apache.polaris.test.commons.keycloak.KeycloakContainer.createRole(KeycloakContainer.java:100)
> at app//org.apache.polaris.test.commons.keycloak.KeycloakContainer.start(KeycloakContainer.java:67)
> at app//org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager.start(KeycloakLifecycleManager.java:45)
> at app//io.quarkus.test.common.TestResourceManager$TestResourceRunnable.run(TestResourceManager.java:684)
> ... 4 more
>
> Caused by:
> jakarta.ws.rs.ForbiddenException: HTTP 403 Forbidden
> at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.handleErrorStatus(ClientInvocation.java:240)
> at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.extractResult(ClientInvocation.java:216)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.extractors.BodyEntityExtractor.extractEntity(BodyEntityExtractor.java:59)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invokeSync(ClientInvoker.java:136)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invoke(ClientInvoker.java:103)
> at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientProxy.invoke(ClientProxy.java:102)
> at app/jdk.proxy3/jdk.proxy3.$Proxy102.grantToken(Unknown Source)
> at app//org.keycloak.admin.client.token.TokenManager.grantToken(TokenManager.java:95)
> at app//org.keycloak.admin.client.token.TokenManager.getAccessToken(TokenManager.java:71)
> at app//org.keycloak.admin.client.token.TokenManager.getAccessTokenString(TokenManager.java:66)
> at app//org.keycloak.admin.client.resource.BearerAuthFilter.filter(BearerAuthFilter.java:52)
> at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.filterRequest(ClientInvocation.java:644)
> ... 13 more
> ```
>
> But does not fix the issue
is this just running the integration test? Mine passes with no issues
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "RussellSpitzer (via GitHub)" <gi...@apache.org>.
RussellSpitzer commented on PR #2505:
URL: https://github.com/apache/polaris/pull/2505#issuecomment-3254109371
On my machine this at least gets us to the real exception
```
RestCatalogKeycloakFileIT > testDropTableWithPurge() FAILED
java.lang.RuntimeException: java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager
at io.quarkus.test.junit.QuarkusIntegrationTestExtension.throwBootFailureException(QuarkusIntegrationTestExtension.java:346)
at io.quarkus.test.junit.QuarkusIntegrationTestExtension.beforeEach(QuarkusIntegrationTestExtension.java:114)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
Caused by:
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager
at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:315)
at java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:320)
at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1807)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
at java.base/java.lang.Thread.run(Thread.java:1583)
Caused by:
java.lang.RuntimeException: Unable to start Quarkus test resource class org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager
at io.quarkus.test.common.TestResourceManager$TestResourceRunnable.run(TestResourceManager.java:690)
at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804)
... 3 more
Caused by:
jakarta.ws.rs.ProcessingException: jakarta.ws.rs.ForbiddenException: HTTP 403 Forbidden
at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.filterRequest(ClientInvocation.java:652)
at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.invoke(ClientInvocation.java:424)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invokeSync(ClientInvoker.java:134)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invoke(ClientInvoker.java:103)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientProxy.invoke(ClientProxy.java:102)
at app/jdk.proxy3/jdk.proxy3.$Proxy111.create(Unknown Source)
at app//org.apache.polaris.test.commons.keycloak.KeycloakContainer.createRole(KeycloakContainer.java:100)
at app//org.apache.polaris.test.commons.keycloak.KeycloakContainer.start(KeycloakContainer.java:67)
at app//org.apache.polaris.test.commons.keycloak.KeycloakLifecycleManager.start(KeycloakLifecycleManager.java:45)
at app//io.quarkus.test.common.TestResourceManager$TestResourceRunnable.run(TestResourceManager.java:684)
... 4 more
Caused by:
jakarta.ws.rs.ForbiddenException: HTTP 403 Forbidden
at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.handleErrorStatus(ClientInvocation.java:240)
at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.extractResult(ClientInvocation.java:216)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.extractors.BodyEntityExtractor.extractEntity(BodyEntityExtractor.java:59)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invokeSync(ClientInvoker.java:136)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invoke(ClientInvoker.java:103)
at app//org.jboss.resteasy.client.jaxrs.internal.proxy.ClientProxy.invoke(ClientProxy.java:102)
at app/jdk.proxy3/jdk.proxy3.$Proxy102.grantToken(Unknown Source)
at app//org.keycloak.admin.client.token.TokenManager.grantToken(TokenManager.java:95)
at app//org.keycloak.admin.client.token.TokenManager.getAccessToken(TokenManager.java:71)
at app//org.keycloak.admin.client.token.TokenManager.getAccessTokenString(TokenManager.java:66)
at app//org.keycloak.admin.client.resource.BearerAuthFilter.filter(BearerAuthFilter.java:52)
at app//org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.filterRequest(ClientInvocation.java:644)
... 13 more
```
But does not fix the issue
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "fabio-rizzo-01 (via GitHub)" <gi...@apache.org>.
fabio-rizzo-01 commented on code in PR #2505:
URL: https://github.com/apache/polaris/pull/2505#discussion_r2322430215
##########
runtime/test-common/src/main/java/org/apache/polaris/test/commons/keycloak/KeycloakProfile.java:
##########
@@ -27,6 +27,8 @@ public class KeycloakProfile implements QuarkusTestProfile {
@Override
public Map<String, String> getConfigOverrides() {
return Map.of(
+ "quarkus.http.test-port",
Review Comment:
I know the property is in there but when I debugged the test that property is gone, so I assumed the override was removing it, even though it shouldn't be the case. Adding it back fixed the test on my machine. Can someone else check if they get the same result?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #2505:
URL: https://github.com/apache/polaris/pull/2505#discussion_r2322247900
##########
runtime/test-common/src/main/java/org/apache/polaris/test/commons/keycloak/KeycloakProfile.java:
##########
@@ -27,6 +27,8 @@ public class KeycloakProfile implements QuarkusTestProfile {
@Override
public Map<String, String> getConfigOverrides() {
return Map.of(
+ "quarkus.http.test-port",
Review Comment:
This property is already set here:
https://github.com/apache/polaris/blob/0c790e0ad48d69d97033bdde08d3bf4bb415210f/runtime/defaults/src/main/resources/application.properties#L56
I'm not sure adding it here will solve the original issue tbh.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KeyCloak integration tests failing on some OSX Dev machines [polaris]
Posted by "fabio-rizzo-01 (via GitHub)" <gi...@apache.org>.
fabio-rizzo-01 closed pull request #2505: KeyCloak integration tests failing on some OSX Dev machines
URL: https://github.com/apache/polaris/pull/2505
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org