Skip to content
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

Merged
merged 1 commit into from
Dec 8, 2022
Merged

Conversation

brendandburns
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 6, 2022
@k8s-ci-robot
Copy link
Contributor

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 6, 2022
@brendandburns brendandburns mentioned this pull request Dec 6, 2022
@joshlong
Copy link
Contributor

joshlong commented Dec 6, 2022

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 :)

@wind57
Copy link
Contributor

wind57 commented Dec 6, 2022

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.

@brendandburns
Copy link
Contributor Author

@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

@joshlong
Copy link
Contributor

joshlong commented Dec 7, 2022

@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!

@yue9944882
Copy link
Member

yue9944882 commented Dec 7, 2022

oddly when i was trying to import and verify the new snapshot module client-java-spring-aot-integration (just in case bumping to jdk17 breaks any compatibility), the maven central snapshot repo says it doesn't exist:

Could not find artifact io.kubernetes:client-java-spring-aot-integration:jar:17.0.0-SNAPSHOT -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.

is it because the uploaded new module takes some time to be present in the central repo?

Copy link
Member

@yue9944882 yue9944882 left a 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

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 8, 2022
@k8s-ci-robot k8s-ci-robot merged commit 8044d91 into master Dec 8, 2022
@brendandburns
Copy link
Contributor Author

@yue9944882 in my experience it takes ~24h for a package to get uploaded from sonatype to Maven central.

--brendan

@brendandburns
Copy link
Contributor Author

@joshlong 17.0.0 release has been cut and pushed to sonatype. I would expect it in Maven Central in ~24h

@joshlong
Copy link
Contributor

joshlong commented Dec 8, 2022

Congrats! And thank you

@brendandburns
Copy link
Contributor Author

brendandburns commented Dec 10, 2022

@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:
https://mvnrepository.com/artifact/io.kubernetes/client-java/17.0.0

@joshlong
Copy link
Contributor

joshlong commented Dec 10, 2022

so some of the modules are being deployed but others are not? i see the client-java-spring-aot-integration in the modules stanza in the parent. i just did a mvn -DskipTests install and I see the spring aot integration deployed to my local ~/.m2/repository. so that's a start. guess I need to try to understand the deployment pipeline a bit more, cuz nothing's leaping out to me here in the pom.xm files.

@joshlong
Copy link
Contributor

à propos: is there any output from during the release that gives a clue as to what may have happened?

@joshlong
Copy link
Contributor

joshlong commented Dec 10, 2022

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 ~/.m2/repository, it seems to work:

<dependency>
	<groupId>io.kubernetes</groupId>
	<artifactId>client-java-spring-aot-integration</artifactId>
	<version>17.0.0</version>
</dependency>

indeed, if I add that, and then a few beans, and then run

./mvnw -DskipTests -Pnative native:compile  && ./target/k8s 

it builds a native image and that works! what a time to be alive :)

@brendandburns
Copy link
Contributor Author

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants