Remove redundant nil check for mc

Author: alleaditya

cmd/sidecar_mounter/main.go

@@ -97,22 +97,20 @@ func main() {
 				klog.Infof("Running cloud profiler on %s", cfg.Service)
 			}
 		}
-		if mc != nil {
-			if mc.EnableSidecarBucketAccessCheck {
-				mc.SidecarRetryConfig.Cap = *storageServiceAndBucketAccessCap
-				mc.SidecarRetryConfig.Steps = *storageServiceAndBucketAccessSteps
-				mc.SidecarRetryConfig.Factor = *storageServiceAndBucketAccessFactor
-				mc.SidecarRetryConfig.Duration = *storageServiceAndBucketAccessDuration
-				mc.SidecarRetryConfig.Jitter = *storageServiceAndBucketAccessJitter
-				err := mounter.SetupTokenAndStorageManager(ctx, nil /*k8sClientset*/, mc)
-				if err != nil {
-					klog.Fatalf("Failed to fetch identity pool and identity provider details required for bucket access check, got error %v", err)
-				}
-			}
-			if err := mounter.Mount(ctx, mc); err != nil {
-				mc.ErrWriter.WriteMsg(fmt.Sprintf("failed to mount bucket %q for volume %q: %v\n", mc.BucketName, mc.VolumeName, err))
+		if mc.EnableSidecarBucketAccessCheck {
+			mc.SidecarRetryConfig.Cap = *storageServiceAndBucketAccessCap
+			mc.SidecarRetryConfig.Steps = *storageServiceAndBucketAccessSteps
+			mc.SidecarRetryConfig.Factor = *storageServiceAndBucketAccessFactor
+			mc.SidecarRetryConfig.Duration = *storageServiceAndBucketAccessDuration
+			mc.SidecarRetryConfig.Jitter = *storageServiceAndBucketAccessJitter
+			err := mounter.SetupTokenAndStorageManager(ctx, nil /*k8sClientset*/, mc)
+			if err != nil {
+				klog.Fatalf("Failed to fetch identity pool and identity provider details required for bucket access check, got error %v", err)
 			}
 		}
+		if err := mounter.Mount(ctx, mc); err != nil {
+			mc.ErrWriter.WriteMsg(fmt.Sprintf("failed to mount bucket %q for volume %q: %v\n", mc.BucketName, mc.VolumeName, err))
+		}
 	}
 
 	c := make(chan os.Signal, 1)