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.

Sign up for GitHub

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

Jump to bottom

Fix TLS config Javadoc typo #44704

Merged
merged 1 commit into from
Nov 26, 2024
Merged

Fix TLS config Javadoc typo #44704

merged 1 commit into from
Nov 26, 2024
+2 −2

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Nov 26, 2024

Noticed while working on the management interface.

@gsmet
Fix TLS config Javadoc typo
d59ea8f 
Noticed while working on the management interface.
@github-actions GitHub Actions
Copy link

🎊 PR Preview 30859cc has been successfully built and deployed to https://quarkus-pr-main-44704-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Nov 26, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit d59ea8f.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Nov 26, 2024 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit d59ea8f.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
✖ JVM Tests - JDK 17 Build Failures Logs Raw logs 🔍
✖ JVM Tests - JDK 21 Build Failures Logs Raw logs 🔍
✖ JVM Tests - JDK 17 Windows Build Failures Logs Raw logs 🔍

Full information is available in the Build summary check run.
You can consult the Develocity build scans.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: integration-tests/oidc-wiremock

📦 integration-tests/oidc-wiremock

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResourceWithFullCertChain line 256 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResourceWithKidOrChain line 415 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessResourceAzure line 84 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testCertChainWithCustomValidator line 207 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testFullCertChainWithOnlyRootInTruststore line 318 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.CodeFlowAuthorizationTest.testCodeFlowUserInfoCachedInIdToken line 433 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

⚙️ JVM Tests - JDK 21 #

- Failing: integration-tests/oidc-wiremock

📦 integration-tests/oidc-wiremock

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResourceWithFullCertChain line 256 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResourceWithKidOrChain line 415 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessResourceAzure line 84 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testCertChainWithCustomValidator line 207 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testFullCertChainWithOnlyRootInTruststore line 318 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

✖ io.quarkus.it.keycloak.CodeFlowAuthorizationTest.testCodeFlowUserInfoCachedInIdToken line 433 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

⚙️ JVM Tests - JDK 17 Windows #

- Failing: extensions/smallrye-graphql-client/deployment integration-tests/oidc-wiremock 
! Skipped: extensions/oidc-client-graphql/deployment integration-tests/smallrye-graphql-client integration-tests/smallrye-graphql-client-keycloak

📦 extensions/smallrye-graphql-client/deployment

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientClientAuthenticationBadKeystoreTest. line 75 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientClientAuthenticationBadKeystoreTest.setupServer(TypesafeGraphQLClientClientAuthenticationBadKeystoreTest.java:58)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientClientAuthenticationCorrectKeystoreTest. line 69 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientClientAuthenticationCorrectKeystoreTest.setupServer(TypesafeGraphQLClientClientAuthenticationCorrectKeystoreTest.java:58)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientReloadKeystoreDefaultTest. line 133 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientReloadKeystoreDefaultTest.setupServer(TypesafeGraphQLClientReloadKeystoreDefaultTest.java:97)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientReloadKeystoreTest. line 134 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientReloadKeystoreTest.setupServer(TypesafeGraphQLClientReloadKeystoreTest.java:98)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientServerAndClientAuthenticationTest. line 70 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientServerAndClientAuthenticationTest.setupServer(TypesafeGraphQLClientServerAndClientAuthenticationTest.java:59)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientServerAuthenticationBadKeystoreOnServerTest. line 73 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientServerAuthenticationBadKeystoreOnServerTest.setupServer(TypesafeGraphQLClientServerAuthenticationBadKeystoreOnServerTest.java:57)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

✖ io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientServerAuthenticationCorrectTruststoreTest. line 68 - History - More details - Source on GitHub

java.util.concurrent.ExecutionException: java.net.BindException: Address already in use: bind
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.SSLTestingTools.runServer(SSLTestingTools.java:52)
	at io.quarkus.smallrye.graphql.client.deployment.ssl.TypesafeGraphQLClientServerAuthenticationCorrectTruststoreTest.setupServer(TypesafeGraphQLClientServerAuthenticationCorrectTruststoreTest.java:57)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptBeforeAllMethod(QuarkusUnitTest.java:392)

