search

quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
13.18k stars 2.51k forks source link

[CI] - Quickstarts Native Build + Quarkus main #6588

Open cescoffier opened 4 years ago

cescoffier commented 4 years ago

This issue reports the current status of the quickstart development branch against Quarkus master 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.

geoand commented 4 years ago

@cescoffier we need to make people aware of this issue and subscribe to it I think

cescoffier commented 4 years ago

Once the PR is merged I will send an email to the list.

cescoffier commented 4 years ago

Unfortunately, the build failed:

geoand commented 4 years ago

Actually I'm not able to reproduce this error locally. Is there perhaps some library we need to install on the image to make this work?

emmanuelbernard commented 4 years ago

@geoand could it be the changes @gwenneg did on the default GraalVM version. Not sure what the build is using.

geoand commented 4 years ago

@emmanuelbernard it shouldn't be that because I have those changes locally as well. CI is using 19.2.1 which is what I used

geoand commented 4 years ago

Jenkins CI also passed the Quickstarts so there is probably something missing we need to install on the workers to make SSL work

geoand commented 4 years ago

I propose that for the time being we disable the rest-client test. Otherwise we'll get too accustomed to the CI failing

gsmet commented 4 years ago

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.

geoand commented 4 years ago

@gsmet Ah thanks! I'll open a PR to disable it

gsmet commented 4 years ago

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.

geoand commented 4 years ago

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.

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

gsmet commented 4 years ago

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.

geoand commented 4 years ago

Here is a PR that if merged will let us know if 19.3 takes care of it or not : https://github.com/quarkusio/quarkus-quickstarts/pull/429

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

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

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

https://github.com/quarkusio/quarkus-quickstarts/pull/430 is also needed :)

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

According to https://github.com/quarkusio/quarkus-quickstarts/actions the last CI run for successful, so maybe there is a bug in the report part of the script?

Let's see what happens during the next run

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

Failed by "capitalization"... Testing "success", value is "Success". But the Github Actions documentation says: "The current status of the job. Possible values are success, failure, or cancelled." We should ignore the case during the test.

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

The build is passing but unfortunately a failure is still reported to this issue

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

Going to add a trace in the script to check the passed status.

cescoffier commented 4 years ago

@geoand can you approve https://github.com/quarkusio/quarkus-quickstarts/pull/432?

geoand commented 4 years ago

Done

cescoffier commented 4 years ago

I've triggered https://github.com/quarkusio/quarkus-quickstarts/commit/dbefe831e917d7dabf3bcb2b033098ebaf45146a/checks?check_suite_id=409450089 that should display the traces.

geoand commented 4 years ago

Cool, thanks

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

@cescoffier looking at the commit hashes (and the pipeline definition), it seems like the periodic CI is running from the development branch. So we need to cherry pick all your CI changes to that branch

cescoffier commented 4 years ago

hum... ok, I've cherry-picked the commits.

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

hum... ok, I've cherry-picked the commits.

Are you sure? For example 52a75e0a is missing from development

cescoffier commented 4 years ago

The build is still failing with:

geoand commented 4 years ago

I just cherry-picked the last fix from master and pushed to development. I think it should fix the issue

Nevermind, push was rejected since @cescoffier did it first :)

cescoffier commented 4 years ago

At least we got configuration about the status in the last build: Success.

geoand commented 4 years ago

Yup, we are closing in on the problem :)

cescoffier commented 4 years ago

Build fixed with:

cescoffier commented 4 years ago

Unfortunately, the build failed:

geoand commented 4 years ago

The latest failure should be fixed by https://github.com/quarkusio/quarkus/pull/6681

cescoffier commented 4 years ago

The build is still failing with:

cescoffier commented 4 years ago

Build fixed with:

cescoffier commented 4 years ago

Unfortunately, the build failed: