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

Update golang to 1.22 #238

Merged

Conversation

lubronzhan
Copy link
Contributor

@lubronzhan lubronzhan commented May 2, 2024

What this PR does / why we need it:
To update to newest golang 1.22

  • Also updated kubebuilder
  • Switched to use envtest to set up binaries

Which issue(s) this PR fixes:

Fixes #

Describe testing done for PR:

Special notes for your reviewer:

Release note:


New PR Checklist

  • Ensure PR contains only public links or terms
  • Use good commit messages
  • Squash the commits in this branch before merge to preserve our git history
  • If this PR is just an idea or POC, use a Draft PR instead of a full PR
  • Add appropriate labels according to what type of issue is being addressed.

@lubronzhan
Copy link
Contributor Author

Hold upgrading k8s 1.30 because of this kubernetes-sigs/controller-runtime#2788

@lubronzhan lubronzhan force-pushed the topic/lubron/bump_golang_1.22 branch 4 times, most recently from 57d1835 to 3a933b8 Compare May 2, 2024 23:02
@codecov-commenter
Copy link

codecov-commenter commented May 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 31.01%. Comparing base (cf5240c) to head (6ddcc60).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #238      +/-   ##
==========================================
- Coverage   36.89%   31.01%   -5.88%     
==========================================
  Files          20       31      +11     
  Lines        2174     2879     +705     
==========================================
+ Hits          802      893      +91     
- Misses       1314     1916     +602     
- Partials       58       70      +12     

see 11 files with indirect coverage changes

@lubronzhan lubronzhan force-pushed the topic/lubron/bump_golang_1.22 branch 5 times, most recently from 8b0c1bc to 14b94a5 Compare May 3, 2024 06:58
hack/tools/Makefile Outdated Show resolved Hide resolved
@lubronzhan lubronzhan force-pushed the topic/lubron/bump_golang_1.22 branch from 14b94a5 to 35edf5a Compare May 3, 2024 22:10
@lubronzhan
Copy link
Contributor Author

Integration test should pass once this is merged #241

Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>
@lubronzhan lubronzhan force-pushed the topic/lubron/bump_golang_1.22 branch from f2b699f to 6ddcc60 Compare May 6, 2024 23:45
Copy link
Contributor

@flawedmatrix flawedmatrix left a comment

Choose a reason for hiding this comment

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

Looks good to me

@lubronzhan lubronzhan merged commit b928065 into vmware-tanzu:main May 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants