-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Switch to Java 17 for release action. #2487
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brendandburns The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
hi - a propos of nothing: when's the next release planned? We'd love to have it in time for a release next week so that Spring Cloud Kubernetes can befit from these new AOT hints :) |
Can I ask how Spring Cloud Kubernetes is going to benefit from these? I don't see anything issue wise related to this in the github repo, unless its an internal thing. Thank you. |
@joshlong once this merges the release pipeline in github actions should be fixed and we will then immediately cut a release. I would expect that we should be able to cut a release by Friday at the latest. @wind57 this specific change doesn't benefit spring, but it unblocks our ability to cut a new release [which got broken, but we didn't notice]. In particular the new release will contain: #2457 which improves aot-graalvm support for this library thanks to @joshlong |
thank you so very much! |
oddly when i was trying to import and verify the new snapshot module
is it because the uploaded new module takes some time to be present in the central repo? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
verified, merging in
/lgtm
@yue9944882 in my experience it takes ~24h for a package to get uploaded from sonatype to Maven central. --brendan |
@joshlong 17.0.0 release has been cut and pushed to sonatype. I would expect it in Maven Central in ~24h |
Congrats! And thank you |
@yue9944882 @joshlong when I look at maven central, I'm not certain that I see the right files: https://mvnrepository.com/artifact/io.kubernetes/client-java-spring-integration Only shows 16.0.2 Can you take a look and see? I'm worried that we may need to do some more modifications to either the pom or the github action. (or am I not looking in the right place?) The client itself is there: |
so some of the modules are being deployed but others are not? i see the |
à propos: is there any output from during the release that gives a clue as to what may have happened? |
hah. i think we got had by a cache issue. different index on a different site shows it's there :) if I add the following to a spring boot 3 ga app with no repositories specified, and nothing in local
indeed, if I add that, and then a few beans, and then run
it builds a native image and that works! what a time to be alive :) |
@joshlong great to get the confirmation that it is working and indeed I see them in maven central now. Must have just been replication lag. |
cc @yue9944882