Skip to content
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

Fail build if GraalVM reachability metadata repository is enabled, but no repository has been configured. #449

Open
amizurov opened this issue Jun 9, 2023 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@amizurov
Copy link

amizurov commented Jun 9, 2023

Is your feature request related to a problem? Please describe.
Hi, folks we are happy with using GraalVM native-image feature for our microservices, but we faced with some inconvenient behaviour with graalvm-reachability data. The main problem here is that our builds can run under a proxy which required an installed CA certificate. If someone forgets about this, the build will be green and everything will look fine, only this error will be in the log:

[ERROR] Failed to download from https://github.com/oracle/graalvm-reachability-metadata/releases/download/0.3.1/graalvm-reachability-metadata-0.3.1.zip: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
[WARNING] GraalVM reachability metadata repository is enabled, but no repository has been configured

But when you run a program or an integration test, you will find that some classes are not available due to lack of reachability-metadata, in which case it is not easy to find the root cause.

Plugin config snippet:

...
<configuration>
      <classesDirectory>${project.build.outputDirectory}</classesDirectory>
      <metadataRepository>
             <version>0.3.1</version>
             <enabled>true</enabled>
       </metadataRepository>
</configuration>
...

Describe the solution you'd like
It's would be nice if we could to fail the build if reachability metadata repository is enabled but not configured.
Maybe such option is already present but we didn't find it. Thanks.

@amizurov amizurov added the enhancement New feature or request label Jun 9, 2023
@dpozinen
Copy link

dpozinen commented Jun 9, 2023

This would be a great addition, because this metadata is a mandatory requirement and is basically akin to a missing dependency. I would go so far as to say this should be the default behavior

@vjovanov vjovanov added bug Something isn't working and removed enhancement New feature or request labels Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants