Respond to comments

Author: dabradley

docs/errors.md

@@ -172,7 +172,7 @@ kubectl logs -n kube-system -l app=csi-azurelustre-controller -c azurelustre --t
 - This should return output such as:
 
 ```text
-There is not enough room in the /subscriptions/<sub-id>/resourceGroups/<rg>/providers/Microsoft.Network/virtualNetworks/<vnet>/subnets/<subnet> subnetID to fit a AMLFS-Durable-Premium-40 SKU cluster: 10 needed, 3 available
+There is not enough room in the /subscriptions/<sub-id>/resourceGroups/<rg>/providers/Microsoft.Network/virtualNetworks/<vnet>/subnets/<subnet> subnet to fit a AMLFS-Durable-Premium-40 SKU cluster: 10 needed, 3 available
 ```
 
 - Verify subnet details in Azure portal

pkg/azurelustre/controllerserver.go

@@ -61,7 +61,7 @@ const (
 
 var (
 	timeRegexp             = regexp.MustCompile(`^([01]?[0-9]|2[0-3]):[0-5][0-9]$`)
-	amlFilesystemNameRegex = regexp.MustCompile(`^[a-zA-Z0-9][a-zA-Z0-9_-]{0,78}[a-zA-Z0-9]$`)
+	amlFilesystemNameRegex = regexp.MustCompile(fmt.Sprintf(`^[a-zA-Z0-9][a-zA-Z0-9_-]{0,%d}[a-zA-Z0-9]$`, amlFilesystemNameMaxLength-2))
 )
 
 type SubnetProperties struct {

pkg/azurelustre/dynamic_provisioning.go

@@ -469,9 +469,9 @@ func (d *DynamicProvisioner) CheckSubnetCapacity(ctx context.Context, subnetInfo
 	}
 
 	if requiredSubnetIPSize > availableIPs {
-		klog.Warningf("There is not enough room in the %s subnetID to fit a %s SKU cluster: %v needed, %v available", subnetInfo.SubnetID, sku, requiredSubnetIPSize, availableIPs)
+		klog.Warningf("There is not enough room in the %s subnet to fit a %s SKU cluster: %v needed, %v available", subnetInfo.SubnetID, sku, requiredSubnetIPSize, availableIPs)
 		return false, nil
 	}
-	klog.V(2).Infof("There is enough room in the %s subnetID to fit a %s SKU cluster: %v needed, %v available", subnetInfo.SubnetID, sku, requiredSubnetIPSize, availableIPs)
+	klog.V(2).Infof("There is enough room in the %s subnet to fit a %s SKU cluster: %v needed, %v available", subnetInfo.SubnetID, sku, requiredSubnetIPSize, availableIPs)
 	return true, nil
 }