-
Notifications
You must be signed in to change notification settings - Fork 37
Upgrade cluster-api to v1.9.6 #427
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
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vishesh92 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
✅ Deploy Preview for kubernetes-sigs-cluster-api-cloudstack ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #427 +/- ##
==========================================
+ Coverage 25.66% 26.98% +1.31%
==========================================
Files 59 72 +13
Lines 5563 6838 +1275
==========================================
+ Hits 1428 1845 +417
- Misses 3996 4832 +836
- Partials 139 161 +22 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Test Results : (tid-654)
|
/lgtm |
Issue #, if available:
Fixes #420
Description of changes:
Followed t he changes mentioned in the CAPI documentation and some changes other minor changes to make it compatible with latest version of CAPI.
https://cluster-api.sigs.k8s.io/developer/providers/migrations/v1.6-to-v1.7
https://cluster-api.sigs.k8s.io/developer/providers/migrations/v1.7-to-v1.8
https://cluster-api.sigs.k8s.io/developer/providers/migrations/v1.8-to-v1.9
Copilot generated summary
This pull request includes various updates and improvements across multiple files, focusing on configuration changes, dependency updates, and code refactoring. Below is a summary of the most important changes:
Configuration Updates
.golangci.yml
: Removedskip-dirs
and addedtests: false
underrun
, and moved excluded directories toexclude-dirs
underissues
. [1] [2]Dependency Updates
Makefile
: Updated thego mod tidy
command to use compatibility version 1.22 instead of 1.21.Code Refactoring
Makefile
: Simplified thegenerate-conversion
command by removing--input-dirs
and using--output-file
directly.api/v1beta2/cloudstackmachine_types_test.go
: Replacedpointer
withptr
for utility functions. [1] [2] [3]Documentation and Metadata Updates
config/crd/bases/infrastructure.cluster.x-k8s.io_cloudstackaffinitygroups.yaml
: Updatedcontroller-gen.kubebuilder.io/version
annotation and improved descriptions for better readability. [1] [2] [3] [4] [5] [6] [7]config/crd/bases/infrastructure.cluster.x-k8s.io_cloudstackclusters.yaml
: Similar updates to annotations and descriptions as in the affinity groups file. [1] [2] [3] [4] [5]These changes collectively enhance the configuration management, update dependencies to newer versions, and improve code readability and maintainability.
Testing performed:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.