chore: bump CAPI to v1.4.4

Signed-off-by: Richard Case <[email protected]>

Author: richardcase

Makefile

@@ -75,7 +75,7 @@ GOLANGCI_LINT_VER := v1.52.1
 GOLANGCI_LINT_BIN := golangci-lint
 GOLANGCI_LINT := $(TOOLS_BIN_DIR)/$(GOLANGCI_LINT_BIN)-$(GOLANGCI_LINT_VER)
 
-KIND_VER := v0.17.0
+KIND_VER := v0.20.0
 KIND_BIN := kind
 KIND := $(TOOLS_BIN_DIR)/$(KIND_BIN)-$(KIND_VER)
 
@@ -102,7 +102,7 @@ SETUP_ENVTEST_VER := v0.0.0-20211110210527-619e6b92dab9
 SETUP_ENVTEST_BIN := setup-envtest
 SETUP_ENVTEST := $(TOOLS_BIN_DIR)/$(SETUP_ENVTEST_BIN)
 
-GO_APIDIFF_VER := v0.4.0
+GO_APIDIFF_VER := v0.6.0
 GO_APIDIFF_BIN := go-apidiff
 GO_APIDIFF := $(TOOLS_BIN_DIR)/$(GO_APIDIFF_BIN)
 
@@ -473,7 +473,7 @@ create-management-cluster: $(KUSTOMIZE) $(ENVSUBST) $(KIND) $(KUBECTL)
 	./hack/install-cert-manager.sh
 
 	# Deploy CAPI
-	curl --retry $(CURL_RETRIES) -sSL https://github.com/kubernetes-sigs/cluster-api/releases/download/v1.3.8/cluster-api-components.yaml | $(ENVSUBST) | $(KUBECTL) apply -f -
+	curl --retry $(CURL_RETRIES) -sSL https://github.com/kubernetes-sigs/cluster-api/releases/download/v1.4.4/cluster-api-components.yaml | $(ENVSUBST) | $(KUBECTL) apply -f -
 
 	# Deploy CAPG
 	$(KIND) load docker-image $(CONTROLLER_IMG)-$(ARCH):$(TAG) --name=clusterapi

Tiltfile

@@ -18,9 +18,9 @@ settings = {
     "deploy_cert_manager": True,
     "preload_images_for_kind": True,
     "kind_cluster_name": "capg",
-    "capi_version": "v1.3.8",
+    "capi_version": "v1.4.4",
     "cert_manager_version": "v1.11.0",
-    "kubernetes_version": "v1.25.10",
+    "kubernetes_version": "v1.26.1",
 }
 
 keys = ["GCP_B64ENCODED_CREDENTIALS"]

api/v1alpha3/doc.go

@@ -14,6 +14,9 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package v1alpha3 contains the v1alpha3 API implementation.
+//
+// Deprecated: This package will be removed in one of the next releases
 package v1alpha3
 
 // +k8s:conversion-gen=sigs.k8s.io/cluster-api-provider-gcp/api/v1beta1

api/v1alpha3/gcpcluster_types.go

@@ -18,7 +18,7 @@ package v1alpha3
 
 import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
