Addressed review comments

Author: rajashekhargundeti

cmd/oci-csi-controller-driver/main.go

@@ -55,6 +55,6 @@ func main() {
 	}
 	logger.With("endpoint", csiOptions.Endpoint).Infof("Starting controller driver go routine.")
 	go csicontrollerdriver.StartControllerDriver(csiOptions, driver.BV)
-	go csicontrollerdriver.StartControllerDriver(csiOptions, driver.FSS) //TODO: Need to check with Mrunal to be it here
+	go csicontrollerdriver.StartControllerDriver(csiOptions, driver.FSS)
 	<-stopCh
 }

pkg/cloudprovider/providers/oci/config/config.go

@@ -15,10 +15,11 @@
 package config
 
 import (
-	"github.com/oracle/oci-cloud-controller-manager/pkg/oci/instance/metadata"
 	"io"
 	"os"
 
+	"github.com/oracle/oci-cloud-controller-manager/pkg/oci/instance/metadata"
+
 	"github.com/oracle/oci-go-sdk/v65/common"
 	"github.com/oracle/oci-go-sdk/v65/common/auth"
 	"github.com/pkg/errors"
@@ -122,7 +123,7 @@ type TagConfig struct {
 	DefinedTags  map[string]map[string]interface{} `yaml:"defined"`
 }
 
-// InitialTags initialTags are optional tags to apply to all LBs, BVs, File systems, Mount Targets and Exports provisioned in the cluster
+// InitialTags are optional tags to apply to all LBs, BVs, File systems, Mount Targets and Exports provisioned in the cluster
 type InitialTags struct {
 	LoadBalancer *TagConfig `yaml:"loadBalancer"`
 	BlockVolume  *TagConfig `yaml:"blockVolume"`

pkg/csi/driver/fss_controller_test.go

@@ -174,10 +174,6 @@ type MockFSSProvisionerClient struct {
 	Storage *MockFileStorageClient
 }
 
-// func (m MockFSSProvisionerClient) ContainerEngine() client.ContainerEngineInterface {
-// 	return &MockContainerEngineClient{}
-// }
-
 func (m MockFSSProvisionerClient) Compute() client.ComputeInterface {
 	return &MockComputeClient{}
 }