This job view page is being replaced by Spyglass soon. Check out the new job view.
PRandyzhangx: feat: support private endpoint for smb protocol
ResultFAILURE
Tests 0 failed / 0 succeeded
Started2021-07-22 13:46
Elapsed4m15s
Revisionaafde6c91b65634eb8f805db26bed45144d1dba3
Refs 738

No Test Failures!


Error lines from build-log.txt

Activated service account credentials for: [pr-kubekins@kubernetes-jenkins-pull.iam.gserviceaccount.com]
+ WRAPPED_COMMAND_PID=31
+ wait 31
+ make verify
go test -v -race ./pkg/... ./test/utils/credentials
=== RUN   TestGetCloudProvider
E0722 13:48:17.220314   11198 azure_config.go:45] Failed to get cloud-config from secret: failed to get secret /: resource name may not be empty
--- PASS: TestGetCloudProvider (0.02s)
=== RUN   TestUpdateSubnetServiceEndpoints
=== RUN   TestUpdateSubnetServiceEndpoints/[fail]_no_subnet
=== RUN   TestUpdateSubnetServiceEndpoints/[success]_subnetPropertiesFormat_is_nil
=== RUN   TestUpdateSubnetServiceEndpoints/[success]_ServiceEndpoints_is_nil
=== RUN   TestUpdateSubnetServiceEndpoints/[success]_storageService_does_not_exists
=== RUN   TestUpdateSubnetServiceEndpoints/[success]_storageService_already_exists
=== RUN   TestUpdateSubnetServiceEndpoints/[fail]_SubnetsClient_is_nil
--- PASS: TestUpdateSubnetServiceEndpoints (0.00s)
    --- PASS: TestUpdateSubnetServiceEndpoints/[fail]_no_subnet (0.00s)
    --- PASS: TestUpdateSubnetServiceEndpoints/[success]_subnetPropertiesFormat_is_nil (0.00s)
    --- PASS: TestUpdateSubnetServiceEndpoints/[success]_ServiceEndpoints_is_nil (0.00s)
    --- PASS: TestUpdateSubnetServiceEndpoints/[success]_storageService_does_not_exists (0.00s)
    --- PASS: TestUpdateSubnetServiceEndpoints/[success]_storageService_already_exists (0.00s)
    --- PASS: TestUpdateSubnetServiceEndpoints/[fail]_SubnetsClient_is_nil (0.00s)
=== RUN   TestGetFileSvcClient
--- PASS: TestGetFileSvcClient (0.00s)
=== RUN   TestCreateFileShare
=== RUN   TestCreateFileShare/ShareOptions_empty
=== RUN   TestCreateFileShare/AccountName_invalid
--- PASS: TestCreateFileShare (0.00s)
... skipping 23 lines ...
--- PASS: TestIsCorruptedDir (0.00s)
=== RUN   TestNewDriver
--- PASS: TestNewDriver (0.00s)
=== RUN   TestGetFileURL
--- PASS: TestGetFileURL (0.00s)
=== RUN   TestGetAccountInfo
W0722 13:48:17.232327   11198 azurefile.go:492] parsing volumeID(uniqe-volumeid-nfs) return with error: error parsing volume id: "uniqe-volumeid-nfs", should at least contain two #
--- PASS: TestGetAccountInfo (0.00s)
=== RUN   TestCreateDisk
--- PASS: TestCreateDisk (4.45s)
=== RUN   TestGetFileShareQuota
--- PASS: TestGetFileShareQuota (0.00s)
=== RUN   TestRun
... skipping 120 lines ...
    --- PASS: TestCreateVolume/Valid_request (0.00s)
    --- PASS: TestCreateVolume/invalid_parameter (0.00s)
