Move dokka execution to a profile #16842

evanchooly · 2021-04-27T15:35:45Z

Move the dokka executions to a profile to isolate the generation to the release execution. This also converts what little java exists in those repositories to kotlin because apparently dokka chokes on mixed-source modules.

quarkus-bot · 2021-04-27T15:52:26Z

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

Failing Jobs - Building `7a1247a`

⚠️ Artifacts of the workflow run were not available thus the report misses some details.

Status	Name	Test failures	Logs	Raw logs
⌛	Initial JDK 11 Build	⚠️ Check →	Logs	Raw logs
⌛	Attach pull request number	⚠️ Check →	Logs	Raw logs
⌛	CI Sanity Check	⚠️ Check →	Logs	Raw logs
🚫	Devtools Tests - JDK ${{ matrix.java.name }}
🚫	Devtools Tests - JDK 11 Windows
🚫	Gradle Tests - JDK 11 ${{ matrix.os.family }}
🚫	JVM Tests - JDK ${{ matrix.java.name }}
🚫	JVM Tests - JDK 11 Windows
🚫	Maven Tests - JDK ${{ matrix.java.name }}
🚫	Maven Tests - JDK 11 Windows
🚫	MicroProfile TCKs Tests
🚫	Native Tests - ${{ matrix.category }}
⌛	Native Tests - Read JSON matrix	⚠️ Check →	Logs	Raw logs
🚫	Native Tests - Windows - ${{ matrix.category }}

quarkus-bot · 2021-04-27T19:53:56Z

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

Failing Jobs - Building `3f5c0bf`

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

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 extensions/panache/hibernate-orm-panache-kotlin/deployment

✖ io.quarkus.hibernate.orm.panache.kotlin.deployment.test.multiple_pu.MultiplePersistenceUnitConfigTest.panacheOperations - More details - Source on GitHub

⚙️ JVM Tests - JDK 11 Windows #

📦 extensions/panache/hibernate-orm-panache-kotlin/deployment

✖ io.quarkus.hibernate.orm.panache.kotlin.deployment.test.multiple_pu.MultiplePersistenceUnitConfigTest.panacheOperations - More details - Source on GitHub

⚙️ JVM Tests - JDK 15 #

📦 extensions/panache/hibernate-orm-panache-kotlin/deployment

✖ io.quarkus.hibernate.orm.panache.kotlin.deployment.test.multiple_pu.MultiplePersistenceUnitConfigTest.panacheOperations - More details - Source on GitHub

quarkus-bot · 2021-04-29T17:47:31Z

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

Failing Jobs - Building `3f5c0bf`

Status	Name	Step	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 11	`Build`	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 11 Windows	`Build`	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 15	`Build`	Test failures	Logs	Raw logs
⌛	MicroProfile TCKs Tests	`Verify`	⚠️ Check →	Logs	Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 extensions/panache/hibernate-orm-panache-kotlin/deployment

✖ io.quarkus.hibernate.orm.panache.kotlin.deployment.test.multiple_pu.MultiplePersistenceUnitConfigTest.panacheOperations - More details - Source on GitHub

⚙️ JVM Tests - JDK 11 Windows #

📦 extensions/panache/hibernate-orm-panache-kotlin/deployment

✖ io.quarkus.hibernate.orm.panache.kotlin.deployment.test.multiple_pu.MultiplePersistenceUnitConfigTest.panacheOperations - More details - Source on GitHub

⚙️ JVM Tests - JDK 15 #

📦 extensions/panache/hibernate-orm-panache-kotlin/deployment

✖ io.quarkus.hibernate.orm.panache.kotlin.deployment.test.multiple_pu.MultiplePersistenceUnitConfigTest.panacheOperations - More details - Source on GitHub

gsmet

Could you revisit this PR with the addition of #17009 ?

Also I added a question.

extensions/panache/hibernate-orm-panache-kotlin/runtime/pom.xml

gsmet · 2021-05-08T09:18:35Z

@evanchooly I think we should limit this PR to its original purpose. The other changes look unrelated?

evanchooly · 2021-05-08T15:26:11Z

Well, it gets tricky here. Dokka apparently doesn't work well with mixed source trees. So I thought "ill just move it all to kotlin!" Which lead to a while.host of interesting issues like missing tests compared to the Java side, etc. I van move off resolving this diffs to a different PR but there are still failures in tests to resolve.

It's definitely snowballing so I might just rebranch and apply lessons learned and side step these other elements as much as I can. But it feels a bit like pulling on that one loose thread and watching the whole sweater unravel.

gsmet · 2021-05-10T12:50:24Z

Well. It won't work any worse than it used to work or will it?

So I would rather:

update Dokka
move things into the release profile
as a first step as it fixes some problems we have.

And then you can continue on figuring out how things can be fixed?

evanchooly · 2021-05-10T14:36:58Z

I cleaned up my overexuberance. :) There is still the one test failing on multiple PUs and I don't know why. the set up looks the same as the java side. but i'm going to go through all the dokka bits and circle back to this one.

quarkus-bot · 2021-05-10T23:35:58Z

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

Failing Jobs - Building `c710e2b`

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

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 extensions/vertx-http/deployment

✖ io.quarkus.vertx.http.hotreload.HotReloadWithFilterTest.testFilterChange line 31 - More details - Source on GitHub

✖ io.quarkus.vertx.http.hotreload.HotReloadWithRouteTest.testRouteChange line 26 - More details - Source on GitHub

✖ io.quarkus.vertx.http.hotreload.VertxInjectionTest.testEditingBeanUsingVertx line 27 - More details - Source on GitHub

⚙️ JVM Tests - JDK 11 Windows #

📦 extensions/vertx-http/deployment

✖ io.quarkus.vertx.http.hotreload.HotReloadWithFilterTest.testFilterChange line 31 - More details - Source on GitHub

✖ io.quarkus.vertx.http.hotreload.HotReloadWithRouteTest.testRouteChange line 26 - More details - Source on GitHub

✖ io.quarkus.vertx.http.hotreload.VertxInjectionTest.testEditingBeanUsingVertx line 27 - More details - Source on GitHub

⚙️ JVM Tests - JDK 16 #

📦 extensions/vertx-http/deployment

✖ io.quarkus.vertx.http.hotreload.HotReloadWithFilterTest.testFilterChange line 31 - More details - Source on GitHub

✖ io.quarkus.vertx.http.hotreload.HotReloadWithRouteTest.testRouteChange line 26 - More details - Source on GitHub

✖ io.quarkus.vertx.http.hotreload.VertxInjectionTest.testEditingBeanUsingVertx line 27 - More details - Source on GitHub

⚙️ MicroProfile TCKs Tests #

📦 tcks/resteasy-reactive/target/testsuite/tests

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.returnIsInvalidForBeingOneCharLongTest line 200 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.beanAnnotatedIsInvalidForBeingFiveCharsLongTest line 253 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.beanAnnotatedReturnIsInvalidForBeingFiveCharsLongTest line 309 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.beanIsInvalidForBeingOneCharLongTest line 144 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.returnIsInvalidForBeingFiveCharsLongTest line 181 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.constraintDeclarationExceptionThrownTest line 365 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.annotation.JAXRSClient0168.beanIsInvalidForBeingFiveCharsLongTest line 126 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.constraintviolationexceptionmapper.JAXRSClient0169.validateExecutableIsInvalidForBeingShortTest line 236 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.constraintviolationexceptionmapper.JAXRSClient0169.beanAnnotatedIsInvalidForBeingFiveCharsLongTest line 154 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.constraintviolationexceptionmapper.JAXRSClient0169.beanAnnotatedReturnIsInvalidForBeingFiveCharsLongTest line 175 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.constraintviolationexceptionmapper.JAXRSClient0169.returnIsInvalidForBeingFiveCharsLongTest line 134 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.constraintviolationexceptionmapper.JAXRSClient0169.constraintDeclarationExceptionThrownTest line 194 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.constraintviolationexceptionmapper.JAXRSClient0169.beanIsInvalidForBeingFiveCharsLongTest line 115 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.validationexceptionmapper.JAXRSClient0170.validateExecutableIsInvalidForBeingShortTest line 234 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.validationexceptionmapper.JAXRSClient0170.beanAnnotatedIsInvalidForBeingFiveCharsLongTest line 153 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.validationexceptionmapper.JAXRSClient0170.beanAnnotatedReturnIsInvalidForBeingFiveCharsLongTest line 174 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.validationexceptionmapper.JAXRSClient0170.returnIsInvalidForBeingFiveCharsLongTest line 133 - More details - Source on GitHub

✖ com.sun.ts.tests.jaxrs.platform.beanvalidation.validationexceptionmapper.JAXRSClient0170.beanIsInvalidForBeingFiveCharsLongTest line 114 - More details - Source on GitHub

quarkus-bot · 2021-05-11T04:47:39Z

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

Failing Jobs - Building `75f1501`

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