Integrate CRS code into regular code structure

Refactor: MOVE CRS to his own addons group

Fix go licenses

Fix webhook alias SetupWebhookWithManager call

Fix envtest, add addonsv1 scheme

Fix Test_* with no addonsv1 scheme

Fix APIGroup for addonsv1

Address reviews and fix missing imports

Generate API

Fix CRS controllers call

Author: mcbenjemaa

.golangci-kal.yml

@@ -62,7 +62,7 @@ issues:
   - path-except: "api//*"
     linters:
       - kal
-  - path: "api/v1beta1/*|api/v1alpha1/*"
+  - path: "api/v1beta1/*|api/v1alpha1/*|api/addons/v1beta1/*"
     text: "Conditions field must be a slice of metav1.Condition"
     linters:
       - kal
@@ -86,11 +86,11 @@ issues:
       - kal
   # It does not make sense to add a maxItems marker on the *List structs as they are not used to generate CRD YAMLs.
   # This exclude will  be removed once https://github.com/JoelSpeed/kal/issues/38 is resolved.
-  - path: "api/v1beta1/*|api/v1alpha1/*"
+  - path: "api/v1beta1/*|api/v1alpha1/*|api/addons/v1beta1/*"
     text: "maxlength: field Items must have a maximum items, add kubebuilder:validation:MaxItems marker"
     linters:
       - kal
-  - path: "api/v1alpha1/*|api/v1beta1/*"
+  - path: "api/v1alpha1/*|api/v1beta1/*|api/addons/v1beta1/*"
     text: "nobools"
     linters:
       - kal

.golangci.yml

@@ -144,11 +144,11 @@ linters-settings:
       - pkg: sigs.k8s.io/cluster-api/exp/api/v1beta1
         alias: expv1
       # CAPI exp addons
-      - pkg: sigs.k8s.io/cluster-api/internal/apis/core/exp/addons/v1alpha3
+      - pkg: sigs.k8s.io/cluster-api/internal/apis/addons/v1alpha3
         alias: addonsv1alpha3
-      - pkg: sigs.k8s.io/cluster-api/internal/apis/core/exp/addons/v1alpha4
+      - pkg: sigs.k8s.io/cluster-api/internal/apis/addons/v1alpha4
         alias: addonsv1alpha4
-      - pkg: sigs.k8s.io/cluster-api/exp/addons/api/v1beta1
+      - pkg: sigs.k8s.io/cluster-api/api/addons/v1beta1
         alias: addonsv1
       # CAPI exp IPAM
       - pkg: sigs.k8s.io/cluster-api/exp/ipam/api/v1beta1
@@ -282,6 +282,12 @@ issues:
   - linters:
       - staticcheck
     text: "SA1019: feature.ClusterResourceSet is deprecated: ClusterResourceSet feature is now GA and the corresponding feature flag will be removed in 1.12 release."
+  - linters:
+      - revive
+    text: "exported: type name will be used as clusterresourceset.ClusterResourceSetBindingReconciler by other packages, and that stutters; consider calling this BindingReconciler"
+  - linters:
+      - revive
+    text: "exported: type name will be used as clusterresourceset.ClusterResourceSetReconciler by other packages, and that stutters; consider calling this Reconciler"
   - linters:
     - revive
     text: "exported: exported method .*\\.(Reconcile|SetupWithManager|SetupWebhookWithManager) should have comment or be unexported"

Makefile

@@ -286,12 +286,10 @@ generate-manifests-core: $(CONTROLLER_GEN) $(KUSTOMIZE) ## Generate manifests e.
 		paths=./internal/apis/core/... \
 		paths=./internal/controllers/... \
 		paths=./internal/webhooks/... \
+		paths=./internal/apis/addons/... \
 		paths=./$(EXP_DIR)/api/... \
 		paths=./$(EXP_DIR)/internal/controllers/... \
 		paths=./$(EXP_DIR)/internal/webhooks/... \
-		paths=./$(EXP_DIR)/addons/api/... \
-		paths=./$(EXP_DIR)/addons/internal/controllers/... \
-		paths=./$(EXP_DIR)/addons/internal/webhooks/... \
 		paths=./$(EXP_DIR)/ipam/api/... \
 		paths=./$(EXP_DIR)/ipam/internal/webhooks/... \
 		paths=./$(EXP_DIR)/runtime/api/... \
@@ -389,12 +387,12 @@ generate-go-deepcopy:  ## Run all generate-go-deepcopy-* targets
 
 .PHONY: generate-go-deepcopy-core
 generate-go-deepcopy-core: $(CONTROLLER_GEN) ## Generate deepcopy go code for core
-	$(MAKE) clean-generated-deepcopy SRC_DIRS="./api,./$(EXP_DIR)/api,./$(EXP_DIR)/addons/api,./$(EXP_DIR)/runtime/api,./$(EXP_DIR)/runtime/hooks/api"
+	$(MAKE) clean-generated-deepcopy SRC_DIRS="./api,./internal/apis/addons,./$(EXP_DIR)/api,./$(EXP_DIR)/runtime/api,./$(EXP_DIR)/runtime/hooks/api"
 	$(CONTROLLER_GEN) \
 		object:headerFile=./hack/boilerplate/boilerplate.generatego.txt \
 		paths=./api/... \
+		paths=./internal/apis/addons/... \
 		paths=./$(EXP_DIR)/api/... \
-		paths=./$(EXP_DIR)/addons/api/... \
 		paths=./$(EXP_DIR)/ipam/api/... \
 		paths=./$(EXP_DIR)/runtime/api/... \
 		paths=./$(EXP_DIR)/runtime/hooks/api/... \
@@ -443,6 +441,7 @@ generate-go-conversions: ## Run all generate-go-conversions-* targets
 .PHONY: generate-go-conversions-core
 generate-go-conversions-core: ## Run all generate-go-conversions-core-* targets
 	$(MAKE) generate-go-conversions-core-api
+	$(MAKE) generate-go-conversions-addons-api
 	$(MAKE) generate-go-conversions-core-exp
 	$(MAKE) generate-go-conversions-core-exp-ipam
 	$(MAKE) generate-go-conversions-core-runtime
@@ -456,16 +455,23 @@ generate-go-conversions-core-api: $(CONVERSION_GEN) ## Generate conversions go c
 		./internal/apis/core/v1alpha3 \
 		./internal/apis/core/v1alpha4
 
+.PHONY: generate-go-conversions-addons-api
+generate-go-conversions-addons-api: $(CONVERSION_GEN) ## Generate conversions go code for addons api
+	$(MAKE) clean-generated-conversions SRC_DIRS="./internal/apis/addons/v1alpha3,./internal/apis/addons/v1alpha4"
+	$(CONVERSION_GEN) \
+		--output-file=zz_generated.conversion.go \
+		--go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
+		./internal/apis/addons/v1alpha3 \
+		./internal/apis/addons/v1alpha4
+
 .PHONY: generate-go-conversions-core-exp
 generate-go-conversions-core-exp: $(CONVERSION_GEN) ## Generate conversions go code for core exp
-	$(MAKE) clean-generated-conversions SRC_DIRS="./internal/apis/core/exp/v1alpha3,./internal/apis/core/exp/addons/v1alpha3,./internal/apis/core/exp/v1alpha4,./internal/apis/core/exp/addons/v1alpha4"
+	$(MAKE) clean-generated-conversions SRC_DIRS="./internal/apis/core/exp/v1alpha3,./internal/apis/core/exp/v1alpha4"
 	$(CONVERSION_GEN) \
 		--output-file=zz_generated.conversion.go \
 		--go-header-file=./hack/boilerplate/boilerplate.generatego.txt \
 		./internal/apis/core/exp/v1alpha3 \
-		./internal/apis/core/exp/v1alpha4 \
-		./internal/apis/core/exp/addons/v1alpha3 \
-		./internal/apis/core/exp/addons/v1alpha4
+		./internal/apis/core/exp/v1alpha4
 
 .PHONY: generate-go-conversions-core-exp-ipam
 generate-go-conversions-core-exp-ipam: $(CONVERSION_GEN) ## Generate conversions go code for core exp IPAM
@@ -527,7 +533,7 @@ generate-go-conversions-test-extension: $(CONVERSION_GEN) ## Generate conversion
 .PHONY: generate-go-openapi
 generate-go-openapi: $(OPENAPI_GEN) ## Generate openapi go code for runtime SDK
 	@mkdir -p ./tmp/sigs.k8s.io; ln -s $(ROOT_DIR) ./tmp/sigs.k8s.io/; cd ./tmp; \
-	for pkg in "api/v1beta1" "$(EXP_DIR)/runtime/hooks/api/v1alpha1"; do \
+	for pkg in "api/v1beta1" "api/addons/v1beta1" "$(EXP_DIR)/runtime/hooks/api/v1alpha1"; do \
 		(cd ../ && $(MAKE) clean-generated-openapi-definitions SRC_DIRS="./$${pkg}"); \
 		echo "** Generating openapi schema for types in ./$${pkg} **"; \
 		$(OPENAPI_GEN) \

exp/addons/api/.import-restrictions renamed to api/addons/.import-restrictions

@@ -205,6 +205,7 @@ type ClusterResourceSetList struct {
 	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
 	// items is the list of ClusterResourceSets.
+	// +required
 	Items []ClusterResourceSet `json:"items"`
 }
 

exp/addons/api/v1beta1/.import-restrictions renamed to api/addons/v1beta1/.import-restrictions

@@ -219,6 +219,7 @@ type ClusterResourceSetBindingList struct {
 	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
 	// items is the list of ClusterResourceSetBindings.
+	// +required
 	Items []ClusterResourceSetBinding `json:"items"`
 }
 

exp/addons/api/v1beta1/clusterresourceset_types.go renamed to api/addons/v1beta1/clusterresourceset_types.go

@@ -19,7 +19,6 @@ package v1beta1
 import clusterv1 "sigs.k8s.io/cluster-api/api/v1beta1"
 
 // Conditions and condition Reasons for the ClusterResourceSet object.
-
 const (
 	// ResourcesAppliedCondition documents that all resources in the ClusterResourceSet object are applied to
 	// all matching clusters. This indicates all resources exist, and no errors during applying them to all clusters.