Add jandex-maven-plugin usage to gRPC Getting Started Guide #19441

nastra · 2021-08-17T06:52:01Z

People seem to be running into
#15057 (including myself)
because a Jandex Index is missing, thus we should update the gRPC
Getting Started Guide and mention this.

docs/src/main/asciidoc/grpc-getting-started.adoc

gsmet

Thanks for taking the time to contribute. I added a couple of suggestions!

docs/src/main/asciidoc/grpc-getting-started.adoc

People seem to be running into quarkusio#15057 (including myself) because a Jandex Index is missing, thus we should update the gRPC Getting Started Guide and mention this.

quarkus-bot · 2021-08-17T18:06:07Z

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building `8023497`

Status	Name	Step	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 11	`Build`	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 16	`Build`	Test failures	Logs	Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/hibernate-reactive-panache

✖ io.quarkus.it.panache.reactive.PanacheFunctionalityTest.testPanacheFunctionality line 49 - More details - Source on GitHub

java.net.SocketTimeoutException: Read timed out
	at java.base/java.net.SocketInputStream.socketRead0(Native Method)
	at java.base/java.net.SocketInputStream.socketRead(SocketInputStream.java:115)
	at java.base/java.net.SocketInputStream.read(SocketInputStream.java:168)
	at java.base/java.net.SocketInputStream.read(SocketInputStream.java:140)
	at org.apache.http.impl.io.AbstractSessionInputBuffer.fillBuffer(AbstractSessionInputBuffer.java:161)
	at org.apache.http.impl.io.SocketInputBuffer.fillBuffer(SocketInputBuffer.java:82)
	at org.apache.http.impl.io.AbstractSessionInputBuffer.readLine(AbstractSessionInputBuffer.java:276)
	at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:138)
	at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:56)
	at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:259)
	at org.apache.http.impl.AbstractHttpClientConnection.receiveResponseHeade...

⚙️ JVM Tests - JDK 16 #

📦 integration-tests/oidc-wiremock

✖ io.quarkus.it.keycloak.BearerOpaqueTokenAuthorizationTest.testSecureAccessSuccessPreferredUsername line 26 - More details - Source on GitHub

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

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:78)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:72)
	at org.codehaus.groovy.runtime.callsite.ConstructorSite$ConstructorSiteNoUnwrapNoCoerce.callConstructor(ConstructorSite.java:105)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callConstructor(AbstractCallSite.java:277)
	at io.restassured.internal.ResponseSpecificationImpl$HamcrestAssertionClosure.validate...

✖ io.quarkus.it.keycloak.BearerOpaqueTokenAuthorizationTest.testDeniedAccessAdminResource line 47 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <403> but was <500>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:78)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:72)
	at org.codehaus.groovy.runtime.callsite.ConstructorSite$ConstructorSiteNoUnwrapNoCoerce.callConstructor(ConstructorSite.java:105)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callConstructor(AbstractCallSite.java:277)
	at io.restassured.internal.ResponseSpecificationImpl$HamcrestAssertionClosure.validate...

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testSecureAccessSuccessPreferredUsername line 29 - More details - Source on GitHub

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

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:78)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:72)
	at org.codehaus.groovy.runtime.callsite.ConstructorSite$ConstructorSiteNoUnwrapNoCoerce.callConstructor(ConstructorSite.java:105)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallConstructor(CallSiteArray.java:59)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callConstructor(AbstractCallS...

✖ io.quarkus.it.keycloak.BearerTokenAuthorizationTest.testAccessAdminResource line 39 - More details - Source on GitHub

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

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:78)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:72)
	at org.codehaus.groovy.runtime.callsite.ConstructorSite$ConstructorSiteNoUnwrapNoCoerce.callConstructor(ConstructorSite.java:105)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callConstructor(AbstractCallSite.java:277)
	at io.restassured.internal.ResponseSpecificationImpl$HamcrestAssertionClosure.validate...

nastra · 2021-08-17T18:21:06Z

@michalszynkiewicz are the failed tests known flaky or something that broke due to my changes?

michalszynkiewicz · 2021-08-18T05:47:47Z

Failures don't look related, I retriggered the tests.

quarkus-bot · 2021-08-18T09:34:37Z

Failing Jobs - Building `8023497`

Status	Name	Step	Test failures	Logs	Raw logs
✖	Gradle Tests - JDK 11 Windows	`Build`	Test failures	Logs	Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ Gradle Tests - JDK 11 Windows #

📦 integration-tests/gradle

