fix: ignore snapshot keys in snapshot storage class

kubernetes-sigs · Sep 20, 2024 · 0044ec7 · 0044ec7
1 parent 8bf4dba
commit 0044ec7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/pkg/azureconstants/azure_constants.go b/pkg/azureconstants/azure_constants.go
@@ -56,6 +56,9 @@ const (
 	PvcNamespaceTag                   = "kubernetes.io-created-for-pvc-namespace"
 	PvcNameTag                        = "kubernetes.io-created-for-pvc-name"
 	PvNameKey                         = "csi.storage.k8s.io/pv/name"
+	VolumeSnapshotNameKey             = "csi.storage.k8s.io/volumesnapshot/name"
+	VolumeSnapshotNamespaceKey        = "csi.storage.k8s.io/volumesnapshot/namespace"
+	VolumeSnapshotContentNameKey      = "csi.storage.k8s.io/volumesnapshotcontent/name"
 	PvNameTag                         = "kubernetes.io-created-for-pv-name"
 	RateLimited                       = "rate limited"
 	RequestedSizeGib                  = "requestedsizegib"

diff --git a/pkg/azuredisk/controllerserver.go b/pkg/azuredisk/controllerserver.go
@@ -1012,6 +1012,12 @@ func (d *Driver) CreateSnapshot(ctx context.Context, req *csi.CreateSnapshotRequ
 			dataAccessAuthMode = v
 		case consts.TagValueDelimiterField:
 			tagValueDelimiter = v
+		case consts.VolumeSnapshotNameKey:
+			// ignore the key
+		case consts.VolumeSnapshotNamespaceKey:
+			// ignore the key
+		case consts.VolumeSnapshotContentNameKey:
+			// ignore the key
 		default:
 			return nil, status.Errorf(codes.Internal, "AzureDisk - invalid option %s in VolumeSnapshotClass", k)
 		}