chore: refactor cloud config read logic

revert

Author: andyzhangx

docs/driver-parameters.md

@@ -15,6 +15,7 @@ storageAccount | specify Azure storage account name| STORAGE_ACCOUNT_NAME | - No
 protocol | specify blobfuse mount or NFSv3 mount | `fuse`, `nfs` | No | `fuse`
 containerName | specify the existing container name | existing container name | No | if empty, driver will create a new container name, starting with `pvc-fuse` for blobfuse or `pvc-nfs` for NFSv3
 server | specify Azure storage account server address | existing server address, e.g. `accountname.privatelink.blob.core.windows.net` | No | if empty, driver will use default `accountname.blob.core.windows.net` or other sovereign cloud account address
+storageEndpointSuffix | specify Azure storage endpoint suffix for agent node | `core.windows.net` | No | if empty, driver will use default storage endpoint suffix `core.windows.net`
 tags | [tags](https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/tag-resources) would be created in newly created storage account | tag format: 'foo=aaa,bar=bbb' | No | ""
 
  - `fsGroup` securityContext setting

pkg/blob/azure.go

@@ -87,23 +87,26 @@ func getCloudProvider(kubeconfig, nodeID string) (*azureprovider.Cloud, error) {
 		}
 	}
 
-	if kubeClient != nil {
+	// reassign kubeClient
+	if kubeClient != nil && az.KubeClient == nil {
 		az.KubeClient = kubeClient
 	}
+
 	if isController {
 		if err != nil {
+			// controller depends on cloud config
 			return az, err
 		}
 		// Disable UseInstanceMetadata for controller to mitigate a timeout issue using IMDS
 		// https://github.com/kubernetes-sigs/azuredisk-csi-driver/issues/168
-		klog.Infoln("disable UseInstanceMetadata for controller")
+		klog.V(2).Infof("disable UseInstanceMetadata for controller server")
 		az.Config.UseInstanceMetadata = false
-		klog.Infoln("Starting the controller server...")
+		klog.V(2).Infof("Starting controller server...")
 	} else {
 		if err != nil {
-			klog.V(2).Infof("there is no azure.json provided for node server, error: %v", err)
+			klog.V(2).Infof("no cloud config provided, error: %v, node server will run without cloud config", err)
 		}
-		klog.V(2).Infof("Starting the node server, nodeID is(%s)", nodeID)
+		klog.V(2).Infof("Starting node server on node(%s)", nodeID)
 	}
 
 	return az, nil