quarkus icon indicating copy to clipboard operation
quarkus copied to clipboard

[CI] - Quickstarts Native Build + Quarkus main

Open cescoffier opened this issue 5 years ago • 1608 comments

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.

  • Quickstart repository: https://github.com/quarkusio/quarkus-quickstarts/tree/development
  • Build page: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 16 '20 15:01 cescoffier

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

geoand avatar Jan 17 '20 08:01 geoand

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

cescoffier avatar Jan 17 '20 09:01 cescoffier

Unfortunately, the build failed:

  • Quarkus commit: 62c47bb17712f31f90fc8fa834544b6f06ef7adb

  • Quickstarts commit: 5de86089c12d09c6b059c7e311260bc31309450e

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 17 '20 14:01 cescoffier

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?

geoand avatar Jan 17 '20 15:01 geoand

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

emmanuelbernard avatar Jan 17 '20 15:01 emmanuelbernard

@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 avatar Jan 17 '20 15:01 geoand

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

geoand avatar Jan 17 '20 16:01 geoand

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

geoand avatar Jan 17 '20 16:01 geoand

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 avatar Jan 17 '20 16:01 gsmet

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

geoand avatar Jan 17 '20 16:01 geoand

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.

gsmet avatar Jan 17 '20 16:01 gsmet

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

geoand avatar Jan 17 '20 16:01 geoand

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.

gsmet avatar Jan 17 '20 16:01 gsmet

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

geoand avatar Jan 17 '20 16:01 geoand

The build is still failing with:

  • Quarkus commit: 373ce14d08ce4fa6075b3c6c68d5e49b4cc3936d

  • Quickstarts commit: 42451e92089316c304f9514d330725a8797e77ac

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 17 '20 19:01 cescoffier

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

geoand avatar Jan 17 '20 20:01 geoand

The build is still failing with:

  • Quarkus commit: fd73233b491e6658bc2501ba8fc3096e8cbf7837

  • Quickstarts commit: 42451e92089316c304f9514d330725a8797e77ac

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 18 '20 01:01 cescoffier

The build is still failing with:

  • Quarkus commit: fd73233b491e6658bc2501ba8fc3096e8cbf7837

  • Quickstarts commit: 42451e92089316c304f9514d330725a8797e77ac

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 18 '20 07:01 cescoffier

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

geoand avatar Jan 18 '20 08:01 geoand

The build is still failing with:

  • Quarkus commit: fd73233b491e6658bc2501ba8fc3096e8cbf7837

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 18 '20 15:01 cescoffier

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

geoand avatar Jan 18 '20 16:01 geoand

The build is still failing with:

  • Quarkus commit: fd73233b491e6658bc2501ba8fc3096e8cbf7837

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 18 '20 20:01 cescoffier

The build is still failing with:

  • Quarkus commit: 2532cfb6ce0b03918efe66d3c22240169c158e76

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 19 '20 02:01 cescoffier

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 avatar Jan 19 '20 08:01 cescoffier

The build is still failing with:

  • Quarkus commit: 2532cfb6ce0b03918efe66d3c22240169c158e76

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 19 '20 09:01 cescoffier

The build is still failing with:

  • Quarkus commit: 2532cfb6ce0b03918efe66d3c22240169c158e76

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 19 '20 14:01 cescoffier

The build is still failing with:

  • Quarkus commit: 2532cfb6ce0b03918efe66d3c22240169c158e76

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 19 '20 20:01 cescoffier

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

geoand avatar Jan 19 '20 21:01 geoand

The build is still failing with:

  • Quarkus commit: 2532cfb6ce0b03918efe66d3c22240169c158e76

  • Quickstarts commit: 4985db39034ff05309fc17227daaa8f8fab27e70

  • Link to build: https://github.com/quarkusio/quarkus-quickstarts/actions

cescoffier avatar Jan 20 '20 03:01 cescoffier

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

cescoffier avatar Jan 20 '20 06:01 cescoffier