🌱 Bump CAPI to v1.8.4 (#5061)

* capi v1.8.0-beta.0 * use source.Kind generics * update e2e testing versions * set terminationMessagePolicy to FallbackToLogsOnError for manager * bump go version to 1.22 * Update metrics options for new types Signed-off-by: Nolan Brubaker <[email protected]> * Update to CAPI 1.8.3 Signed-off-by: Nolan Brubaker <[email protected]> * Fix test linting Signed-off-by: Nolan Brubaker <[email protected]> * Use latest versions of setup-envtest Signed-off-by: Nolan Brubaker <[email protected]> * Update generation tools for Kube 1.30 We carry a very small patch for the conversion-gen tool so that we can support having types of the same name in both the standard and experimental APIs. During the v1.30 timeframe, the Kubernetes code generation tools were refactored to more modern Go module standards, while also cleaning up the code. This commit carries the v1.30 (specifically, commit 304c1999892b41b6a3ff5dae260253f39cf53660) changes for the main.go and conversion.go files, alongside our patch. This commit also includes the updated conversion code output by our vendored/forked conversion_gen tool. Signed-off-by: Nolan Brubaker <[email protected]> * Update to CAPI 1.8.4 --------- Signed-off-by: Nolan Brubaker <[email protected]> Co-authored-by: Nolan Brubaker <[email protected]>
kubernetes-sigs · Oct 16, 2024 · 7e0812f · 7e0812f
1 parent 7cc5ee3
commit 7e0812f
Show file tree

Hide file tree

Showing 45 changed files with 571 additions and 529 deletions.
diff --git a/Makefile b/Makefile
@@ -148,8 +148,8 @@ E2E_SKIP_EKS_UPGRADE ?= "false"
 EKS_SOURCE_TEMPLATE ?= eks/cluster-template-eks-control-plane-only.yaml
 
 # set up `setup-envtest` to install kubebuilder dependency
-export KUBEBUILDER_ENVTEST_KUBERNETES_VERSION ?= 1.28.3
-SETUP_ENVTEST_VER := v0.0.0-20240531134648-6636df17d67b
+export KUBEBUILDER_ENVTEST_KUBERNETES_VERSION ?= 1.30.2
+SETUP_ENVTEST_VER := v0.0.0-20240923090159-236e448db12c
 SETUP_ENVTEST_BIN := setup-envtest
 SETUP_ENVTEST := $(abspath $(TOOLS_BIN_DIR)/$(SETUP_ENVTEST_BIN)-$(SETUP_ENVTEST_VER))
 SETUP_ENVTEST_PKG := sigs.k8s.io/controller-runtime/tools/setup-envtest
@@ -194,14 +194,15 @@ endif
 .PHONY: defaulters
 defaulters: $(DEFAULTER_GEN) ## Generate all Go types
  $(DEFAULTER_GEN) \
- --input-dirs=./api/v1beta2 \
- --input-dirs=./$(EXP_DIR)/api/v1beta2 \
- --input-dirs=./controlplane/rosa/api/v1beta2 \
- --input-dirs=./cmd/clusterawsadm/api/bootstrap/v1beta1 \
- --input-dirs=./cmd/clusterawsadm/api/bootstrap/v1alpha1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1beta1 \
- --v=0 $(GEN_OUTPUT_BASE) \
- --go-header-file=./hack/boilerplate/boilerplate.generatego.txt
+ --v=0 \
+ --go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+ --output-file=zz_generated.defaults.go \
+ ./api/v1beta2 \
+ ./$(EXP_DIR)/api/v1beta2 \
+ ./controlplane/rosa/api/v1beta2 \
+ ./cmd/clusterawsadm/api/bootstrap/v1beta1 \
+ ./cmd/clusterawsadm/api/bootstrap/v1alpha1
 
 .PHONY: generate
 generate: ## Generate code
@@ -251,44 +252,39 @@ generate-go-apis: ## Alias for .build/generate-go-apis
  $(MAKE) defaulters
 
  $(CONVERSION_GEN) \
- --input-dirs=./api/v1beta1 \
- --input-dirs=./cmd/clusterawsadm/api/bootstrap/v1alpha1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1beta1 \
- --build-tag=ignore_autogenerated_conversions \
- --output-file-base=zz_generated.conversion $(GEN_OUTPUT_BASE) \
- --go-header-file=./hack/boilerplate/boilerplate.generatego.txt
+ --output-file=zz_generated.conversion.go \
+ --go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+ ./api/v1beta1 \
+ ./cmd/clusterawsadm/api/bootstrap/v1alpha1
 
  $(CONVERSION_GEN) \
- --input-dirs=./$(EXP_DIR)/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api-provider-aws/v2/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1beta1 \
- --build-tag=ignore_autogenerated_conversions \
- --output-file-base=zz_generated.conversion $(GEN_OUTPUT_BASE) \
- --go-header-file=./hack/boilerplate/boilerplate.generatego.txt
+ --output-file=zz_generated.conversion.go \
+ --go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+ ./$(EXP_DIR)/api/v1beta1
 
  $(CONVERSION_GEN) \
- --input-dirs=./bootstrap/eks/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api-provider-aws/v2/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1beta1 \
- --build-tag=ignore_autogenerated_conversions \
- --output-file-base=zz_generated.conversion $(GEN_OUTPUT_BASE) \
- --go-header-file=./hack/boilerplate/boilerplate.generatego.txt
+ --output-file=zz_generated.conversion.go \
+ --go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+ ./bootstrap/eks/api/v1beta1
 
  $(CONVERSION_GEN) \
- --input-dirs=./controlplane/eks/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api-provider-aws/v2/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1beta1 \
- --build-tag=ignore_autogenerated_conversions \
- --output-file-base=zz_generated.conversion $(GEN_OUTPUT_BASE) \
- --go-header-file=./hack/boilerplate/boilerplate.generatego.txt
+ --output-file=zz_generated.conversion.go \
+ --go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+ ./controlplane/eks/api/v1beta1
 
  $(CONVERSION_GEN) \
- --input-dirs=./controlplane/rosa/api/v1beta2 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api-provider-aws/v2/api/v1beta1 \
  --extra-peer-dirs=sigs.k8s.io/cluster-api/api/v1beta1 \
- --build-tag=ignore_autogenerated_conversions \
- --output-file-base=zz_generated.conversion $(GEN_OUTPUT_BASE) \
- --go-header-file=./hack/boilerplate/boilerplate.generatego.txt
+ --output-file=zz_generated.conversion.go \
+ --go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+ ./controlplane/rosa/api/v1beta2
 
  touch $@
 

diff --git a/api/v1beta1/zz_generated.conversion.go b/api/v1beta1/zz_generated.conversion.go
diff --git a/api/v1beta1/zz_generated.deepcopy.go b/api/v1beta1/zz_generated.deepcopy.go
diff --git a/bootstrap/eks/api/v1beta1/zz_generated.conversion.go b/bootstrap/eks/api/v1beta1/zz_generated.conversion.go
diff --git a/bootstrap/eks/api/v1beta1/zz_generated.deepcopy.go b/bootstrap/eks/api/v1beta1/zz_generated.deepcopy.go
diff --git a/bootstrap/eks/api/v1beta2/doc.go b/bootstrap/eks/api/v1beta2/doc.go
@@ -17,4 +17,5 @@ limitations under the License.
 // Package v1beta2 contains API Schema definitions for the Amazon EKS Bootstrap v1beta2 API group.
 // +gencrdrefdocs:force //nolint: revive
 // +groupName=bootstrap.cluster.x-k8s.io
+// +k8s:conversion-gen=sigs.k8s.io/cluster-api-provider-aws/v2/bootstrap/eks/api/v1beta1
 package v1beta2
diff --git a/bootstrap/eks/controllers/eksconfig_controller.go b/bootstrap/eks/controllers/eksconfig_controller.go
@@ -313,9 +313,9 @@ func (r *EKSConfigReconciler) SetupWithManager(ctx context.Context, mgr ctrl.Man
  }
 
  err = c.Watch(
- source.Kind(mgr.GetCache(), &clusterv1.Cluster{}),
- handler.EnqueueRequestsFromMapFunc((r.ClusterToEKSConfigs)),
- predicates.ClusterUnpausedAndInfrastructureReady(logger.FromContext(ctx).GetLogger()),
+ source.Kind[client.Object](mgr.GetCache(), &clusterv1.Cluster{},
+  handler.EnqueueRequestsFromMapFunc((r.ClusterToEKSConfigs)),
+  predicates.ClusterUnpausedAndInfrastructureReady(logger.FromContext(ctx).GetLogger())),
  )
  if err != nil {
  return errors.Wrap(err, "failed adding watch for Clusters to controller manager")

diff --git a/cmd/clusterawsadm/api/bootstrap/v1alpha1/zz_generated.conversion.go b/cmd/clusterawsadm/api/bootstrap/v1alpha1/zz_generated.conversion.go
diff --git a/config/crd/bases/controlplane.cluster.x-k8s.io_awsmanagedcontrolplanes.yaml b/config/crd/bases/controlplane.cluster.x-k8s.io_awsmanagedcontrolplanes.yaml
@@ -959,10 +959,15 @@ spec:
  description: The key to select.
  type: string
  name:
+ default: ""
  description: |-
  Name of the referent.
- More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ This field is effectively required, but due to backwards compatibility is
+ allowed to be empty. Instances of this type with an empty value here are
+ almost certainly wrong.
  TODO: Add other useful fields. apiVersion, kind, uid?
+ More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
  type: string
  optional:
  description: Specify whether the ConfigMap or its
@@ -1022,10 +1027,15 @@ spec:
  be a valid secret key.
  type: string
  name:
+ default: ""
  description: |-
  Name of the referent.
- More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ This field is effectively required, but due to backwards compatibility is
+ allowed to be empty. Instances of this type with an empty value here are
+ almost certainly wrong.
  TODO: Add other useful fields. apiVersion, kind, uid?
+ More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
  type: string
  optional:
  description: Specify whether the Secret or its key
@@ -3010,10 +3020,15 @@ spec:
  description: The key to select.
  type: string
  name:
+ default: ""
  description: |-
  Name of the referent.
- More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ This field is effectively required, but due to backwards compatibility is
+ allowed to be empty. Instances of this type with an empty value here are
+ almost certainly wrong.
  TODO: Add other useful fields. apiVersion, kind, uid?
+ More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
  type: string
  optional:
  description: Specify whether the ConfigMap or its
@@ -3073,10 +3088,15 @@ spec:
  be a valid secret key.
  type: string
  name:
+ default: ""
  description: |-
  Name of the referent.
- More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ This field is effectively required, but due to backwards compatibility is
+ allowed to be empty. Instances of this type with an empty value here are
+ almost certainly wrong.
  TODO: Add other useful fields. apiVersion, kind, uid?
+ More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
  type: string
  optional:
  description: Specify whether the Secret or its key

diff --git a/config/crd/bases/controlplane.cluster.x-k8s.io_rosacontrolplanes.yaml b/config/crd/bases/controlplane.cluster.x-k8s.io_rosacontrolplanes.yaml
@@ -174,10 +174,15 @@ spec:
  - ocmApiUrl: Optional, defaults to 'https://api.openshift.com'
  properties:
  name:
+ default: ""
  description: |-
  Name of the referent.
- More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ This field is effectively required, but due to backwards compatibility is
+ allowed to be empty. Instances of this type with an empty value here are
+ almost certainly wrong.
  TODO: Add other useful fields. apiVersion, kind, uid?
+ More info: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
+ TODO: Drop `kubebuilder:default` when controller-gen doesn't need it https://github.com/kubernetes-sigs/kubebuilder/issues/3896.
  type: string
  type: object
  x-kubernetes-map-type: atomic

diff --git a/config/crd/bases/infrastructure.cluster.x-k8s.io_awsclustercontrolleridentities.yaml b/config/crd/bases/infrastructure.cluster.x-k8s.io_awsclustercontrolleridentities.yaml
@@ -92,11 +92,13 @@ spec:
  items:
  type: string
  type: array
+ x-kubernetes-list-type: atomic
  required:
  - key
  - operator
  type: object
  type: array
+ x-kubernetes-list-type: atomic
  matchLabels:
  additionalProperties:
  type: string
@@ -186,11 +188,13 @@ spec:
  items:
  type: string
  type: array
+ x-kubernetes-list-type: atomic
  required:
  - key
  - operator
  type: object
  type: array
+ x-kubernetes-list-type: atomic
  matchLabels:
  additionalProperties:
  type: string

diff --git a/config/crd/bases/infrastructure.cluster.x-k8s.io_awsclusterroleidentities.yaml b/config/crd/bases/infrastructure.cluster.x-k8s.io_awsclusterroleidentities.yaml
@@ -92,11 +92,13 @@ spec:
  items:
  type: string
  type: array
+ x-kubernetes-list-type: atomic
  required:
  - key
  - operator
  type: object
  type: array
+ x-kubernetes-list-type: atomic
  matchLabels:
  additionalProperties:
  type: string
@@ -245,11 +247,13 @@ spec:
  items:
  type: string
  type: array
+ x-kubernetes-list-type: atomic
  required:
  - key
  - operator
  type: object
  type: array
+ x-kubernetes-list-type: atomic
  matchLabels:
  additionalProperties:
  type: string

diff --git a/config/crd/bases/infrastructure.cluster.x-k8s.io_awsclusterstaticidentities.yaml b/config/crd/bases/infrastructure.cluster.x-k8s.io_awsclusterstaticidentities.yaml
@@ -92,11 +92,13 @@ spec:
  items:
  type: string
  type: array
+ x-kubernetes-list-type: atomic
  required:
  - key
  - operator
  type: object
  type: array
+ x-kubernetes-list-type: atomic
  matchLabels:
  additionalProperties:
  type: string
@@ -196,11 +198,13 @@ spec:
  items:
  type: string
  type: array
+ x-kubernetes-list-type: atomic
  required:
  - key
  - operator
  type: object
  type: array
+ x-kubernetes-list-type: atomic
  matchLabels:
  additionalProperties:
  type: string

diff --git a/config/manager/manager.yaml b/config/manager/manager.yaml
@@ -48,6 +48,7 @@ spec:
  - ALL
  runAsUser: 65532
  runAsGroup: 65532
+ terminationMessagePolicy: FallbackToLogsOnError
  securityContext:
  runAsNonRoot: true
  seccompProfile:

diff --git a/controllers/awscluster_controller.go b/controllers/awscluster_controller.go
@@ -413,10 +413,10 @@ func (r *AWSClusterReconciler) SetupWithManager(ctx context.Context, mgr ctrl.Ma
  }
 
  return controller.Watch(
- source.Kind(mgr.GetCache(), &clusterv1.Cluster{}),
- handler.EnqueueRequestsFromMapFunc(r.requeueAWSClusterForUnpausedCluster(ctx, log)),
- predicates.ClusterUnpaused(log.GetLogger()),
- )
+ source.Kind[client.Object](mgr.GetCache(), &clusterv1.Cluster{},
+  handler.EnqueueRequestsFromMapFunc(r.requeueAWSClusterForUnpausedCluster(ctx, log)),
+  predicates.ClusterUnpaused(log.GetLogger()),
+  ))
 }
 
 func (r *AWSClusterReconciler) requeueAWSClusterForUnpausedCluster(_ context.Context, log logger.Wrapper) handler.MapFunc {

diff --git a/controllers/awsmachine_controller.go b/controllers/awsmachine_controller.go
@@ -292,9 +292,9 @@ func (r *AWSMachineReconciler) SetupWithManager(ctx context.Context, mgr ctrl.Ma
 
  requeueAWSMachinesForUnpausedCluster := r.requeueAWSMachinesForUnpausedCluster(log)
  return controller.Watch(
- source.Kind(mgr.GetCache(), &clusterv1.Cluster{}),
- handler.EnqueueRequestsFromMapFunc(requeueAWSMachinesForUnpausedCluster),
- predicates.ClusterUnpausedAndInfrastructureReady(log.GetLogger()),
+ source.Kind[client.Object](mgr.GetCache(), &clusterv1.Cluster{},
+  handler.EnqueueRequestsFromMapFunc(requeueAWSMachinesForUnpausedCluster),
+  predicates.ClusterUnpausedAndInfrastructureReady(log.GetLogger())),
  )
 }
 

diff --git a/controllers/awsmanagedcluster_controller.go b/controllers/awsmanagedcluster_controller.go
@@ -134,17 +134,17 @@ func (r *AWSManagedClusterReconciler) SetupWithManager(ctx context.Context, mgr
 
  // Add a watch for clusterv1.Cluster unpaise
  if err = controller.Watch(
- source.Kind(mgr.GetCache(), &clusterv1.Cluster{}),
- handler.EnqueueRequestsFromMapFunc(util.ClusterToInfrastructureMapFunc(ctx, infrav1.GroupVersion.WithKind("AWSManagedCluster"), mgr.GetClient(), &infrav1.AWSManagedCluster{})),
- predicates.ClusterUnpaused(log.GetLogger()),
+ source.Kind[client.Object](mgr.GetCache(), &clusterv1.Cluster{},
+  handler.EnqueueRequestsFromMapFunc(util.ClusterToInfrastructureMapFunc(ctx, infrav1.GroupVersion.WithKind("AWSManagedCluster"), mgr.GetClient(), &infrav1.AWSManagedCluster{})),
+  predicates.ClusterUnpaused(log.GetLogger())),
  ); err != nil {
  return fmt.Errorf("failed adding a watch for ready clusters: %w", err)
  }
 
  // Add a watch for AWSManagedControlPlane
  if err = controller.Watch(
- source.Kind(mgr.GetCache(), &ekscontrolplanev1.AWSManagedControlPlane{}),
- handler.EnqueueRequestsFromMapFunc(r.managedControlPlaneToManagedCluster(ctx, log)),
+ source.Kind[client.Object](mgr.GetCache(), &ekscontrolplanev1.AWSManagedControlPlane{},
+  handler.EnqueueRequestsFromMapFunc(r.managedControlPlaneToManagedCluster(ctx, log))),
  ); err != nil {
  return fmt.Errorf("failed adding watch on AWSManagedControlPlane: %w", err)
  }