-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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] - Quickstarts Native Build + Quarkus main #6588
Comments
@cescoffier we need to make people aware of this issue and subscribe to it I think |
Once the PR is merged I will send an email to the list. |
Unfortunately, the build failed:
|
Actually I'm not able to reproduce this error locally. |
@emmanuelbernard it shouldn't be that because I have those changes locally as well. |
Jenkins CI also passed the Quickstarts so there is probably something missing we need to install on the workers to make SSL work |
I propose that for the time being we disable the rest-client test. Otherwise we'll get too accustomed to the CI failing |
SSL doesn’t work when using Docker to build an image AFAIK. That’s why we don’t have any SSL test enabled on the Azure CI. If we want to change that, we need to use plain GraalVM. |
@gsmet Ah thanks! I'll open a PR to disable it |
FTR the reason is that we can’t point to the SunEc library as it’s in the Docker image. It might be fixed with GraalVM 19.3 if everything is statically linked. |
This is done because SSL doesn't work from docker builds of the native image Fixes: quarkusio/quarkus#6588 (comment)
Hm.. we probably can pull it out of the image and place it one the file system. But that is something that would require some trial and error |
I would rather have us downloading GraalVM. But again I think this issue might be solved by 19.3. If it is, let’s not waste time. |
Here is a PR that if merged will let us know if 19.3 takes care of it or not : quarkusio/quarkus-quickstarts#429 |
The build is still failing with:
|
The latest failure was still with 19.2.1 and not 19.3.1 so let's see if the next run will work. If not I have a PR ready to go excluding the rest-client quickstart from CI |
The build is still failing with:
|
1 similar comment
The build is still failing with:
|
quarkusio/quarkus-quickstarts#430 is also needed :) |
The build is still failing with:
|
Build fixed:
|
Unfortunately, the build failed:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
Build fixed:
|
Unfortunately, the build failed:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
@sberyozkin |
I filed #45083 for it @sberyozkin |
The build is still failing:
|
Build fixed:
|
Unfortunately, the build failed:
|
The build is still failing:
|
The build is still failing:
|
The build is still failing:
|
This issue reports the current status of the quickstart
development
branch against Quarkusmaster
build. This build verifies the native compilation and also runs the native integration-tests.Don't close this issue manually. This issue will be closed when the build gets fixed. It will be re-opened if it fails again later. You can subscribe to be notify of the status.
The text was updated successfully, but these errors were encountered: