capacity: reject storage kind if capacity tracking is disabled

pohly · pohly · commit 65e48f2ca94f · 2021-02-23T11:34:39.000+01:00
The parameter becomes invalid when the driver is not configured
to have different storage kinds.
diff --git a/pkg/hostpath/hostpath.go b/pkg/hostpath/hostpath.go
@@ -286,6 +286,8 @@ func (hp *hostPath) createVolume(volID, name string, cap int64, volAccessType ac
 			}
 		}()
 		kind = actualKind
+	} else if kind != "" {
+		return nil, status.Error(codes.InvalidArgument, fmt.Sprintf("capacity tracking disabled, specifying kind %q is invalid", kind))
 	}
 
 	path := getVolumePath(volID)

Original file line number	Diff line number	Diff line change
`@@ -286,6 +286,8 @@ func (hp *hostPath) createVolume(volID, name string, cap int64, volAccessType ac`
`286`	`286`	`}`
`287`	`287`	`}()`
`288`	`288`	`kind = actualKind`
	`289`	`+ } else if kind != "" {`
	`290`	`+ return nil, status.Error(codes.InvalidArgument, fmt.Sprintf("capacity tracking disabled, specifying kind %q is invalid", kind))`
`289`	`291`	`}`
`290`	`292`
`291`	`293`	`path := getVolumePath(volID)`