-
Notifications
You must be signed in to change notification settings - Fork 10
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
Getting io/dekorate/helm/config/HelmChartConfigFluentImpl error using Quarkus 3.5.0 #287
Comments
This usually is caused by a compatibility issue with the Quarkus Helm + Quarkus Kubernetes extensions in use. Since I can't reproduce it, can you share a reproducer or provide more details about your project configuration? |
Here is a small reproducer created from the Quarkus app generator (v3.5.x) with the following extensions: <dependency>
<groupId>io.quarkiverse.helm</groupId>
<artifactId>quarkus-helm</artifactId>
<version>1.1.0</version>
</dependency>
<dependency>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-keycloak-admin-client-reactive</artifactId>
</dependency>
<dependency>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-smallrye-stork</artifactId>
</dependency> Run
|
Thanks for the reproducer. Honestly, I have no idea why the classloader is trying to load So, I guess this is caused by the latest Dekorate release. @iocanel does it ring any bell to you? |
Sounds like a version mismatch to me. |
Odd, even if i remove all the dependencies (i also dont use the Keycloak authorizer - that was added when i selected keycloak-admin) and only have Helm I still get the issue. Am I possibly doing something wrong that it works for you but not me? |
Quarkus Helm does not enforce any dekorate specific version, but just uses the provided by Quarkus. |
Sure, but if the Quarkus bom used refers to an earlier version of dekorate it can lead to mismatches. |
I tried the reproducer with Quarkus 3.4.3 and it worked, so I think your reproducer is ok.
I tried Quarkus Helm from main and I saw no issues now. I don't fully understand why it could lead into version mismatch, but you're right. @kjq I will release a new version of Quarkus Helm to fix this issue. Thanks both! |
The problem is most likely with the Sundrio-generated classes which are incompatible across different versions (we removed intermediate classes). |
But I'm not using the class HelmChartConfigFluentImpl, but HelmChartConfigFluent (via the builder). |
It might be early, but building with the Quarkus 3.5.0 release (because Renovate updated it) gives this error that I comes from the Helm extension:
The text was updated successfully, but these errors were encountered: