doc: deprecate azure blob disk feature

andyzhangx · andyzhangx · commit f007c68ae9b5 · 2020-07-08T10:48:58.000Z
diff --git a/pkg/volume/azure_dd/azure_common.go b/pkg/volume/azure_dd/azure_common.go
@@ -59,8 +59,8 @@ var (
 		string(api.AzureDataDiskCachingReadWrite))
 
 	supportedDiskKinds = sets.NewString(
-		string(api.AzureSharedBlobDisk),
-		string(api.AzureDedicatedBlobDisk),
+		string(api.AzureSharedBlobDisk),    // deprecated
+		string(api.AzureDedicatedBlobDisk), // deprecated
 		string(api.AzureManagedDisk))
 
 	// only for Windows node
diff --git a/pkg/volume/azure_dd/azure_provision.go b/pkg/volume/azure_dd/azure_provision.go
@@ -303,7 +303,7 @@ func (p *azureDiskProvisioner) Provision(selectedNode *v1.Node, allowedTopologie
 		if err != nil {
 			return nil, err
 		}
-	} else {
+	} else { // Attention: blob disk feature is deprecated
 		if kind == v1.AzureDedicatedBlobDisk {
 			_, diskURI, _, err = diskController.CreateVolume(name, account, storageAccountType, location, requestGiB)
 			if err != nil {
diff --git a/staging/src/k8s.io/legacy-cloud-providers/azure/azure_blobDiskController.go b/staging/src/k8s.io/legacy-cloud-providers/azure/azure_blobDiskController.go
@@ -40,6 +40,7 @@ import (
 	"k8s.io/klog/v2"
 )
 
+// Attention: blob disk feature is deprecated
 const (
 	vhdContainerName         = "vhds"
 	useHTTPSForBlobBasedDisk = true

Original file line number	Diff line number	Diff line change
`@@ -303,7 +303,7 @@ func (p azureDiskProvisioner) Provision(selectedNode v1.Node, allowedTopologie`
`303`	`303`	`if err != nil {`
`304`	`304`	`return nil, err`
`305`	`305`	`}`
`306`		`- } else {`
	`306`	`+ } else { // Attention: blob disk feature is deprecated`
`307`	`307`	`if kind == v1.AzureDedicatedBlobDisk {`
`308`	`308`	`_, diskURI, _, err = diskController.CreateVolume(name, account, storageAccountType, location, requestGiB)`
`309`	`309`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,7 @@ import (`
`40`	`40`	`"k8s.io/klog/v2"`
`41`	`41`	`)`
`42`	`42`
	`43`	`+// Attention: blob disk feature is deprecated`
`43`	`44`	`const (`
`44`	`45`	`vhdContainerName = "vhds"`
`45`	`46`	`useHTTPSForBlobBasedDisk = true`