📦 integration-tests/oidc-wiremock

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResourceWithFullCertChain line 256 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResourceWithKidOrChain line 415 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessResourceAzure line 84 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testCertChainWithCustomValidator line 207 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testFullCertChainWithOnlyRootInTruststore line 318 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

✖ io.quarkus.it.keycloak.CodeFlowAuthorizationTest.testCodeFlowUserInfoCachedInIdToken line 433 - History - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <401>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

Flaky tests - Develocity

⚙️ JVM Tests - JDK 21

📦 extensions/infinispan-cache/deployment

✖ io.quarkus.cache.infinispan.InfinispanCacheTest.testGetAsyncWithParallelCalls - History

  • expected: "thread1" but was: "thread2" - org.opentest4j.AssertionFailedError
org.opentest4j.AssertionFailedError: 

expected: "thread1"
 but was: "thread2"
	at io.quarkus.cache.infinispan.InfinispanCacheTest.testGetAsyncWithParallelCalls(InfinispanCacheTest.java:283)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
	at io.quarkus.test.QuarkusUnitTest.interceptTestMethod(QuarkusUnitTest.java:427)

@gsmet
Copy link
Member Author

gsmet commented Nov 26, 2024

@michalvavrik @sberyozkin could one of you have a look at these failures ^?

@gsmet gsmet merged commit 2dc7c1e into quarkusio:main Nov 26, 2024
52 of 55 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.18 - main milestone Nov 26, 2024
@michalvavrik
Copy link
Member

@michalvavrik @sberyozkin could one of you have a look at these failures ^?

Yeah, I'll check what has changed. I just experienced it locally and it will block both of my PRs when I push changes. @sberyozkin please let me know in a channel of your choice if you figure it out (pinging me in any PR should be enough).

@michalvavrik
Copy link
Member

It seems that we just hit the expiration time:

Verification of the token issued to client quarkus-app has failed: The JWT is no longer valid - the evaluation time NumericDate{1732638044 -> Nov 26, 2024, 5:20:44 PM CET} is on or after the Expiration Time (exp=NumericDate{-31557014167219200}) claim value..

@sberyozkin
Copy link
Member

Oops, sorry @gsmet, @michalvavrik, I'll have a look, I think there is one token there, azure one, recorded there as a test resource, provided by a user, to test a strange Azure case. Not sure what to do right now. I'll have to sign off and head home shortly, will have a look into the OIDC wiremock tests tomorrow morning at the latest, sorry about this noise - I'm sure this is not a certificate issue.

Not sure what is happening with the GraphQL tests, may be the certs have to be dynamically generated - @jmartisk, did you work on those tests ?

@jmartisk
Copy link
Contributor

Not sure what is happening with the GraphQL tests, may be the certs have to be dynamically generated - @jmartisk, did you work on those tests ?

The certs are indeed dynamically generated, but from looking at the failure ("address already in use"), it seems the problem there was a leftover HTTP server that some previous test failed to tidy up

@sberyozkin
Copy link
Member

sberyozkin commented Nov 26, 2024 •
edited
Loading

@jmartisk Sure, let me try to fix OIDC wiremock ones and see if that makes a diff, though I'll have to fix it tomorrow, getting ready to get out of the office

@sberyozkin
Copy link
Member

The most obvious solution is to update the OIDC lifespan grace property to be Long, as opposed to Int and make the Azure token last for another 20 years or whatever, right now it is set in the test to the maximum Int value.

I don't think I can use use Azure in tests, I can generate it myself to make it a bit closer to how Azure creates this token, may be it is another option, I'll have to think....

@gsmet gsmet modified the milestones: 3.18 - main, 3.17.1 Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@geoand geoand geoand approved these changes

@cescoffier cescoffier cescoffier approved these changes

Assignees
No one assigned
Labels
area/redis triage/flaky-test
Projects
None yet
Milestone
3.17.2
Development

Successfully merging this pull request may close these issues.
6 participants
@gsmet @michalvavrik @sberyozkin @jmartisk @cescoffier @geoand