Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Bump Cluster API to v0.3.22 #223

Merged
merged 1 commit into from
Jul 26, 2021
Merged

Bump Cluster API to v0.3.22 #223

merged 1 commit into from
Jul 26, 2021

Conversation

randomvariable
Copy link
Contributor

@randomvariable randomvariable commented Jul 16, 2021 •
edited by ykakarap
Loading

What this PR does / why we need it:
Bumping components. Also ensures go mod dependencies match the provider versions.

Which issue(s) this PR fixes:

Fixes #

Describe testing done for PR:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Cluster API updated to v0.3.22

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 kind label according to what type of issue is being addressed.

@randomvariable randomvariable requested review from anujc25, danniel1205 and a team as code owners July 16, 2021 11:01
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210716111937/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@randomvariable randomvariable mentioned this pull request Jul 16, 2021
Closed
9 tasks
@ykakarap
Copy link

//usebom:sandbox/bom/363935924100290581/tkg-bom:v1.4.0-zshippable

@ykakarap
Copy link

//usebom:sandbox/v1.4.0-zshippable/363935924100290581/tkg-compatibility:v1

@randomvariable randomvariable requested a review from shyaamsn as a code owner July 16, 2021 19:24
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210716193820/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210721101037/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@randomvariable
Copy link
Contributor Author

//usebom:sandbox/v1.4.0-zshippable/363935924100290581/tkg-compatibility:v1

@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210721104519/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@stmcginnis
Copy link
Contributor

@vincepri do we have a CAPI release coming that will include the change away from using the alpine-based haproxy image for the bootstrap cluster?

@dims dims mentioned this pull request Jul 21, 2021
Closed
9 tasks
@randomvariable
Copy link
Contributor Author

@vincepri do we have a CAPI release coming that will include the change away from using the alpine-based haproxy image for the bootstrap cluster?

Yes, will be after the CAPI meeting right now

@randomvariable
Copy link
Contributor Author

/hold
for CAPI v0.3.22

@randomvariable randomvariable changed the title Bump Cluster API to v0.3.21 [wip] Bump Cluster API to v0.3.22 Jul 21, 2021
@randomvariable
Copy link
Contributor Author

@ykakarap , could you do the honours today as I'm probably going to be EOD in an hour or two.

@randomvariable
Copy link
Contributor Author

Also, don't forget to update go.mod for CAPI and infrastructure-docker. These were massively out of date.

@vincepri
Copy link

Tag has been created

@ykakarap
Copy link

@randomvariable I will take care of this :)

@ykakarap ykakarap force-pushed the bump-capi-v0.3.21 branch from 375d1c2 to 2615d4b Compare July 21, 2021 20:49
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210721210517/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@ykakarap
Copy link

//usebom:sandbox/v1.4.0-zshippable/364689159224307150/tkg-compatibility:v1

@ykakarap ykakarap force-pushed the bump-capi-v0.3.21 branch from 2615d4b to b65af12 Compare July 22, 2021 20:23
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210722203658/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@ykakarap ykakarap force-pushed the bump-capi-v0.3.21 branch from b65af12 to 24f241c Compare July 23, 2021 20:09
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210723202659/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@ykakarap ykakarap force-pushed the bump-capi-v0.3.21 branch from 24f241c to a0d93fd Compare July 26, 2021 18:15
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/223/20210726182935/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@ykakarap
Copy link

//usebom:sandbox/v1.4.0-zshippable/365398958941432577/tkg-compatibility:v1

@ykakarap ykakarap changed the title [wip] Bump Cluster API to v0.3.22 Bump Cluster API to v0.3.22 Jul 26, 2021
vuil
vuil approved these changes Jul 26, 2021
View reviewed changes
Copy link
Contributor

@vuil vuil left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@ykakarap ykakarap merged commit 5a42a2b into main Jul 26, 2021
@ykakarap ykakarap deleted the bump-capi-v0.3.21 branch July 26, 2021 23:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@vuil vuil vuil approved these changes

@anujc25 anujc25 Awaiting requested review from anujc25

@danniel1205 danniel1205 Awaiting requested review from danniel1205

@shyaamsn shyaamsn Awaiting requested review from shyaamsn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@randomvariable @ykakarap @stmcginnis @vincepri @vuil