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

[WIP] Bump Cluster API and providers#3513

Closed
ykakarap wants to merge 8 commits intovmware-tanzu:mainfrom
ykakarap:bump-capi-and-providers
Closed

[WIP] Bump Cluster API and providers#3513
ykakarap wants to merge 8 commits intovmware-tanzu:mainfrom
ykakarap:bump-capi-and-providers

Conversation

@ykakarap
Copy link
Copy Markdown

@ykakarap ykakarap commented Sep 29, 2022

What this PR does / why we need it

This PR bumps the version of ClusterAPI and the providers.

Go dependencies will be merged in #3157

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

Release note


Additional information

Special notes for your reviewer

@ykakarap ykakarap requested a review from a team as a code owner September 29, 2022 23:21
@ykakarap ykakarap requested review from a team September 29, 2022 23:21
@ykakarap ykakarap requested a review from a team as a code owner September 29, 2022 23:21
@github-actions
Copy link
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20220929233642/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
Copy Markdown

github-actions Bot commented Oct 1, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221001002702/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.

@sedefsavas sedefsavas force-pushed the bump-capi-and-providers branch from 940c3e9 to 621dd2d Compare October 3, 2022 22:14
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 3, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221003222749/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.

@wjun
Copy link
Copy Markdown
Contributor

wjun commented Oct 11, 2022

@ykakarap Could you help to bump capz to 1.5.1 that includes a fix for azure clusterclass creation: https://jira.eng.vmware.com/browse/TKG-12985 ?

@ykakarap ykakarap force-pushed the bump-capi-and-providers branch 2 times, most recently from 254e4ea to 283cbf1 Compare October 14, 2022 08:11
@github-actions
Copy link
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221014082041/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
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221014082758/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-and-providers branch from 283cbf1 to 4927f9d Compare October 17, 2022 16:46
@github-actions
Copy link
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221017170040/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
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221017175427/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 requested a review from prkalle as a code owner October 17, 2022 22:51
@github-actions
Copy link
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221017230512/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
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221018201140/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.

Signed-off-by: Ashutosh Kumar <[email protected]>
@ykakarap ykakarap force-pushed the bump-capi-and-providers branch from 091df3f to 39950d3 Compare October 18, 2022 22:35
@ykakarap
Copy link
Copy Markdown
Author

Closing this in favor of PR: #3699

@ykakarap ykakarap closed this Oct 18, 2022
@github-actions
Copy link
Copy Markdown

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3513/20221018224507/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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants