-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Application tests failing w/ Quarkus 3.13.0 + io.smallrye.config:smallrye-config-source-keystore
#42240
Comments
Hmm. |
@radcortez I suspect this is one for you? |
Yes, smallrye/smallrye-config#1082 looks suspicious but it's in Quarkus since 3.9 🤷 CC @dmlloyd |
@radcortez Could you clarify if it's a corner case? Because if not, we might have to respin 3.13. |
Ah - could be related to a "specialty" in how we run time tests. |
Hm - what does |
Looking at the static init in |
But passing the uninitialized |
Might be some combination that we are not aware. The code is already in Quarkus for some time: #39295 I'll have a look. |
It still doesn't explain why it worked in the previous version, considering that the probable causes were already there some time ago. |
The 1st |
Right - all related parts are "old". |
With 3.12.3 I don't see |
This is the stack trace up to
|
Yep! It only happens with 3.13.0 if |
io.smallrye.config:smallrye-config-source-keystore
But there must be something else... from the code, the issue should also happen in older versions. |
You can probably work around this by avoiding the call to the Using the old name triggers the interceptor call which causes the issue (in combination with all the other stuff). |
Unfortunately the workaround doesn't help :( |
Yes, so in theory, if you replace all of the old name references with the new ones, the compatibility should not be triggered. |
Anyway, I'm working on a fix... this was to avoid waiting for one. |
LMK if you need a review or so |
Looks like smallrye/smallrye-config#1205 solves the issue |
Fixes quarkusio#42240 (cherry picked from commit 54a4d3e)
Describe the bug
All
@QuarkusTest
and@QuarkusIntegrationTest
fail with an NPE atSmallRyeConfig.java:751
.Quarkus 3.12.3 worked fine.
Expected behavior
No response
Actual behavior
Full stack trace
How to Reproduce?
git clone https://github.com/projectnessie/nessie.git
git checkout renovate/quarkusplatform
./gradlew :nessie-quarkus:test --tests TestBasicOperations
Output of
uname -a
orver
No response
Output of
java -version
21
Quarkus version or git rev
4.13.0
Build tool (ie. output of
mvnw --version
orgradlew --version
)Gradle 8.9
Additional information
No response
The text was updated successfully, but these errors were encountered: