Merge pull request #21 from csi-driver/set-diferent-tmp-path

andyzhangx · web-flow · commit bb1e32c755c2 · 2019-04-13T08:58:44.000+08:00
use different tmp-path value for different blobfuse mounts
diff --git a/pkg/blobfuse/nodeserver.go b/pkg/blobfuse/nodeserver.go
@@ -114,8 +114,8 @@ func (d *Driver) NodePublishVolume(ctx context.Context, req *csi.NodePublishVolu
 		}
 	}
 
-	// todo: use could set tmp-path and other mountOptions
-	cmd := exec.Command("/usr/blob/blobfuse", targetPath, "--tmp-path=/mnt/blobfuse/", "--container-name="+containerName)
+	// todo: user could set tmp-path and other mountOptions
+	cmd := exec.Command("/usr/blob/blobfuse", targetPath, "--tmp-path=/mnt/"+volumeID, "--container-name="+containerName)
 	cmd.Env = append(os.Environ(), "AZURE_STORAGE_ACCOUNT="+accountName, "AZURE_STORAGE_ACCESS_KEY="+accountKey)
 	err = cmd.Run()
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -114,8 +114,8 @@ func (d Driver) NodePublishVolume(ctx context.Context, req csi.NodePublishVolu`
`114`	`114`	`}`
`115`	`115`	`}`
`116`	`116`
`117`		`- // todo: use could set tmp-path and other mountOptions`
`118`		`- cmd := exec.Command("/usr/blob/blobfuse", targetPath, "--tmp-path=/mnt/blobfuse/", "--container-name="+containerName)`
	`117`	`+ // todo: user could set tmp-path and other mountOptions`
	`118`	`+ cmd := exec.Command("/usr/blob/blobfuse", targetPath, "--tmp-path=/mnt/"+volumeID, "--container-name="+containerName)`
`119`	`119`	`cmd.Env = append(os.Environ(), "AZURE_STORAGE_ACCOUNT="+accountName, "AZURE_STORAGE_ACCESS_KEY="+accountKey)`
`120`	`120`	`err = cmd.Run()`
`121`	`121`	`if err != nil {`