Merge pull request #283758 from khdownie/kendownie080624

post GA fixes

Author: Jill Grant

articles/storage/container-storage/TOC.yml

@@ -17,7 +17,7 @@
   items:
   - name: Use with Azure Disks
     href: use-container-storage-with-managed-disks.md
-  - name: Use with Azure Elastic SAN (Preview)
+  - name: Use with Azure Elastic SAN
     href: use-container-storage-with-elastic-san.md
   - name: Use with Ephemeral Disk
     items:

articles/storage/container-storage/container-storage-aks-quickstart.md

@@ -122,15 +122,25 @@ Follow these guidelines when choosing a VM type for the cluster nodes. You must
 
 ## Install Azure Container Storage on your AKS cluster
 
-If you already have a preview instance of Azure Container Storage running on your cluster, you can skip this section. However, we recommend updating to the latest generally available (GA) version by running the following command. Remember to replace `<cluster-name>` and `<resource-group>` with your own values.
+The installation command is different depending on whether you already have a preview instance of Azure Container Storage running on your AKS cluster, or if you're installing Azure Container Storage on the cluster for the first time.
+
+### Upgrade a preview installation to GA
+
+If you already have a preview instance of Azure Container Storage running on your cluster, we recommend updating to the latest generally available (GA) version by running the following command. If you're installing Azure Container Storage for the first time on the cluster, proceed instead to [Install Azure Container Storage and create a storage pool](#install-azure-container-storage-and-create-a-storage-pool). You can also [Install Azure Container Storage on specific node pools](#install-azure-container-storage-on-specific-node-pools).
 
 ```azurecli-interactive
 az k8s-extension update --cluster-type managedClusters --cluster-name <cluster-name> --resource-group <resource-group> --name azurecontainerstorage --version 1.1.0 --auto-upgrade false --release-train stable
 ```
 
-Once your AKS cluster meets the [VM requirements](#ensure-vm-type-for-your-cluster-meets-the-following-criteria), run the following command to install Azure Container Storage on the cluster and create a storage pool. Replace `<cluster-name>` and `<resource-group>` with your own values. Replace `<storage-pool-type>` with `azureDisk`, `ephemeralDisk`, or `elasticSan`. If you select `ephemeralDisk`, you can also specify `--storage-pool-option`, and the values can be `NVMe` or `Temp`.
+Remember to replace `<cluster-name>` and `<resource-group>` with your own values.
+
+### Install Azure Container Storage and create a storage pool
+
+Before installing, ensure that your AKS cluster meets the [VM requirements](#ensure-vm-type-for-your-cluster-meets-the-following-criteria).
+
+Run the following command to install Azure Container Storage on the cluster and create a storage pool. Replace `<cluster-name>` and `<resource-group>` with your own values. Replace `<storage-pool-type>` with `azureDisk`, `ephemeralDisk`, or `elasticSan`. If you select `ephemeralDisk`, you can also specify `--storage-pool-option`, and the values can be `NVMe` or `Temp`.
 
-Running this command will enable Azure Container Storage on the system node pool, which by default is named `nodepool1`\*. If you want to enable it on other node pools, see [Install Azure Container Storage on specific node pools](#install-azure-container-storage-on-specific-node-pools). If you want to specify additional storage pool parameters, see [this table](container-storage-faq.md#storage-pool-parameters).
+Running this command will enable Azure Container Storage on the system node pool, which by default is named `nodepool1`\*. If you want to enable it on other node pools, see [Install Azure Container Storage on specific node pools](#install-azure-container-storage-on-specific-node-pools). If you want to specify additional parameters, see [Azure Container Storage storage pool parameters](container-storage-storage-pool-parameters.md).
 
 \*If there are any existing node pools with the `acstor.azure.com/io-engine:acstor` label then Azure Container Storage will be installed there by default. Otherwise, it's installed on the system node pool.
 

articles/storage/container-storage/use-container-storage-with-elastic-san.md

@@ -4,7 +4,7 @@ description: As a preview, you can configure Azure Container Storage to use Azur
 author: khdownie
 ms.service: azure-container-storage
 ms.topic: how-to
-ms.date: 07/23/2024
+ms.date: 08/06/2024
 ms.author: kendownie
 ms.custom: references_regions
 ---
@@ -19,11 +19,9 @@ As a preview feature, you can configure Azure Container Storage to use Azure Ela
 
 [!INCLUDE [container-storage-prerequisites](../../../includes/container-storage-prerequisites.md)]
 
-
 - Ensure you have either an [Azure Container Storage Owner](../../role-based-access-control/built-in-roles/containers.md#azure-container-storage-owner) role or [Azure Container Storage Contributor](../../role-based-access-control/built-in-roles/containers.md#azure-container-storage-contributor) role on your subscription. Either of these roles grant permissions that allow Azure Container Storage to communicate with the Elastic SAN resource. To make this change, go to your subscription page on the Azure portal. Select **Access control (IAM) > Add role assignment** and search for either "Azure Container Storage Owner" or "Azure Container Storage Contributor" in the **Job function roles** tab. Select **View > Assignments > Add assignment** and add your account.
 
-> [!NOTE]
-> To use Azure Container Storage with Azure Elastic SAN (preview), your AKS cluster should have a node pool of at least three [general purpose VMs](../../virtual-machines/sizes-general.md) such as **standard_d4s_v5** for the cluster nodes, each with a minimum of four virtual CPUs (vCPUs).
+- To use Azure Container Storage with Azure Elastic SAN (preview), your AKS cluster must have a node pool of at least three [general purpose VMs](../../virtual-machines/sizes-general.md) such as **standard_d4s_v5** for the cluster nodes, each with a minimum of four virtual CPUs (vCPUs).
 
 ## Limitations
 

articles/storage/container-storage/use-container-storage-with-managed-disks.md

@@ -17,8 +17,7 @@ ms.custom: references_regions
 
 [!INCLUDE [container-storage-prerequisites](../../../includes/container-storage-prerequisites.md)]
 
-> [!NOTE]
-> To use Azure Container Storage with Azure managed disks, your AKS cluster should have a node pool of at least three [general purpose VMs](../../virtual-machines/sizes-general.md) such as **standard_d4s_v5** for the cluster nodes, each with a minimum of four virtual CPUs (vCPUs).
+- To use Azure Container Storage with Azure managed disks, your AKS cluster must have a node pool of at least three [general purpose VMs](../../virtual-machines/sizes-general.md) such as **standard_d4s_v5** for the cluster nodes, each with a minimum of four virtual CPUs (vCPUs).
 
 ## Create and attach persistent volumes