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 kube-vip to v0.5.0 and go version to v1.18 #679

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

Madhur97
Copy link
Contributor

@Madhur97 Madhur97 commented Aug 19, 2022

What this PR does / why we need it:
This PR updates kubevip to v0.5.0
Update to golang to v1.18, as it is the minimum version supported by kubevip v0.5.0
Updated golangci-lint to v1.46.2 as supported by golang v1.18
Removed a redundant statement in installer/bundle_downloader_test.go as suggested by golangci-lint.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #670

Special notes for your reviewer

@Madhur97 Madhur97 changed the title Update kube-vip [WIP] Update kube-vip Aug 19, 2022
@Madhur97 Madhur97 force-pushed the kube-vip branch 4 times, most recently from 57f628c to 161363e Compare August 19, 2022 09:24
@Madhur97 Madhur97 marked this pull request as draft August 19, 2022 09:25
@Madhur97 Madhur97 force-pushed the kube-vip branch 12 times, most recently from 259b637 to a48a9a7 Compare August 21, 2022 19:33
@Madhur97 Madhur97 force-pushed the kube-vip branch 4 times, most recently from a8659ef to a85e3d1 Compare August 24, 2022 05:11
@codecov-commenter
Copy link

Codecov Report

Merging #679 (b961e16) into main (c1b6475) will increase coverage by 0.14%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #679      +/-   ##
==========================================
+ Coverage   68.13%   68.27%   +0.14%     
==========================================
  Files          20       20              
  Lines        2068     2068              
==========================================
+ Hits         1409     1412       +3     
+ Misses        546      544       -2     
+ Partials      113      112       -1     
Impacted Files Coverage Δ
agent/reconciler/host_reconciler.go 81.54% <0.00%> (ø)
...s/infrastructure/bootstrapkubeconfig_controller.go 58.49% <0.00%> (+5.66%) ⬆️

@Madhur97 Madhur97 marked this pull request as ready for review August 24, 2022 05:51
@Madhur97 Madhur97 requested a review from anusha94 August 24, 2022 05:52
@Madhur97 Madhur97 changed the title [WIP] Update kube-vip Update kube-vip Aug 24, 2022
Copy link
Contributor

@shamsher31 shamsher31 left a comment

Choose a reason for hiding this comment

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

I think the Go upgrade should be separate PR as per the PR title?

@sachinkumarsingh092
Copy link
Contributor

@shamsher31 I think it's done due to the minimum go version requirement for kube-vip update being v1.18.

Copy link
Contributor

@sachinkumarsingh092 sachinkumarsingh092 left a comment

Choose a reason for hiding this comment

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

lgtm

@Madhur97
Copy link
Contributor Author

I think the Go upgrade should be separate PR as per the PR title?

As @sachinkumarsingh092 also added in other comment, the Go version is updated because it is needed by kubevip v0.5.0. We can update the PR title to include information about Go version update.

Copy link
Contributor

@shubham14bajpai shubham14bajpai left a comment

Choose a reason for hiding this comment

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

LGTM

@Madhur97 Madhur97 removed the request for review from anusha94 August 26, 2022 12:10
@Madhur97 Madhur97 changed the title Update kube-vip Update kube-vip to v0.5.0 and go version to v1.18 Aug 26, 2022
Copy link
Contributor

@anusha94 anusha94 left a comment

Choose a reason for hiding this comment

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

lgtm

@anusha94 anusha94 merged commit fa482ab into vmware-tanzu:main Aug 29, 2022
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.

Update kube-vip to v0.5.0
7 participants