feat: print columns, field selectors, status handling, unit test (#32)

Author: robertgraeff

api/crds/manifests/openmcp.cloud_clusterproviders.yaml

@@ -16,7 +16,11 @@ spec:
     singular: clusterprovider
   scope: Cluster
   versions:
-  - name: v1alpha1
+  - additionalPrinterColumns:
+    - jsonPath: .status.phase
+      name: Phase
+      type: string
+    name: v1alpha1
     schema:
       openAPIV3Schema:
         description: ClusterProvider is the Schema for the clusterproviders API.
@@ -160,6 +164,8 @@ spec:
             - observedGeneration
             type: object
         type: object
+    selectableFields:
+    - jsonPath: .status.phase
     served: true
     storage: true
     subresources:

api/crds/manifests/openmcp.cloud_platformservices.yaml

@@ -16,7 +16,11 @@ spec:
     singular: platformservice
   scope: Cluster
   versions:
-  - name: v1alpha1
+  - additionalPrinterColumns:
+    - jsonPath: .status.phase
+      name: Phase
+      type: string
+    name: v1alpha1
     schema:
       openAPIV3Schema:
         description: PlatformService is the Schema for the platformservices API.
@@ -160,6 +164,8 @@ spec:
             - observedGeneration
             type: object
         type: object
+    selectableFields:
+    - jsonPath: .status.phase
     served: true
     storage: true
     subresources:

api/crds/manifests/openmcp.cloud_serviceproviders.yaml

@@ -16,7 +16,11 @@ spec:
     singular: serviceprovider
   scope: Cluster
   versions:
-  - name: v1alpha1
+  - additionalPrinterColumns:
+    - jsonPath: .status.phase
+      name: Phase
+      type: string
+    name: v1alpha1
     schema:
       openAPIV3Schema:
         description: ServiceProvider is the Schema for the serviceproviders API.
@@ -160,6 +164,8 @@ spec:
             - observedGeneration
             type: object
         type: object
+    selectableFields:
+    - jsonPath: .status.phase
     served: true
     storage: true
     subresources:

api/provider/v1alpha1/clusterprovider_types.go

@@ -38,6 +38,8 @@ type ClusterProviderStatus struct {
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:scope=Cluster
 // +kubebuilder:metadata:labels="openmcp.cloud/cluster=platform"
+// +kubebuilder:selectablefield:JSONPath=".status.phase"
+// +kubebuilder:printcolumn:JSONPath=".status.phase",name="Phase",type=string
 
 // ClusterProvider is the Schema for the clusterproviders API.
 type ClusterProvider struct {

api/provider/v1alpha1/platformservice_types.go

@@ -38,6 +38,8 @@ type PlatformServiceStatus struct {
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:scope=Cluster
 // +kubebuilder:metadata:labels="openmcp.cloud/cluster=platform"
+// +kubebuilder:selectablefield:JSONPath=".status.phase"
+// +kubebuilder:printcolumn:JSONPath=".status.phase",name="Phase",type=string
 
 // PlatformService is the Schema for the platformservices API.
 type PlatformService struct {

api/provider/v1alpha1/serviceprovider_types.go

@@ -38,6 +38,8 @@ type ServiceProviderStatus struct {
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:scope=Cluster
 // +kubebuilder:metadata:labels="openmcp.cloud/cluster=platform"
+// +kubebuilder:selectablefield:JSONPath=".status.phase"
+// +kubebuilder:printcolumn:JSONPath=".status.phase",name="Phase",type=string
 
 // ServiceProvider is the Schema for the serviceproviders API.
 type ServiceProvider struct {

cmd/openmcp-operator/app/run.go

@@ -312,9 +312,8 @@ func (o *RunOptions) Run(ctx context.Context) error {
 	if slices.Contains(o.Controllers, strings.ToLower(provider.ControllerName)) {
 		utilruntime.Must(clientgoscheme.AddToScheme(mgr.GetScheme()))
 		utilruntime.Must(api.AddToScheme(mgr.GetScheme()))
-
-		if err = provider.NewDeploymentController().SetupWithManager(mgr, o.ProviderGVKList, o.Environment); err != nil {
-			return fmt.Errorf("unable to setup provider controllers: %w", err)
+		if err = provider.NewDeploymentController().SetupWithManager(&setupLog, mgr, o.ProviderGVKList, o.Environment); err != nil {
+			return fmt.Errorf("unable to setup deployment controllers: %w", err)
 		}
 	}