Merge pull request #451 from andyzhangx/get-keyvault-logging

chore: add get key vault logs

Author: andyzhangx

pkg/blob/azure.go

@@ -121,9 +121,10 @@ func (d *Driver) getKeyVaultSecretContent(ctx context.Context, vaultURL string,
 		return "", fmt.Errorf("failed to get keyvaultClient: %v", err)
 	}
 
+	klog.V(2).Infof("get secret from vaultURL(%v), sercretName(%v), secretVersion(%v)", vaultURL, secretName, secretVersion)
 	secret, err := kvClient.GetSecret(ctx, vaultURL, secretName, secretVersion)
 	if err != nil {
-		return "", fmt.Errorf("failed to use vaultURL(%v), sercretName(%v), secretVersion(%v) to get secret: %v", vaultURL, secretName, secretVersion, err)
+		return "", fmt.Errorf("get secret from vaultURL(%v), sercretName(%v), secretVersion(%v) failed with error: %v", vaultURL, secretName, secretVersion, err)
 	}
 	return *secret.Value, nil
 }

pkg/blob/azure_test.go

@@ -235,7 +235,7 @@ func TestGetKeyVaultSecretContent(t *testing.T) {
 	}
 	d.cloud.AADClientID = "unit-test"
 	d.cloud.AADClientSecret = "unit-test"
-	expectedErr = fmt.Errorf("failed to use vaultURL(unit-test), sercretName(unit-test), secretVersion(v1) to get secret: keyvault.BaseClient#GetSecret: Failure preparing request: StatusCode=0 -- Original Error: autorest: No scheme detected in URL unit-test")
+	expectedErr = fmt.Errorf("get secret from vaultURL(unit-test), sercretName(unit-test), secretVersion(v1) failed with error: keyvault.BaseClient#GetSecret: Failure preparing request: StatusCode=0 -- Original Error: autorest: No scheme detected in URL unit-test")
 	_, err = d.getKeyVaultSecretContent(context.TODO(), valueURL, secretName, secretVersion)
 	if !reflect.DeepEqual(expectedErr, err) {
 		t.Errorf("actualErr: (%v), expectedErr: (%v)", err, expectedErr)