-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
quarkus-kubernetes / quarkus-openshift extension pulls in additional 10+ MB of dependencies #15705
Comments
/cc @geoand |
Hm... I wonder if this is related to the change @aloubyansky made with regards to stricter dependency management |
Could be |
Happy Sunday guys :)! |
I'll have to take a closer look tomorrow |
I don't know what dependency requirements are in this case to suggest exclusions. |
The quarkus-kubernetes and quarkus-openshift extensions should not result in any extra dependencies added to the runtime classpath. |
Thanks @geoand Let me know if I can help with moving some dependencies around w/o breaking the validation. |
Will do!
I'll check tomorrow and let you know
…On Sun, Mar 14, 2021, 14:50 Alexey Loubyansky ***@***.***> wrote:
Thanks @geoand <https://github.com/geoand> Let me know if I can help with
moving some dependencies around w/o breaking the validation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#15705 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABBMDP3MWMUB7VC4RHTIM6LTDSWK5ANCNFSM4ZE5LOCA>
.
|
#15706 is how I noticed quarkus-kubernetes / quarkus-openshift problem. It seems there are more extensions which are causing increased memory requirements for native build with Quarkus master. |
#15716 should take care of the issue |
Ensure that quarkus-kubernetes does not pollute the runtime classpath
Nice, numbers are back to 1.12.2.Final like numbers with the latest Quarkus master
|
Thanks for verifying @rsvoboda! |
quarkus-kubernetes / quarkus-openshift extension pulls in additional 10+ MB of dependencies
Affected version: Quarkus master
Quarkus 1.12.2.Final
Quarkus master
Reproducer:
A lot of new io.fabric8 deps appear in dependency tree and
target/quarkus-app/
directory:The text was updated successfully, but these errors were encountered: