Skip to content

chore: upgrade kubernetes dependencies#266

Merged
xcaspar merged 3 commits intochaosblade-io:masterfrom
spencercjh:chore/upgrade
Oct 20, 2025
Merged

chore: upgrade kubernetes dependencies#266
xcaspar merged 3 commits intochaosblade-io:masterfrom
spencercjh:chore/upgrade

Conversation

@spencercjh
Copy link
Member

No description provided.

Signed-off-by: spencercjh <spencercjh@gmail.com>
@spencercjh spencercjh requested review from Copilot and xcaspar October 17, 2025 12:56
@spencercjh spencercjh self-assigned this Oct 17, 2025
@spencercjh spencercjh added the dependencies Pull requests that update a dependency file label Oct 17, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades Kubernetes dependencies from version 0.20.6 to 0.31.0, along with controller-runtime from 0.6.0 to 0.19.4. The changes update API signatures and patterns to align with the newer versions of these dependencies.

  • Updated API signatures to use context.Context instead of stop channels
  • Modified function signatures in controller-runtime to match new type-safe patterns
  • Updated import paths that have been reorganized in newer versions

Reviewed Changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
go.mod Upgraded k8s.io dependencies from 0.20.6 to 0.31.0 and controller-runtime from 0.6.0 to 0.19.4
pkg/webhook/pod/mutator.go Changed decoder field from pointer to value type in admission.Decoder
pkg/hookfs/server.go Updated Start method to accept context.Context instead of stop channel
pkg/controller/chaosblade/controller.go Updated controller methods to accept context.Context parameter and use new Watch API
pkg/apis/chaosblade/v1alpha1/zz_generated.openapi.go Removed autogenerated OpenAPI file
pkg/apis/chaosblade/meta/zz_generated.openapi.go Removed autogenerated OpenAPI file
pkg/apis/chaosblade/v1alpha1/register.go Updated import path for scheme package
cmd/manager/main.go Updated manager creation to use new Cache.Options and webhook.NewServer API
cmd/hookfs/main.go Updated signal handling to use context instead of channel
channel/client.go Updated NewClientFunc signature to match new client.NewClientFunc type

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Signed-off-by: spencercjh <spencercjh@gmail.com>
@spencercjh spencercjh requested a review from Copilot October 17, 2025 13:29
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Signed-off-by: spencercjh <spencercjh@gmail.com>
@xcaspar xcaspar merged commit 3a3986c into chaosblade-io:master Oct 20, 2025
4 checks passed
xcaspar added a commit that referenced this pull request Oct 20, 2025
xcaspar added a commit that referenced this pull request Oct 20, 2025
@xcaspar
Copy link
Member

xcaspar commented Oct 20, 2025

I have reverted this merge to avoid affecting version 1.8.0. Please submit a new PR. Thank you!

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

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants