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

Incorrect error page in dev mode when quarkus-kubernetes is in the classpath #5174

Closed
emmanuelbernard opened this issue Nov 4, 2019 · 5 comments · Fixed by #5181
Closed
Assignees
Labels
kind/bug Something isn't working
Milestone

Comments

@emmanuelbernard
Copy link
Member

Create a project with quarkus-kubernetes as extension.

Then start in dev mode.

Then create a new java file and forget to add a mandatory import (e.g. List getAll() without importing java.util;.List). You then get the following nasty error

The web page shows

java.lang.RuntimeException: Compilation failed[warning: No SupportedSourceVersion annotation found on io.sundr.transform.internal.VelocityTransformationProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.transform.internal.VelocityTransformationProcessor' less than -source '1.8', warning: No SupportedSourceVersion annotation found on io.sundr.resourcecify.internal.processor.ResourcecifyProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.resourcecify.internal.processor.ResourcecifyProcessor' less than -source '1.8', warning: No SupportedSourceVersion annotation found on io.sundr.builder.internal.processor.BuildableProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.builder.internal.processor.BuildableProcessor' less than -source '1.8', warning: No SupportedSourceVersion annotation found on io.sundr.builder.internal.processor.ExternalBuildableProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.builder.internal.processor.ExternalBuildableProcessor' less than -source '1.8', /work/fruits-app/src/main/java/com/example/FruitResource.java:36: error: cannot find symbol

The stacktrace below has been reversed to show the root cause first. Click Here to see the original stacktrace

java.lang.RuntimeException: Compilation failed[warning: No SupportedSourceVersion annotation found on io.sundr.transform.internal.VelocityTransformationProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.transform.internal.VelocityTransformationProcessor' less than -source '1.8', warning: No SupportedSourceVersion annotation found on io.sundr.resourcecify.internal.processor.ResourcecifyProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.resourcecify.internal.processor.ResourcecifyProcessor' less than -source '1.8', warning: No SupportedSourceVersion annotation found on io.sundr.builder.internal.processor.BuildableProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.builder.internal.processor.BuildableProcessor' less than -source '1.8', warning: No SupportedSourceVersion annotation found on io.sundr.builder.internal.processor.ExternalBuildableProcessor, returning RELEASE_6., warning: Supported source version 'RELEASE_6' from annotation processor 'io.sundr.builder.internal.processor.ExternalBuildableProcessor' less than -source '1.8', /work/fruits-app/src/main/java/com/example/FruitResource.java:36: error: cannot find symbol
    public List<FruityVice> getAll() {
           ^
  symbol:   class List
  location: class com.example.FruitResource]
	at io.quarkus.dev.JavaCompilationProvider.compile(JavaCompilationProvider.java:59)
	at io.quarkus.dev.ClassLoaderCompiler.compile(ClassLoaderCompiler.java:181)
	at io.quarkus.dev.RuntimeUpdatesProcessor.checkForChangedClasses(RuntimeUpdatesProcessor.java:186)
	at io.quarkus.dev.RuntimeUpdatesProcessor.doScan(RuntimeUpdatesProcessor.java:121)
	at io.quarkus.vertx.http.deployment.devmode.VertxHotReplacementSetup$1.handle(VertxHotReplacementSetup.java:51)
	at io.quarkus.vertx.http.deployment.devmode.VertxHotReplacementSetup$1.handle(VertxHotReplacementSetup.java:43)
	at io.vertx.core.impl.ContextImpl.lambda$executeBlocking$2(ContextImpl.java:316)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.lang.Thread.run(Thread.java:748)

Does annotation processors leaks somehow. CC @geoand

@emmanuelbernard emmanuelbernard added the kind/bug Something isn't working label Nov 4, 2019
@geoand
Copy link
Contributor

geoand commented Nov 4, 2019

@iocanel we stumbled upon this one. Is perhaps one of the jar dependencies adding annotation processing stuff by mistake?

@iocanel
Copy link
Contributor

iocanel commented Nov 4, 2019

Can you assign the issue to me?

@iocanel
Copy link
Contributor

iocanel commented Nov 4, 2019

VelocityTransformationProcessor is a dekorate transitive.
It should be marked as compile and optional to avoid issues like this.
Apparently, stopped being the case possibly when we added support for Openshift.

@geoand
Copy link
Contributor

geoand commented Nov 4, 2019

I can't seem to assign it to you, but consider it your

@iocanel
Copy link
Contributor

iocanel commented Nov 4, 2019

I created a pull request that should fix the issue for now.
And also verified that I don't get this issue following the steps @emmanuelbernard described.

@gsmet gsmet added the backport? label Nov 4, 2019
gsmet added a commit that referenced this issue Nov 4, 2019
fix (#5174): Add sundrio exclusion in kubernetes extension.
@gsmet gsmet added this to the 0.28.1 milestone Nov 4, 2019
@gsmet gsmet removed the backport? label Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants