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.

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

Disable quarkus-cli scenarios on daily builds #631

Merged
merged 1 commit into from
May 5, 2022

Conversation

pjgg
Copy link
Contributor

@pjgg pjgg commented May 4, 2022

Summary

This PR disables Quarkus-cli daily build.
depends on: quarkusio/registry.quarkus.io#97

Checklist:

@pjgg pjgg requested review from fedinskiy and mjurc and removed request for fedinskiy May 4, 2022 07:41
java -jar $PWD/quarkus/devtools/cli/target/quarkus-cli-999-SNAPSHOT-runner.jar "\$@"
EOF
chmod +x ./quarkus-dev-cli
# TODO https://github.com/quarkusio/registry.quarkus.io/pull/97
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's make an issue in this project out of this so we don't forget about it and we're able to plan it away, yes?

Then link it here. Other todos too.

@pjgg pjgg force-pushed the feat/disable_quarkus_cli branch from 4e914ff to ad7f206 Compare May 4, 2022 08:08
@pjgg pjgg requested a review from mjurc May 4, 2022 08:08
@rsvoboda
Copy link
Member

rsvoboda commented May 4, 2022

wow, IllegalStateException: Tests already in progress

 09:25:29,283 INFO  [app] __  ____  __  _____   ___  __ ____  ______ 
09:25:29,283 INFO  [app]  --/ __ \/ / / / _ | / _ \/ //_/ / / / __/ 
09:25:29,283 INFO  [app]  -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \   
09:25:29,283 INFO  [app] --\___\_\____/_/ |_/_/|_/_/|_|\____/___/   
09:25:29,284 INFO  [app] 09:25:26,052 test-http 1.0.0-SNAPSHOT on JVM (powered by Quarkus 2.7.5.Final) started in 0.492s. Listening on: http://localhost:1101
09:25:29,284 INFO  [app] 09:25:26,064 Profile DevModeHttpMinimumReactiveIT activated. Live Coding activated.
09:25:29,284 INFO  [app] 09:25:26,066 Installed features: [cdi, resteasy-reactive, resteasy-reactive-jackson, smallrye-context-propagation, vertx]
09:25:29,284 INFO  [app] 09:25:26,066 Live reload total time: 0.529s 
09:25:29,284 INFO  [app] 09:25:27,102 Internal error running tests: java.lang.IllegalStateException: Tests already in progress
09:25:29,285 INFO  [app] 	at io.quarkus.deployment.dev.testing.ModuleTestRunner.prepare(ModuleTestRunner.java:49)
09:25:29,285 INFO  [app] 	at io.quarkus.deployment.dev.testing.TestSupport.runInternal(TestSupport.java:345)
09:25:29,285 INFO  [app] 	at io.quarkus.deployment.dev.testing.TestSupport$2.run(TestSupport.java:304)
09:25:29,286 INFO  [app] 	at java.base/java.lang.Thread.run(Thread.java:829)
09:25:29,286 INFO  [app] 09:25:28,079 Restarting quarkus due to changes in HelloResource.class.

Doesn't seem to be related to the PR, but worth reporting in Quarkus repo @pjgg

@pjgg pjgg merged commit 639b562 into quarkus-qe:main May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants