search

quarkusio / quarkus-quickstarts

Quarkus quickstart code
https://quarkus.io
Apache License 2.0
1.92k stars 1.43k forks source link

hibernate-search-standalone-elasticsearch-quickstart #1395

Closed yrodiere closed 3 months ago

yrodiere commented 3 months ago

Relates to (and requires) https://github.com/quarkusio/quarkus/pull/39415

quarkus-bot[bot] commented 3 months ago

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

Status for workflow Pull Request Build - development

This is the status report for running Pull Request Build - development on commit a6404ed8b718c2e94dec7d59d1d30b70f98ed7c6.

Failing Jobs

Status Name Step Failures Logs Raw logs
✖ Build - JDK 17 Build with Maven :warning: Check → Logs Raw logs
quarkus-bot[bot] commented 3 months ago

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

Status for workflow Pull Request Build - development

This is the status report for running Pull Request Build - development on commit 0368ff97d6e5507ffcc8dfb5a7bd173189f8c893.

Failing Jobs

Status Name Step Failures Logs Raw logs
✖ Build - JDK 17 Build with Maven :warning: Check → Logs Raw logs
yrodiere commented 3 months ago

https://github.com/quarkusio/quarkus/pull/39415 was merged into main; restarting CI to check it works fine now, then we can merge.

quarkus-bot[bot] commented 3 months ago

Status for workflow Pull Request Build - development

This is the status report for running Pull Request Build - development on commit 0368ff97d6e5507ffcc8dfb5a7bd173189f8c893.

Failing Jobs

Status Name Step Failures Logs Raw logs
✖ Build - JDK 17 Build with Maven Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

:gear: Build - JDK 17 #

- Failing: hibernate-orm-multi-tenancy-quickstart quartz-quickstart

:package: hibernate-orm-multi-tenancy-quickstart

✖ org.acme.hibernate.orm.FruitsEndpointTest.testListAllFruitsBaseSchema - More details - Source on GitHub

``` java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.test.junit.QuarkusTestExtension.throwBootFailureException(QuarkusTestExtension.java:643) at io.quarkus.test.junit.QuarkusTestExtension.interceptTestClassConstructor(QuarkusTestExtension.java:727) at java.base/java.util.Optional.orElseGet(Optional.java:364) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) Caused by: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source) ```

:package: quartz-quickstart

✖ org.acme.quartz.TaskResourceTest.tasks - More details - Source on GitHub

``` java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.test.junit.QuarkusTestExtension.throwBootFailureException(QuarkusTestExtension.java:643) at io.quarkus.test.junit.QuarkusTestExtension.interceptTestClassConstructor(QuarkusTestExtension.java:727) at java.base/java.util.Optional.orElseGet(Optional.java:364) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) Caused by: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source) ```
yrodiere commented 3 months ago

Failures are unrelated; merging.

I'll look into the hibernate-orm-multi-tenancy-quickstart failure.

Hey @mkouba / @manovotn , do you think you could have a look at the quartz-quickstart failure? It seems to be failing on the development branch too, not just on this PR. See https://github.com/quarkusio/quarkus-quickstarts/actions/runs/8320248102/job/22764691285

yrodiere commented 3 months ago

I'll look into the hibernate-orm-multi-tenancy-quickstart failure.

Someone is already on it: https://github.com/quarkusio/quarkus-quickstarts/pull/1398

yrodiere commented 3 months ago

Hey @mkouba / @manovotn , do you think you could have a look at the quartz-quickstart failure? It seems to be failing on the development branch too, not just on this PR. See https://github.com/quarkusio/quarkus-quickstarts/actions/runs/8320248102/job/22764691285

And it looks like this will solve the quartz issue too: https://github.com/quarkusio/quarkus-quickstarts/pull/1398

Sorry for the noise :)

gsmet commented 3 months ago

Actually, this shouldn't have been merged before I branch the quickstarts for 3.9.

We don't have a branching mechanism for the quickstarts yet, we should probably have but it's low priority. I will have to revert it and it will need to be reapplied after 3.9 is branched.

(That's the reason why this PR was pending)

yrodiere commented 3 months ago

Damn, I really thought my PR got into 3.9... :disappointed: