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

Apply OpenYurt to support K8s v1.28 #2047

Merged
merged 1 commit into from
May 16, 2024

Conversation

rambohe-ch
Copy link
Member

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage

/kind feature

What this PR does / why we need it:

  1. upgrade k8s.io/xxx to v0.28.9
  2. upgrade controller-runtime to v0.16.5
  3. upgrade go version from v1.18 to v1.20
  4. upgrade kind to v0.22.0 to local up K8s v1.28 cluster.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


other Note

@rambohe-ch rambohe-ch force-pushed the upgrade-k8s-io branch 5 times, most recently from 879610f to d4639d4 Compare May 16, 2024 03:56
2. upgrade controller-runtime to v0.16.5
Copy link

sonarcloud bot commented May 16, 2024

Quality Gate Passed Quality Gate passed

Issues
15 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.1% Duplication on New Code

See analysis details on SonarCloud

Copy link

codecov bot commented May 16, 2024

Codecov Report

Attention: Patch coverage is 62.05924% with 269 lines in your changes are missing coverage. Please review.

Project coverage is 55.51%. Comparing base (37addb5) to head (8c5712e).

Files Patch % Lines
pkg/util/iptables/iptables.go 55.31% 40 Missing and 2 partials ⚠️
pkg/util/iptables/testing/fake.go 76.12% 26 Missing and 11 partials ⚠️
pkg/util/iptables/testing/parse.go 80.68% 26 Missing and 2 partials ⚠️
...r/controller/csrapprover/csrapprover_controller.go 11.76% 15 Missing ⚠️
...ebhook/yurtappset/v1beta1/yurtappset_validation.go 30.00% 14 Missing ⚠️
.../daemonpodupdater/daemon_pod_updater_controller.go 0.00% 12 Missing ⚠️
...rdinator/delegatelease/delegatelease_controller.go 0.00% 9 Missing ⚠️
...rvice/gateway_internal_service_enqueue_handlers.go 0.00% 8 Missing ⚠️
...n/gatewaypickup/gateway_pickup_enqueue_handlers.go 0.00% 8 Missing ⚠️
...service/gateway_public_service_enqueue_handlers.go 0.00% 8 Missing ⚠️
... and 35 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2047      +/-   ##
==========================================
+ Coverage   55.37%   55.51%   +0.14%     
==========================================
  Files         185      186       +1     
  Lines       17821    18057     +236     
==========================================
+ Hits         9869    10025     +156     
- Misses       6934     7006      +72     
- Partials     1018     1026       +8     
Flag Coverage Δ
unittests 55.51% <62.05%> (+0.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@YTGhost YTGhost left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@YTGhost YTGhost merged commit 4b0e950 into openyurtio:master May 16, 2024
13 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