Fixed VirtualMCPCompositeToolDefinition printer columns output (#5380)

* Fixed VirtualMCPCompositeToolDefinition printer columns output

Signed-off-by: Sanskarzz <sanskar.gur@gmail.com>

* Improved coverage and fixed CI lint error

Signed-off-by: Sanskarzz <sanskar.gur@gmail.com>

* Fixed Operator E2E CI failures

Signed-off-by: Sanskarzz <sanskar.gur@gmail.com>

* removed malformed Steps and Refs printer column

Signed-off-by: Sanskarzz <sanskar.gur@gmail.com>

---------

Signed-off-by: Sanskarzz <sanskar.gur@gmail.com>

Author: Sanskarzz

cmd/thv-operator/api/v1alpha1/types.go

@@ -339,9 +339,7 @@ type MCPToolConfigList struct {
 //+kubebuilder:subresource:status
 //+kubebuilder:resource:shortName=vmcpctd;compositetool,categories=toolhive
 //+kubebuilder:printcolumn:name="Workflow",type="string",JSONPath=".spec.name",description="Workflow name"
-//+kubebuilder:printcolumn:name="Steps",type="integer",JSONPath=".spec.steps[*]",description="Number of steps"
 //+kubebuilder:printcolumn:name="Status",type="string",JSONPath=".status.validationStatus",description="Validation status"
-//+kubebuilder:printcolumn:name="Refs",type="integer",JSONPath=".status.referencingVirtualServers[*]",description="Refs"
 //+kubebuilder:printcolumn:name="Age",type="date",JSONPath=".metadata.creationTimestamp",description="Age"
 //+kubebuilder:printcolumn:name="Ready",type="string",JSONPath=".status.conditions[?(@.type=='Ready')].status"
 

cmd/thv-operator/api/v1beta1/virtualmcpcompositetooldefinition_types.go

@@ -102,9 +102,7 @@ const (
 //+kubebuilder:subresource:status
 //+kubebuilder:resource:shortName=vmcpctd;compositetool,categories=toolhive
 //+kubebuilder:printcolumn:name="Workflow",type="string",JSONPath=".spec.name",description="Workflow name"
-//+kubebuilder:printcolumn:name="Steps",type="integer",JSONPath=".spec.steps[*]",description="Number of steps"
 //+kubebuilder:printcolumn:name="Status",type="string",JSONPath=".status.validationStatus",description="Validation status"
-//+kubebuilder:printcolumn:name="Refs",type="integer",JSONPath=".status.referencingVirtualServers[*]",description="Refs"
 //+kubebuilder:printcolumn:name="Age",type="date",JSONPath=".metadata.creationTimestamp",description="Age"
 //+kubebuilder:printcolumn:name="Ready",type="string",JSONPath=".status.conditions[?(@.type=='Ready')].status"
 

deploy/charts/operator-crds/files/crds/toolhive.stacklok.dev_virtualmcpcompositetooldefinitions.yaml

@@ -24,18 +24,10 @@ spec:
       jsonPath: .spec.name
       name: Workflow
       type: string
-    - description: Number of steps
-      jsonPath: .spec.steps[*]
-      name: Steps
-      type: integer
     - description: Validation status
       jsonPath: .status.validationStatus
       name: Status
       type: string
-    - description: Refs
-      jsonPath: .status.referencingVirtualServers[*]
-      name: Refs
-      type: integer
     - description: Age
       jsonPath: .metadata.creationTimestamp
       name: Age
@@ -431,18 +423,10 @@ spec:
       jsonPath: .spec.name
       name: Workflow
       type: string
-    - description: Number of steps
-      jsonPath: .spec.steps[*]
-      name: Steps
-      type: integer
     - description: Validation status
       jsonPath: .status.validationStatus
       name: Status
       type: string
-    - description: Refs
-      jsonPath: .status.referencingVirtualServers[*]
-      name: Refs
-      type: integer
     - description: Age
       jsonPath: .metadata.creationTimestamp
       name: Age

deploy/charts/operator-crds/templates/toolhive.stacklok.dev_virtualmcpcompositetooldefinitions.yaml

@@ -27,18 +27,10 @@ spec:
       jsonPath: .spec.name
       name: Workflow
       type: string
-    - description: Number of steps
-      jsonPath: .spec.steps[*]
-      name: Steps
-      type: integer
     - description: Validation status
       jsonPath: .status.validationStatus
       name: Status
       type: string
-    - description: Refs
-      jsonPath: .status.referencingVirtualServers[*]
-      name: Refs
-      type: integer
     - description: Age
       jsonPath: .metadata.creationTimestamp
       name: Age
@@ -434,18 +426,10 @@ spec:
       jsonPath: .spec.name
       name: Workflow
       type: string
-    - description: Number of steps
-      jsonPath: .spec.steps[*]
-      name: Steps
-      type: integer
     - description: Validation status
       jsonPath: .status.validationStatus
       name: Status
       type: string
-    - description: Refs
-      jsonPath: .status.referencingVirtualServers[*]
-      name: Refs
-      type: integer
     - description: Age
       jsonPath: .metadata.creationTimestamp
       name: Age