Skip to content

Upgrade Go to v1.24 #16

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

Merged
merged 23 commits into from
Jun 3, 2025

Conversation

ryotarai
Copy link
Collaborator

No description provided.

@ryotarai ryotarai force-pushed the ryotarai/upgrade-k8s-libraries-v0.32 branch from e6d6aad to e9d0d75 Compare May 13, 2025 00:36
@ryotarai ryotarai changed the title Upgrade k8s libraries v0.32 Upgrade Go to v1.24 May 13, 2025
@ryotarai ryotarai force-pushed the ryotarai/upgrade-k8s-libraries-v0.32 branch 2 times, most recently from a03ddc9 to f92d1f4 Compare May 26, 2025 14:40
@ryotarai ryotarai marked this pull request as ready for review May 27, 2025 01:03
@utam0k utam0k self-assigned this Jun 2, 2025
@utam0k
Copy link
Member

utam0k commented Jun 2, 2025

📝 This PR also supports Kubernetes 1.32.

Comment on lines +63 to +64
// NOTE(ryotarai): The injecting mechanism is removed in https://github.com/kubernetes-sigs/controller-runtime/pull/2134
// For now, the decoder and client are injected manually, but we might want to replace this with sigs.k8s.io/controller-runtime/pkg/builder.WebhookManagedBy
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀 May I ask you to create the issue for addressing this comment?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Created an issue for this: #20

Copy link
Member

@utam0k utam0k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@utam0k utam0k merged commit a1ccba4 into pfnet:master Jun 3, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants