CR comment - changed pollInterval to be the last argument for lvmnode…

… Start Signed-off-by: Omri Cohen <[email protected]>
openebs · Sep 7, 2023 · 05695bd · 05695bd
1 parent b283985
commit 05695bd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/pkg/driver/agent.go b/pkg/driver/agent.go
@@ -66,7 +66,8 @@ func NewNode(d *CSIDriver) csi.NodeServer {
 
 	// start the lvm node resource watcher
 	go func() {
-		err := lvmnode.Start(d.config.NodeControllerPollingInterval, &ControllerMutex, stopCh)
+		err := lvmnode.
+			Start(&ControllerMutex, stopCh, d.config.NodeControllerPollingInterval)
 		if err != nil {
 			klog.Fatalf("Failed to start LVM node controller: %s", err.Error())
 		}

diff --git a/pkg/mgmt/lvmnode/start.go b/pkg/mgmt/lvmnode/start.go
@@ -35,7 +35,7 @@ import (
 )
 
 // Start starts the lvmnode controller.
-func Start(pollInterval int, controllerMtx *sync.RWMutex, stopCh <-chan struct{}) error {
+func Start(controllerMtx *sync.RWMutex, stopCh <-chan struct{}, pollInterval int) error {
 
 	// Get in cluster config
 	cfg, err := k8sapi.Config().Get()