-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[CI] - Kogito extension + Quarkus master #8694
Comments
Build fixed:
|
Unfortunately, the build failed:
|
Build fixed:
|
Unfortunately, the build failed:
|
@mariofusco the break was a problem on our side. #9231 should take care of the problem |
The build is still failing:
|
@mariofusco I am seeing this:
We recently removed |
@geoand PR sent to kogito https://github.com/kiegroup/kogito-runtimes/pull/505 thanks for having reported this. |
Build fixed:
|
Unfortunately, the build failed:
|
The build is still failing:
|
Seems like there was a connection timeout during dependency download. @mariofusco would you perhaps consider doing something like quarkusio/quarkus-platform#69 in order to avoid such failures in the future? |
The build is still failing:
|
@mariofusco can you take a look at my comment above? The build seems to have failed again due to dependency transfer issues |
@geoand I can add those maven timeouts as you suggested but I'm afraid the problem is not (only) there. Now I tried to build locally our extension against latest quarkus master and I have a few tests failing. I'll investigate the problem and keep you updated. |
Thanks @mariofusco |
@geoand this pull request should fix both problems https://github.com/kiegroup/kogito-runtimes/pull/531 I'll keep you updated. Thanks for having reported it. |
Coolio! |
Unfortunately, the build failed:
|
@evacchi there seems to be an issue with native image now:
|
@ruromero @ricardozanini as I remembered, the new CE SDK has issues with native image. We may need to register something for reflection or remove/rewrite some method using GraalVM's substitutions. Seek help with @mariofusco who knows more details -- basically in the Kogito extension there are already usages of both the techniques (the KogtioAssetProcessor class in the |
@evacchi the projects that really need to marshall/unmarshall CloudEvents (i.e. depend on cloudevents java-sdk, 2.0.0-milestone2) require to have the CloudEventsProvider registered. This won't happen in milestone3 as per cloudevents/sdk-java#233 |
The build is still failing:
|
@mariofusco but the PR has not yet been merged |
@mariofusco it is now merged, would the CI build start automatically? |
I'll restart it |
Actually I can't restart it since I don't have right in your repo. Please go to: https://github.com/kiegroup/kogito-runtimes/actions/runs/276018215 and perform the restart |
I tried to run it, but it seems that it was skipped. @mariofusco any clues? |
Ah right, I remember why. It need to be triggered by the ecosystem-ci run. I'll do that |
The build is still failing:
|
Please retrigger the build, it should be fixed by https://github.com/kiegroup/kogito-runtimes/pull/790 |
The build is still failing:
|
Seems it needs https://github.com/kiegroup/kogito-runtimes/pull/792 to be in |
Fix merged. |
Please retrigger the build, PR https://github.com/kiegroup/kogito-runtimes/pull/792 is merged |
Done |
Build fixed:
|
Unfortunately, the build failed:
|
Superceded by #12486 |
This issue will be open and closed dependent on the state of https://github.com/kiegroup/kogito-runtimes building against Quarkus master snapshot.
If you have interest in being notified of this subscribe to the issue.
The text was updated successfully, but these errors were encountered: