add printercolumn for syncStatus in SriovNetworkNodeState

k8snetworkplumbingwg · Feb 7, 2023 · 47d715b · 47d715b
1 parent 2d8ba0d
commit 47d715b
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 11 deletions.
diff --git a/api/v1/sriovnetworknodestate_types.go b/api/v1/sriovnetworknodestate_types.go
@@ -90,6 +90,7 @@ type SriovNetworkNodeStateStatus struct {
 
 //+kubebuilder:object:root=true
 //+kubebuilder:subresource:status
+//+kubebuilder:printcolumn:name="Sync Status",type=string,JSONPath=`.status.syncStatus`
 
 // SriovNetworkNodeState is the Schema for the sriovnetworknodestates API
 type SriovNetworkNodeState struct {

diff --git a/config/crd/bases/sriovnetwork.openshift.io_sriovnetworknodestates.yaml b/config/crd/bases/sriovnetwork.openshift.io_sriovnetworknodestates.yaml
@@ -15,7 +15,11 @@ spec:
     singular: sriovnetworknodestate
   scope: Namespaced
   versions:
-  - name: v1
+  - additionalPrinterColumns:
+    - jsonPath: .status.syncStatus
+      name: Sync Status
+      type: string
+    name: v1
     schema:
       openAPIV3Schema:
         description: SriovNetworkNodeState is the Schema for the sriovnetworknodestates

diff --git a/deployment/sriov-network-operator/crds/sriovnetwork.openshift.io_sriovnetworknodestates.yaml b/deployment/sriov-network-operator/crds/sriovnetwork.openshift.io_sriovnetworknodestates.yaml
@@ -15,7 +15,11 @@ spec:
     singular: sriovnetworknodestate
   scope: Namespaced
   versions:
-  - name: v1
+  - additionalPrinterColumns:
+    - jsonPath: .status.syncStatus
+      name: Sync Status
+      type: string
+    name: v1
     schema:
       openAPIV3Schema:
         description: SriovNetworkNodeState is the Schema for the sriovnetworknodestates

diff --git a/pkg/daemon/daemon.go b/pkg/daemon/daemon.go
@@ -103,14 +103,15 @@ type Daemon struct {
 }
 
 const (
-	rdmaScriptsPath     = "/bindata/scripts/enable-rdma.sh"
-	udevScriptsPath     = "/bindata/scripts/load-udev.sh"
-	annoKey             = "sriovnetwork.openshift.io/state"
-	annoIdle            = "Idle"
-	annoDraining        = "Draining"
-	annoMcpPaused       = "Draining_MCP_Paused"
-	syncStatusSucceeded = "Succeeded"
-	syncStatusFailed    = "Failed"
+	rdmaScriptsPath      = "/bindata/scripts/enable-rdma.sh"
+	udevScriptsPath      = "/bindata/scripts/load-udev.sh"
+	annoKey              = "sriovnetwork.openshift.io/state"
+	annoIdle             = "Idle"
+	annoDraining         = "Draining"
+	annoMcpPaused        = "Draining_MCP_Paused"
+	syncStatusSucceeded  = "Succeeded"
+	syncStatusFailed     = "Failed"
+	syncStatusInProgress = "InProgress"
 )
 
 var namespace = os.Getenv("NAMESPACE")
@@ -448,7 +449,7 @@ func (dn *Daemon) nodeStateSyncHandler() error {
 	}
 
 	dn.refreshCh <- Message{
-		syncStatus:    "InProgress",
+		syncStatus:    syncStatusInProgress,
 		lastSyncError: "",
 	}