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

Bump CAPZ version to v1.6.1 #4255

Merged
merged 2 commits into from
Jan 11, 2023
Merged

Bump CAPZ version to v1.6.1 #4255

merged 2 commits into from
Jan 11, 2023

Conversation

randomvariable
Copy link
Contributor

@randomvariable randomvariable commented Jan 7, 2023 •
edited
Loading

What this PR does / why we need it

Redo of #4244 on a non-forked branch.

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

Release note

Cluster API Provider Azure updated to v1.6.1

Additional information

Special notes for your reviewer

@randomvariable randomvariable requested review from a team as code owners January 7, 2023 09:06
@github-actions
Copy link

github-actions bot commented Jan 7, 2023

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

@codecov
Copy link

codecov bot commented Jan 7, 2023 •
edited
Loading

Codecov Report

Merging #4255 (e345d65) into main (504445f) will decrease coverage by 0.89%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4255      +/-   ##
==========================================
- Coverage   49.40%   48.51%   -0.90%     
==========================================
  Files         451      481      +30     
  Lines       44755    46875    +2120     
==========================================
+ Hits        22111    22741     +630     
- Misses      20560    22000    +1440     
- Partials     2084     2134      +50
Impacted Files Coverage Δ
...ons/controllers/packageinstallstatus_controller.go 79.15% <0.00%> (ø)
cmd/cli/plugin/cluster/main.go 0.00% <0.00%> (ø)
cmd/cli/plugin/cluster/list.go 11.36% <0.00%> (ø)
cmd/cli/plugin/isolated-cluster/main.go 0.00% <0.00%> (ø)
cmd/cli/plugin/cluster/machinehealthcheck.go 100.00% <0.00%> (ø)
cmd/cli/plugin/cluster/available_upgrade.go 16.32% <0.00%> (ø)
.../cli/plugin/cluster/get_machinehealthcheck_node.go 9.30% <0.00%> (ø)
...olated-cluster/imagepushop/publishimagesfromtar.go 73.17% <0.00%> (ø)
cmd/cli/plugin/cluster/delete.go 12.50% <0.00%> (ø)
cmd/cli/plugin/cluster/set_machinehealthcheck.go 23.33% <0.00%> (ø)
... and 23 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link

github-actions bot commented Jan 9, 2023

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

github-actions bot commented Jan 9, 2023

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

github-actions bot commented Jan 9, 2023

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

github-actions bot commented Jan 9, 2023

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

randomvariable commented Jan 9, 2023 •
edited
Loading

//usebom:sandbox/v2.1.0-zshippable/442614961592635156/tkg-compatibility:v14

@randomvariable randomvariable mentioned this pull request Jan 9, 2023
Closed
@github-actions
Copy link

github-actions bot commented Jan 9, 2023

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

github-actions bot commented Jan 9, 2023

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

meghanajangi and others added 2 commits January 10, 2023 10:57
@meghanajangi
Bump CAPZ version to v1.6.1
d51ed56 
Signed-off-by: Meghana Jangi <mjangi@vmware.com>
Signed-off-by: Naadir Jeewa <jeewan@vmware.com>
Co-authored-by: Meghana Jangi <mjangi@vmware.com>
Co-authored-by: Naadir Jeewa <jeewan@vmware.com>
CAPZ: Include metadata bump
e345d65 
Signed-off-by: Naadir Jeewa <jeewan@vmware.com>
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/4255/20230110110815/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/v2.1.0-zshippable/442614961592635156/tkg-compatibility:v14

@sonasingh46
Copy link
Contributor

The changes looks good to me and should be good to merge.
The CAPZ integration test failure ain't related to the bump but look something with the addons-manager

@wjun wjun added the ok-to-merge PRs should be labelled with this before merging label Jan 11, 2023
@sonasingh46 sonasingh46 merged commit 75a91bd into main Jan 11, 2023
@sonasingh46 sonasingh46 deleted the bump_capz_v1.6.1 branch January 11, 2023 09:26
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers

@sonasingh46 sonasingh46 sonasingh46 approved these changes

@yastij yastij yastij approved these changes

@wjun wjun wjun approved these changes

Assignees
No one assigned
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@randomvariable @sonasingh46 @wjun @yastij @vmwclabot @meghanajangi