Skip to content

Conversation

@ppalaga
Copy link
Collaborator

@ppalaga ppalaga commented May 4, 2020

Let's have a simple manually maintained dev mode test for Camel Quarkus to prevent issues like apache/camel-quarkus#1171

@ppalaga ppalaga force-pushed the 200504-devmode-test branch from b3731fd to 98f0c03 Compare May 6, 2020 14:28
@ppalaga
Copy link
Collaborator Author

ppalaga commented May 6, 2020

98f0c03 rebased to force restarting the CI.

@ppalaga ppalaga force-pushed the 200504-devmode-test branch from 98f0c03 to efa227c Compare May 13, 2020 12:18
@ppalaga
Copy link
Collaborator Author

ppalaga commented May 13, 2020

efa227c rebased to force restarting the CI.

@ppalaga
Copy link
Collaborator Author

ppalaga commented May 13, 2020

Strange that the dev mode test gets stuck on the CI although it is passing for me locally. Let me investigate tomorrow.

@ppalaga ppalaga force-pushed the 200504-devmode-test branch from 0060271 to 4ba476c Compare May 13, 2020 21:47
@ppalaga
Copy link
Collaborator Author

ppalaga commented May 13, 2020

The test output suggests, that the execution is stuck somewhere between creating the test deployment app and invoking the test.

@ppalaga
Copy link
Collaborator Author

ppalaga commented May 14, 2020

Reported quarkusio/quarkus#9299

@ppalaga ppalaga marked this pull request as draft May 14, 2020 14:28
@ppalaga ppalaga force-pushed the 200504-devmode-test branch from 5defbec to 0061a82 Compare May 14, 2020 14:56
@ppalaga ppalaga marked this pull request as ready for review May 14, 2020 19:53
@ppalaga
Copy link
Collaborator Author

ppalaga commented May 14, 2020

The CI is passing. Please review.

@ppalaga
Copy link
Collaborator Author

ppalaga commented May 25, 2020

Could please somebody review this and merge if possible?

@gsmet gsmet merged commit aa1f345 into quarkusio:master May 25, 2020
@gsmet
Copy link
Member

gsmet commented May 25, 2020

Sorry for the delay.

@ppalaga
Copy link
Collaborator Author

ppalaga commented May 25, 2020

Thanks, no problem.

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.

2 participants