✖ io.quarkus.gradle.devmode.ModuleWithParentDependencyDevModeTest.main line 14 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeTestUtils that uses java.util.function.Supplier, java.util.function.Supplierjava.util.concurrent.atomic.AtomicReference, java.util.concurrent.atomic.AtomicReferencejava.lang.String, java.lang.Stringboolean was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:166)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:939)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:908)
	at io.quarkus.test.devmode.util.DevModeTestUtils.getHttpResponse(DevModeTestUtils.java:130)
	at io.quarkus.gradle.devmode.QuarkusDevGradleTestBase.getHttpResponse(QuarkusDevGradleTestBase.java:126)
	at io.quarkus.gradle.devmode.QuarkusDevGradleTestBase.getHttpRes...

michalszynkiewicz · 2021-08-18T11:02:46Z

Now it failed on a test that succeeded before. Again, this test doesn't look related to me.
@gsmet, WDYT?

gsmet

Looks good, thanks!

quarkus-bot bot added the area/documentation label Aug 17, 2021

nastra mentioned this pull request Aug 17, 2021

gRPC server not starting if proto grammar in maven dependency #15057

Closed

michalszynkiewicz reviewed Aug 17, 2021

View reviewed changes

docs/src/main/asciidoc/grpc-getting-started.adoc Outdated Show resolved Hide resolved

gsmet requested changes Aug 17, 2021

View reviewed changes

docs/src/main/asciidoc/grpc-getting-started.adoc Outdated Show resolved Hide resolved

docs/src/main/asciidoc/grpc-getting-started.adoc Outdated Show resolved Hide resolved

nastra force-pushed the grpc-jandex-index branch from 1a52e3a to bbf5493 Compare August 17, 2021 10:00

quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/platform Issues related to definition and interaction with Quarkus Platform labels Aug 17, 2021

nastra requested review from gsmet and michalszynkiewicz August 17, 2021 10:17

michalszynkiewicz reviewed Aug 17, 2021

View reviewed changes

docs/src/main/asciidoc/grpc-getting-started.adoc Outdated Show resolved Hide resolved

michalszynkiewicz reviewed Aug 17, 2021

View reviewed changes

docs/src/main/asciidoc/grpc-getting-started.adoc Outdated Show resolved Hide resolved

nastra force-pushed the grpc-jandex-index branch from bbf5493 to c884f8c Compare August 17, 2021 14:04

nastra added 2 commits August 17, 2021 16:05

Add jandex-maven-plugin usage to gRPC Getting Started Guide

a053d12

People seem to be running into quarkusio#15057 (including myself) because a Jandex Index is missing, thus we should update the gRPC Getting Started Guide and mention this.

Add property for jandex-maven-plugin version

8023497

nastra force-pushed the grpc-jandex-index branch from c884f8c to 8023497 Compare August 17, 2021 14:05

michalszynkiewicz approved these changes Aug 18, 2021

View reviewed changes

gsmet approved these changes Aug 23, 2021

View reviewed changes

gsmet merged commit 695e081 into quarkusio:main Aug 23, 2021

quarkus-bot bot added this to the 2.3 - main milestone Aug 23, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add jandex-maven-plugin usage to gRPC Getting Started Guide #19441

Add jandex-maven-plugin usage to gRPC Getting Started Guide #19441

nastra commented Aug 17, 2021

gsmet left a comment

quarkus-bot bot commented Aug 17, 2021 •

edited

Loading

Failing Jobs - Building `8023497`

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/hibernate-reactive-panache

⚙️ JVM Tests - JDK 16 #

📦 integration-tests/oidc-wiremock

nastra commented Aug 17, 2021

michalszynkiewicz commented Aug 18, 2021

quarkus-bot bot commented Aug 18, 2021

michalszynkiewicz commented Aug 18, 2021

gsmet left a comment

Add jandex-maven-plugin usage to gRPC Getting Started Guide #19441

Add jandex-maven-plugin usage to gRPC Getting Started Guide #19441

Conversation

nastra commented Aug 17, 2021

gsmet left a comment

Choose a reason for hiding this comment

quarkus-bot bot commented Aug 17, 2021 • edited Loading

Failing Jobs - Building 8023497

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/hibernate-reactive-panache

⚙️ JVM Tests - JDK 16 #

📦 integration-tests/oidc-wiremock

nastra commented Aug 17, 2021

michalszynkiewicz commented Aug 18, 2021

quarkus-bot bot commented Aug 18, 2021

Failing Jobs - Building 8023497

Test Failures

⚙️ Gradle Tests - JDK 11 Windows #

📦 integration-tests/gradle

michalszynkiewicz commented Aug 18, 2021

gsmet left a comment

Choose a reason for hiding this comment

quarkus-bot bot commented Aug 17, 2021 •

edited

Loading

Failing Jobs - Building `8023497`

Failing Jobs - Building `8023497`