Skip to content

Conversation

@petr-muller
Copy link

@petr-muller petr-muller commented Oct 13, 2025

Summary

Updates Kubernetes dependencies from v0.28.3 to v0.32.9 to support Kubernetes 1.32.

Changes

Dependency Updates

  • k8s.io/api: v0.28.3 → v0.32.9
  • k8s.io/apimachinery: v0.28.3 → v0.32.9
  • k8s.io/client-go: v0.28.3 → v0.32.9
  • sigs.k8s.io/controller-runtime: v0.16.5 → v0.19.4
  • sigs.k8s.io/prow: updated to latest (v0.0.0-20251011135252-7dbcd42f9afa)
  • Go: 1.23.0 → 1.24.0 (required by prow dependency)

Code Changes

Controller-runtime v0.18.0 introduced breaking changes with generics support for Sources, Event Handlers, and Predicates. Updated the codebase to use the new typed APIs:

cleaner/v2/cleaner.go:

  • Updated source.Kind() call to pass handler as parameter with typed generics

cmd/boskos/boskos.go:

  • Changed event channels from event.GenericEvent to event.TypedGenericEvent[ctrlruntimeclient.Object]
  • Converted constHandler() to generic function returning handler.TypedEventHandler[T, reconcile.Request]
  • Updated resourceUpdatePredicate() to use predicate.TypedPredicate[*crds.ResourceObject] and predicate.TypedFuncs
  • Modified all source.Kind() and source.Channel() calls to new API where handlers and predicates are passed as parameters

Test plan

  • All unit tests pass (203 tests in 12.178s)
  • Build succeeds for all commands
  • Runtime testing in actual Kubernetes 1.32 cluster

🤖 Generated with Claude Code

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

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: petr-muller
Once this PR has been reviewed and has the lgtm label, please assign cjwagner for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details 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 sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Oct 13, 2025
@petr-muller petr-muller changed the title Bump Kubernetes dependencies to v1.32 WIP: Bump Kubernetes dependencies to v1.32 Oct 13, 2025
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 13, 2025
Updates k8s.io/* dependencies from v0.28.3 to v0.32.9 and related
dependencies to maintain compatibility.

Key dependency updates:
- k8s.io/api: v0.28.3 → v0.32.9
- k8s.io/apimachinery: v0.28.3 → v0.32.9
- k8s.io/client-go: v0.28.3 → v0.32.9
- sigs.k8s.io/controller-runtime: v0.16.5 → v0.19.4
- sigs.k8s.io/prow: updated to latest
- Go: 1.23.0 → 1.24.0 (required by prow)

Breaking changes in controller-runtime v0.18.0 introduced generics
support for Sources, Event Handlers, and Predicates. Updated code
to use the new typed APIs:

- Updated source.Kind() and source.Channel() to pass handlers and
  predicates as parameters instead of to ctrl.Watch()
- Converted handlers to use TypedEventHandler with generic type
  parameters
- Converted predicates to use TypedPredicate and TypedFuncs
- Changed event channels from GenericEvent to TypedGenericEvent

All tests pass with the updated dependencies.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@petr-muller
Copy link
Author

/test pull-boskos-build-test-verify

1 similar comment
@petr-muller
Copy link
Author

/test pull-boskos-build-test-verify

@krzyzacy
Copy link
Contributor

/retest

@petr-muller
Copy link
Author

/test pull-boskos-build-test-verify\

@k8s-ci-robot
Copy link
Contributor

@petr-muller: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test pull-boskos-build-test-verify

Use /test all to run all jobs.

Details

In response to this:

/test pull-boskos-build-test-verify\

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@petr-muller
Copy link
Author

/test pull-boskos-build-test-verify

@krzyzacy
Copy link
Contributor

krzyzacy commented Jan 5, 2026

@krzyzacy
Copy link
Contributor

krzyzacy commented Jan 7, 2026

/test pull-boskos-build-test-verify

(will take a closer look if it times out again and not a flake)

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 8, 2026
@k8s-ci-robot
Copy link
Contributor

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants