Fixed typos in comments.

Author: kcantrel

Infrastructure_as_Code/Ansible/Volume_Management/README.md

@@ -8,12 +8,13 @@ Workload Factory Link, please refer to the [NetApp Workload Factory documentatio
 
 The list of playbooks included in this folder is as follows:
 | Playbook Name | Description |
+|:-----|:------|
 | clone_volume.yaml | Clones an existing volume.|
 | create_cifs_share.yaml | Creates a new CIFS share on an existing volume.|
 | create_cifs_unix_symlink_mapping.yaml | Creates a CIFS symlink mapping. |
 | create_snapshot.yaml | Creates a snapshot of an existing volume.|
 | create_volume.yaml | Creates a new volume.|
-| create_volume_and_share.yaml | Creates a new volume with a CIFS share that points to it. It also sets autosize to grow and enables ONTAP efficiencies.|
+| create_volume_and_share.yaml | Creates a new volume with a CIFS share that points to it. It also enables ONTAP efficiencies and potentially sets the autosize mode.|
 | delete_cifs_share.yaml | Deletes an existing CIFS share.|
 | delete_snapshot.yaml | Deletes an existing snapshot.|
 | delete_volume.yaml | Deletes an existing volume.|
@@ -90,7 +91,7 @@ ok: [localhost]
 TASK [Set use_lambda to true if lambda_function_name is provided.] *********************************************************
 ok: [localhost]
 
-TASK [Set aws_provide to "default" if not provided.] ***********************************************************************
+TASK [Set aws_profile to "default" if not provided.] ***********************************************************************
 ok: [localhost]
 
 TASK [Set junction path to "/<volume_name>" if not provided.] **************************************************************

Infrastructure_as_Code/Ansible/Volume_Management/clone_volume.yaml

@@ -1,7 +1,7 @@
 # Title: clone_volume.yaml
 
 ---
-- name: Playbook to clones a volume in an FSx for ONTAP file system.
+- name: Playbook to clone a volume in an FSx for ONTAP file system.
   hosts: localhost
   collections:
     - netapp.ontap
@@ -30,7 +30,7 @@
         use_lambda: true
       when: lambda_function_name is defined
 
-    - name: Set aws_provide to "default" if not provided.
+    - name: Set aws_profile to "default" if not provided.
       set_fact:
         aws_profile: "default"
       when: aws_profile is not defined

Infrastructure_as_Code/Ansible/Volume_Management/create_cifs_share.yaml

@@ -1,4 +1,4 @@
-# Title: create_a_cifs_share
+# Title: create_cifs_share.yaml
 
 ---
 - name: Playbook to create a CIFS share
@@ -30,7 +30,7 @@
         use_lambda: true
       when: lambda_function_name is defined
 
-    - name: Set aws_provide to "default" if not provided.
+    - name: Set aws_profile to "default" if not provided.
       set_fact:
         aws_profile: "default"
       when: aws_profile is not defined

Infrastructure_as_Code/Ansible/Volume_Management/create_cifs_unix_symlink_mapping.yaml

@@ -32,7 +32,7 @@
         use_lambda: true
       when: lambda_function_name is defined
 
-    - name: Set aws_provide to "default" if not provided.
+    - name: Set aws_profile to "default" if not provided.
       set_fact:
         aws_profile: "default"
       when: aws_profile is not defined

Infrastructure_as_Code/Ansible/Volume_Management/create_volume.yaml

@@ -56,7 +56,7 @@
         password: "{{ lookup('amazon.aws.aws_secret', '{{ secret_name }}.password', nested=true) }}"
       no_log: true
 
-    - name: Set security_style to unix if not provide.
+    - name: Set security_style to unix if not provided.
       set_fact:
         security_style: "unix"
       when: security_style is not defined

Infrastructure_as_Code/Ansible/Volume_Management/create_volume_and_share.yaml

@@ -61,7 +61,7 @@
         junction_path: "/{{ volume_name }}"
       when: junction_path is not defined
 
-    - name: Set security_style to ntfs if not provide.
+    - name: Set security_style to ntfs if not provided.
       set_fact:
         security_style: "ntfs"
       when: security_style is not defined
@@ -81,7 +81,7 @@
         share_name: "{{ volume_name }}"
       when: share_name is not defined
 
-    - name: Give efficiency_mode a default if it hasn't already set.
+    - name: Give efficiency_mode a default if it hasn't already been set.
       set_fact:
         efficiency_mode: "enabled"
       when: efficiency_mode is not defined
@@ -127,8 +127,8 @@
           aws_region: "{{ aws_region }}"
           function_name: "{{ lambda_function_name }}"
         mode: "{{ autosize_mode }}"
-        maximum_size: "{{ autosize_maximum_size | default(omit) }}"
-        minimum_size: "{{ autosize_minimum_size | default(omit) }}"
+        maximum_size: "{{ autosize_max_size | default(omit) }}"
+        minimum_size: "{{ autosize_min_size | default(omit) }}"
         hostname: "{{ fsxn_hostname }}"
         username: "{{ username }}"
         password: "{{ password }}"

Infrastructure_as_Code/Ansible/Volume_Management/delete_cifs_share.yaml

@@ -1,4 +1,4 @@
-# Title: delete_a_cifs_share
+# Title: delete_cifs_share.yaml
 
 ---
 - name: Playbook to delete a CIFS share
@@ -29,7 +29,7 @@
         use_lambda: true
       when: lambda_function_name is defined
 
-    - name: Set aws_provide to "default" if not provided.
+    - name: Set aws_profile to "default" if not provided.
       set_fact:
         aws_profile: "default"
       when: aws_profile is not defined
@@ -55,7 +55,7 @@
         password: "{{ lookup('amazon.aws.aws_secret', '{{ secret_name }}.password', nested=true) }}"
       no_log: true
 
-    - name: Create CIFS Share
+    - name: Delete CIFS Share
       netapp.ontap.na_ontap_cifs:
         state: absent
         name: "{{ delete_share_name }}"

Infrastructure_as_Code/Ansible/Volume_Management/delete_volume_and_share.yaml

@@ -19,7 +19,6 @@
       when: vars[item] is undefined
       loop:
         - volume_name
-        - volume_size
         - vserver
         - secret_name
         - fsxn_hostname
@@ -30,7 +29,7 @@
         use_lambda: true
       when: lambda_function_name is defined
 
-    - name: Set aws_provide to "default" if not provided.
+    - name: Set aws_profile to "default" if not provided.
       set_fact:
         aws_profile: "default"
       when: aws_profile is not defined

Infrastructure_as_Code/Ansible/Volume_Management/get_volume_info.yaml

@@ -49,6 +49,11 @@
         lambda_function_name: ""
       when: lambda_function_name is not defined
 
+    - name: Ensure autosize_mode is valid.
+      fail:
+        msg: "autosize_mode must be either 'grow', 'grow_shrink', or 'off'."
+      when: autosize_mode not in ['grow', 'off', 'grow_shrink']
+
     - name: Get username and password from AWS secret
       set_fact:
         username: "{{ lookup('amazon.aws.aws_secret', '{{ secret_name }}.username', nested=true) }}"