-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
conflict between micrometer and stork #23623
Comments
/cc @ebullient |
@vsevel could you check if removing the client from the MyServiceDiscovery makes the error disappear? |
@michalszynkiewicz |
Is that a case of the discovery client making outbound http requests (that should in theory be measured) before things have been initialized? |
I do not think it is about making the calls (it works when I lazy init the proxies), rather than trying to initialize the rest proxy too early. |
Haven't started looking at it yet but it seems to that stork initialization happens to early |
Not sure if there is anything the micrometer initialization should change... |
fixes quarkusio#23623 (cherry picked from commit ed20b56)
Describe the bug
When I add in the same application the stork extension and the micrometer extension I get an error.
Expected behavior
Should not conflict with each other.
Actual behavior
When I build the application I get:
How to Reproduce?
Get https://github.com/vsevel/stork-issue/tree/micrometer_stork_conflict
Output of
uname -a
orver
No response
Output of
java -version
No response
GraalVM version (if different from Java)
No response
Quarkus version or git rev
2.7.1
Build tool (ie. output of
mvnw --version
orgradlew --version
)No response
Additional information
No response
The text was updated successfully, but these errors were encountered: