Add printer columns to all CRDs (#4489)

Add printer columns to all CRDs for informative kubectl output

`kubectl get` output for config CRDs (MCPOIDCConfig, MCPTelemetryConfig,
MCPExternalAuthConfig, MCPToolConfig) was minimal — showing only type/name
and age. Operators had to use `-o yaml` to check readiness or which servers
reference a config. This adds Ready and References columns to all config
CRDs, renames MCPOIDCConfig's column to "Source" for clarity, and fixes the
MCPGroup `printerColumn` typo that prevented its columns from rendering.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: ChrisJBurns

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

@@ -722,6 +722,8 @@ type MCPExternalAuthConfigStatus struct {
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:shortName=extauth;mcpextauth,categories=toolhive
 // +kubebuilder:printcolumn:name="Type",type=string,JSONPath=`.spec.type`
+// +kubebuilder:printcolumn:name="Ready",type=string,JSONPath=`.status.conditions[?(@.type=='Valid')].status`
+// +kubebuilder:printcolumn:name="References",type=string,JSONPath=`.status.referencingServers`
 // +kubebuilder:printcolumn:name="Age",type=date,JSONPath=`.metadata.creationTimestamp`
 
 // MCPExternalAuthConfig is the Schema for the mcpexternalauthconfigs API.

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

@@ -75,11 +75,10 @@ const (
 //+kubebuilder:object:root=true
 //+kubebuilder:subresource:status
 //+kubebuilder:resource:shortName=mcpg;mcpgroup,categories=toolhive
-//nolint:lll
-//+kubebuilder:printerColumn:name="Servers",type="integer",JSONPath=".status.serverCount",description="The number of MCPServers in this group"
-//+kubebuilder:printerColumn:name="Phase",type="string",JSONPath=".status.phase",description="The phase of the MCPGroup"
-//+kubebuilder:printerColumn:name="Age",type="date",JSONPath=".metadata.creationTimestamp",description="The age of the MCPGroup"
+//+kubebuilder:printcolumn:name="Servers",type="integer",JSONPath=".status.serverCount"
+//+kubebuilder:printcolumn:name="Phase",type="string",JSONPath=".status.phase"
 //+kubebuilder:printcolumn:name="Ready",type="string",JSONPath=".status.conditions[?(@.type=='MCPServersChecked')].status"
+//+kubebuilder:printcolumn:name="Age",type="date",JSONPath=".metadata.creationTimestamp"
 
 // MCPGroup is the Schema for the mcpgroups API
 type MCPGroup struct {

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

@@ -154,7 +154,9 @@ type MCPOIDCConfigStatus struct {
 // +kubebuilder:object:root=true
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:shortName=mcpoidc,categories=toolhive
-// +kubebuilder:printcolumn:name="Type",type=string,JSONPath=`.spec.type`
+// +kubebuilder:printcolumn:name="Source",type=string,JSONPath=`.spec.type`
+// +kubebuilder:printcolumn:name="Ready",type=string,JSONPath=`.status.conditions[?(@.type=='Valid')].status`
+// +kubebuilder:printcolumn:name="References",type=string,JSONPath=`.status.referencingServers`
 // +kubebuilder:printcolumn:name="Age",type=date,JSONPath=`.metadata.creationTimestamp`
 
 // MCPOIDCConfig is the Schema for the mcpoidcconfigs API.

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

@@ -68,8 +68,8 @@ type MCPTelemetryConfigStatus struct {
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:shortName=mcpotel,categories=toolhive
 // +kubebuilder:printcolumn:name="Endpoint",type=string,JSONPath=`.spec.endpoint`
-// +kubebuilder:printcolumn:name="Tracing",type=boolean,JSONPath=`.spec.tracingEnabled`
-// +kubebuilder:printcolumn:name="Metrics",type=boolean,JSONPath=`.spec.metricsEnabled`
+// +kubebuilder:printcolumn:name="Ready",type=string,JSONPath=`.status.conditions[?(@.type=='Valid')].status`
+// +kubebuilder:printcolumn:name="References",type=string,JSONPath=`.status.referencingServers`
 // +kubebuilder:printcolumn:name="Age",type=date,JSONPath=`.metadata.creationTimestamp`
 
 // MCPTelemetryConfig is the Schema for the mcptelemetryconfigs API.

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

@@ -105,9 +105,8 @@ type MCPToolConfigStatus struct {
 // +kubebuilder:object:root=true
 // +kubebuilder:subresource:status
 // +kubebuilder:resource:shortName=tc;toolconfig,categories=toolhive
-// +kubebuilder:printcolumn:name="Filter Count",type=integer,JSONPath=`.spec.toolsFilter[*]`
-// +kubebuilder:printcolumn:name="Override Count",type=integer,JSONPath=`.spec.toolsOverride`
-// +kubebuilder:printcolumn:name="Referenced By",type=string,JSONPath=`.status.referencingServers`
+// +kubebuilder:printcolumn:name="Ready",type=string,JSONPath=`.status.conditions[?(@.type=='Valid')].status`
+// +kubebuilder:printcolumn:name="References",type=string,JSONPath=`.status.referencingServers`
 // +kubebuilder:printcolumn:name="Age",type=date,JSONPath=`.metadata.creationTimestamp`
 
 // MCPToolConfig is the Schema for the mcptoolconfigs API.

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

@@ -23,6 +23,12 @@ spec:
     - jsonPath: .spec.type
       name: Type
       type: string
+    - jsonPath: .status.conditions[?(@.type=='Valid')].status
+      name: Ready
+      type: string
+    - jsonPath: .status.referencingServers
+      name: References
+      type: string
     - jsonPath: .metadata.creationTimestamp
       name: Age
       type: date

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

@@ -20,9 +20,18 @@ spec:
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
+    - jsonPath: .status.serverCount
+      name: Servers
+      type: integer
+    - jsonPath: .status.phase
+      name: Phase
+      type: string
     - jsonPath: .status.conditions[?(@.type=='MCPServersChecked')].status
       name: Ready
       type: string
+    - jsonPath: .metadata.creationTimestamp
+      name: Age
+      type: date
     name: v1alpha1
     schema:
       openAPIV3Schema:

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

@@ -20,7 +20,13 @@ spec:
   versions:
   - additionalPrinterColumns:
     - jsonPath: .spec.type
-      name: Type
+      name: Source
+      type: string
+    - jsonPath: .status.conditions[?(@.type=='Valid')].status
+      name: Ready
+      type: string
+    - jsonPath: .status.referencingServers
+      name: References
       type: string
     - jsonPath: .metadata.creationTimestamp
       name: Age

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

@@ -22,12 +22,12 @@ spec:
     - jsonPath: .spec.endpoint
       name: Endpoint
       type: string
-    - jsonPath: .spec.tracingEnabled
-      name: Tracing
-      type: boolean
-    - jsonPath: .spec.metricsEnabled
-      name: Metrics
-      type: boolean
+    - jsonPath: .status.conditions[?(@.type=='Valid')].status
+      name: Ready
+      type: string
+    - jsonPath: .status.referencingServers
+      name: References
+      type: string
     - jsonPath: .metadata.creationTimestamp
       name: Age
       type: date

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

@@ -20,14 +20,11 @@ spec:
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
-    - jsonPath: .spec.toolsFilter[*]
-      name: Filter Count
-      type: integer
-    - jsonPath: .spec.toolsOverride
-      name: Override Count
-      type: integer
+    - jsonPath: .status.conditions[?(@.type=='Valid')].status
+      name: Ready
+      type: string
     - jsonPath: .status.referencingServers
-      name: Referenced By
+      name: References
       type: string
     - jsonPath: .metadata.creationTimestamp
       name: Age