-	clusterv1 "sigs.k8s.io/cluster-api/api/v1alpha3"
+	clusterv1 "sigs.k8s.io/cluster-api/api/v1alpha3" //nolint: staticcheck
 )
 
 const (

api/v1alpha4/doc.go

@@ -14,6 +14,9 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// Package v1alpha4 contains the v1alpha4 API implementation.
+//
+// Deprecated: This package will be removed in one of the next releases.
 package v1alpha4
 
 // +k8s:conversion-gen=sigs.k8s.io/cluster-api-provider-gcp/api/v1beta1

api/v1alpha4/gcpcluster_types.go

@@ -18,7 +18,7 @@ package v1alpha4
 
 import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
-	clusterv1 "sigs.k8s.io/cluster-api/api/v1alpha4"
+	clusterv1 "sigs.k8s.io/cluster-api/api/v1alpha4" //nolint: staticcheck
 )
 
 const (

cloud/scope/machine.go

@@ -147,12 +147,12 @@ func (m *MachineScope) Role() string {
 
 // GetInstanceID returns the GCPMachine instance id by parsing Spec.ProviderID.
 func (m *MachineScope) GetInstanceID() *string {
-	parsed, err := noderefutil.NewProviderID(m.GetProviderID())
+	parsed, err := noderefutil.NewProviderID(m.GetProviderID()) //nolint: staticcheck
 	if err != nil {
 		return nil
 	}
 
-	return pointer.StringPtr(parsed.ID())
+	return pointer.String(parsed.ID()) //nolint: staticcheck
 }
 
 // GetProviderID returns the GCPMachine providerID from the spec.
@@ -171,7 +171,7 @@ func (m *MachineScope) GetProviderID() string {
 // SetProviderID sets the GCPMachine providerID in spec.
 func (m *MachineScope) SetProviderID() {
 	providerID, _ := providerid.New(m.ClusterGetter.Project(), m.Zone(), m.Name())
-	m.GCPMachine.Spec.ProviderID = pointer.StringPtr(providerID.String())
+	m.GCPMachine.Spec.ProviderID = pointer.String(providerID.String())
 }
 
 // GetInstanceStatus returns the GCPMachine instance status.
@@ -191,7 +191,7 @@ func (m *MachineScope) SetReady() {
 
 // SetFailureMessage sets the GCPMachine status failure message.
 func (m *MachineScope) SetFailureMessage(v error) {
-	m.GCPMachine.Status.FailureMessage = pointer.StringPtr(v.Error())
+	m.GCPMachine.Status.FailureMessage = pointer.String(v.Error())
 }
 
 // SetFailureReason sets the GCPMachine status failure reason.
@@ -253,7 +253,7 @@ func (m *MachineScope) InstanceAdditionalDiskSpec() []*compute.AttachedDisk {
 		additionalDisk := &compute.AttachedDisk{
 			AutoDelete: true,
 			InitializeParams: &compute.AttachedDiskInitializeParams{
-				DiskSizeGb: pointer.Int64PtrDerefOr(disk.Size, 30),
+				DiskSizeGb: pointer.Int64Deref(disk.Size, 30),
 				DiskType:   path.Join("zones", m.Zone(), "diskTypes", string(*disk.DeviceType)),
 			},
 		}
@@ -341,7 +341,7 @@ func (m *MachineScope) InstanceSpec(log logr.Logger) *compute.Instance {
 		Labels: infrav1.Build(infrav1.BuildParams{
 			ClusterName: m.ClusterGetter.Name(),
 			Lifecycle:   infrav1.ResourceLifecycleOwned,
-			Role:        pointer.StringPtr(m.Role()),
+			Role:        pointer.String(m.Role()),
 			// TODO(vincepri): Check what needs to be added for the cloud provider label.
 			Additional: m.ClusterGetter.AdditionalLabels().AddLabels(m.GCPMachine.Spec.AdditionalLabels),
 		}),

cloud/scope/managedcontrolplane.go

@@ -166,7 +166,7 @@ func (s *ManagedControlPlaneScope) GetAllNodePools(ctx context.Context) ([]infra
 	if s.AllManagedMachinePools == nil || len(s.AllManagedMachinePools) == 0 {
 		listOptions := []client.ListOption{
 			client.InNamespace(s.GCPManagedControlPlane.Namespace),
-			client.MatchingLabels(map[string]string{clusterv1.ClusterLabelName: s.Cluster.Name}),
+			client.MatchingLabels(map[string]string{clusterv1.ClusterNameLabel: s.Cluster.Name}),
 		}
 
 		machinePoolList := &clusterv1exp.MachinePoolList{}

cloud/services/compute/instances/reconcile.go

@@ -136,7 +136,7 @@ func (s *Service) createOrGetInstance(ctx context.Context) (*compute.Instance, e
 	instanceKey := meta.ZonalKey(instanceName, s.scope.Zone())
 	instanceSpec.Metadata.Items = append(instanceSpec.Metadata.Items, &compute.MetadataItems{
 		Key:   "user-data",
-		Value: pointer.StringPtr(bootstrapData),
+		Value: pointer.String(bootstrapData),
 	})
 
 	log.V(2).Info("Looking for instance", "name", instanceName, "zone", s.scope.Zone())

clusterctl-settings.json

@@ -2,6 +2,6 @@
   "name": "infrastructure-gcp",
   "config": {
     "componentsFile": "infrastructure-components.yaml",
-    "nextVersion": "v1.3.0"
+    "nextVersion": "v1.4.0"
   }
-}
+}