=== RUN   TestDeleteVolume
=== RUN   TestDeleteVolume/Volume_ID_missing
=== RUN   TestDeleteVolume/Controller_capability_missing
=== RUN   TestDeleteVolume/Invalid_volume_ID
E0722 13:48:21.713040   11198 controllerserver.go:453] GetFileShareInfo(vol_1) in DeleteVolume failed with error: error parsing volume id: "vol_1", should at least contain two #
=== RUN   TestDeleteVolume/Delete_file_share_returns_error
=== RUN   TestDeleteVolume/Valid_request
I0722 13:48:21.714072   11198 azurefile.go:674] remove tag(skip-matching) on account(f5713de20cde511e8ba4900) resourceGroup(vol_1)
W0722 13:48:21.714276   11198 controllerserver.go:484] RemoveStorageAccountTag(skip-matching) under rg(vol_1) account(f5713de20cde511e8ba4900) failed with Retriable: false, RetryAfter: 0s, HTTPStatusCode: 0, RawError: StorageAccountClient is nil
--- PASS: TestDeleteVolume (0.00s)
    --- PASS: TestDeleteVolume/Volume_ID_missing (0.00s)
    --- PASS: TestDeleteVolume/Controller_capability_missing (0.00s)
    --- PASS: TestDeleteVolume/Invalid_volume_ID (0.00s)
    --- PASS: TestDeleteVolume/Delete_file_share_returns_error (0.00s)
    --- PASS: TestDeleteVolume/Valid_request (0.00s)
=== RUN   TestControllerGetVolume
--- PASS: TestControllerGetVolume (0.00s)
=== RUN   TestControllerGetCapabilities
--- PASS: TestControllerGetCapabilities (0.00s)
=== RUN   TestValidateVolumeCapabilities
W0722 13:48:21.715547   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
--- PASS: TestValidateVolumeCapabilities (0.00s)
=== RUN   TestControllerPublishVolume
W0722 13:48:21.717216   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
--- PASS: TestControllerPublishVolume (0.00s)
=== RUN   TestControllerUnpublishVolume
--- PASS: TestControllerUnpublishVolume (0.00s)
=== RUN   TestCreateSnapshot
W0722 13:48:21.719153   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
--- PASS: TestCreateSnapshot (0.00s)
=== RUN   TestDeleteSnapshot
W0722 13:48:21.719624   11198 azurefile.go:492] parsing volumeID(vol_1#) return with error: error parsing volume id: "vol_1#", should at least contain two #
--- PASS: TestDeleteSnapshot (0.00s)
=== RUN   TestControllerExpandVolume
=== RUN   TestControllerExpandVolume/Volume_ID_missing
=== RUN   TestControllerExpandVolume/Volume_Capacity_range_missing
=== RUN   TestControllerExpandVolume/Volume_capabilities_missing
=== RUN   TestControllerExpandVolume/Invalid_Volume_ID
... skipping 10 lines ...
    --- PASS: TestControllerExpandVolume/Volume_capabilities_missing (0.00s)
    --- PASS: TestControllerExpandVolume/Invalid_Volume_ID (0.00s)
    --- PASS: TestControllerExpandVolume/Disk_name_not_empty (0.00s)
    --- PASS: TestControllerExpandVolume/Resize_file_share_returns_error (0.00s)
    --- PASS: TestControllerExpandVolume/Valid_request (0.00s)
=== RUN   TestGetShareURL
W0722 13:48:21.723079   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
--- PASS: TestGetShareURL (0.00s)
=== RUN   TestGetServiceURL
W0722 13:48:21.723774   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
E0722 13:48:21.724032   11198 controllerserver.go:866] parse serviceURLTemplate error: parse "https://^f5713de20cde511e8ba4900.file.abc": invalid character "^" in host name
--- PASS: TestGetServiceURL (0.00s)
=== RUN   TestSnapshotExists
W0722 13:48:21.724690   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
--- PASS: TestSnapshotExists (0.00s)
=== RUN   TestGetCapacity
--- PASS: TestGetCapacity (0.00s)
=== RUN   TestListVolumes
--- PASS: TestListVolumes (0.00s)
=== RUN   TestListSnapshots
... skipping 15 lines ...
--- PASS: TestGetPluginCapabilities (0.00s)
=== RUN   TestNodeGetInfo
--- PASS: TestNodeGetInfo (0.00s)
=== RUN   TestNodeGetCapabilities
--- PASS: TestNodeGetCapabilities (0.00s)
=== RUN   TestNodePublishVolume
E0722 13:48:21.729383   11198 nodeserver.go:432] MakeDir failed on target: /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/azure.go (mkdir /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/azure.go: not a directory)
--- PASS: TestNodePublishVolume (0.00s)
=== RUN   TestNodeUnpublishVolume
W0722 13:48:21.730459   11198 mount_helper_common.go:34] Warning: Unmount skipped because path does not exist: /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/abc.go
--- PASS: TestNodeUnpublishVolume (0.00s)
=== RUN   TestNodeStageVolume
W0722 13:48:21.731537   11198 azurefile.go:492] parsing volumeID(vol_1) return with error: error parsing volume id: "vol_1", should at least contain two #
I0722 13:48:21.732256   11198 mount_linux.go:408] Disk "/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount/test_disk" appears to be unformatted, attempting to format as type: "ext4" with options: [-F -m0 /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount/test_disk]
E0722 13:48:21.732436   11198 mount_linux.go:414] format of disk "/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount/test_disk" failed: type:("ext4") target:("/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/source_test") options:("barrier=1,errors=remount-ro,loop,noatime,defaults") errcode:(formatting failed) output:() 
I0722 13:48:21.732878   11198 mount_linux.go:408] Disk "/home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount/test_disk" appears to be unformatted, attempting to format as type: "ext4" with options: [-F -m0 /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount/test_disk]
I0722 13:48:21.732952   11198 mount_linux.go:418] Disk successfully formatted (mkfs): ext4 - /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount/test_disk /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/source_test
--- PASS: TestNodeStageVolume (0.00s)
=== RUN   TestNodeUnstageVolume
W0722 13:48:21.733921   11198 mount_helper_common.go:34] Warning: Unmount skipped because path does not exist: /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/abc.go
W0722 13:48:21.734008   11198 mount_helper_common.go:34] Warning: Unmount skipped because path does not exist: /home/prow/go/src/sigs.k8s.io/azurefile-csi-driver/pkg/azurefile/proxy-mount
--- PASS: TestNodeUnstageVolume (0.00s)
=== RUN   TestNodeGetVolumeStats
--- PASS: TestNodeGetVolumeStats (0.02s)
=== RUN   TestEnsureMountPoint
W0722 13:48:21.751636   11198 nodeserver.go:423] ReadDir ./false_is_likely_target failed with open ./false_is_likely_target: no such file or directory, unmount this directory
E0722 13:48:21.751778   11198 nodeserver.go:432] MakeDir failed on target: ./azure.go (mkdir ./azure.go: not a directory)
--- PASS: TestEnsureMountPoint (0.00s)
=== RUN   TestMakeDir
--- PASS: TestMakeDir (0.00s)
=== RUN   TestNodeExpandVolume
--- PASS: TestNodeExpandVolume (0.00s)
=== RUN   TestSimpleLockEntry
... skipping 276 lines ...
-
 			ctx, cancel := getContextWithCancel()
 			defer cancel()
-			if rerr := az.StorageAccountClient.Create(ctx, resourceGroup, accountName, cp); rerr != nil {
+			rerr := az.StorageAccountClient.Create(ctx, resourceGroup, accountName, cp)
+			if rerr != nil {
 				return "", "", fmt.Errorf("failed to create storage account %s, error: %v", accountName, rerr)
 			}
 
@@ -316,10 +315,7 @@ func (az *Cloud) createPrivateDNSZone(ctx context.Context) error {
 	location := LocationGlobal
 	privateDNSZone := privatedns.PrivateZone{Location: &location}
 	if err := az.privatednsclient.CreateOrUpdate(ctx, az.ResourceGroup, PrivateDNSZoneName, privateDNSZone, true); err != nil {
-		if strings.Contains(err.Error(), "exists already") {
-			klog.V(2).Infof("private dns zone(%s) in resourceGroup (%s) already exists", PrivateDNSZoneName, az.ResourceGroup)
-			return nil
-		}
+		return err
 	}
 	return nil
 }
@@ -328,11 +324,12 @@ func (az *Cloud) createVNetLink(ctx context.Context, vNetLinkName string) error
 	klog.V(2).Infof("Creating virtual link for vnet(%s) and DNS Zone(%s) in resourceGroup(%s)", vNetLinkName, PrivateDNSZoneName, az.ResourceGroup)
 	location := LocationGlobal
 	vnetID := fmt.Sprintf("/subscriptions/%s/resourceGroups/%s/providers/Microsoft.Network/virtualNetworks/%s", az.SubscriptionID, az.ResourceGroup, az.VnetName)
+	registrationEnabled := false
 	parameters := privatedns.VirtualNetworkLink{
 		Location: &location,
... skipping 17 lines ...
+		return false
 	}
-	return false
+	return true
 }

error
make: *** [Makefile:69: verify] Error 1
+ EXIT_VALUE=2
+ set +o xtrace