[FEATURE]: dellemc.powermax.storagegroup - Make vol_name an optional argument

[jonathanhellman8]

Currently when using the storagegroup module to create new volumes to an existing storage group, vol_name needs to be provided to label the volume. Is there any way of not making vol_name a required input? There's no requirement in the storage array to label a volume when creating volumes for a SG. Currently I add a randomly generated string to vol_name to be able to use this module to create volumes to a SG over and over again without any vol_names conflicting.

Or is there any other way of achieving the very same thing without having to add a random string to vol_name?

• name: Create new volumes for existing SG dellemc.powermax.storagegroup: unispherehost: "{{unispherehost}}" universion: "{{universion}}" verifycert: "{{verifycert}}" user: "{{user}}" password: "{{password}}" serial_no: "{{serial_no}}" sg_name: "foo" state: "present" volumes:
  • vol_name: "bar" size: 1 cap_unit: "GB" vol_state: "present-in-group"

When not using a unique vol_name it does return:

"msg": "A volume with identifier bar but different size 6.0 GB already exists. Use a different identifier for volume creation. Currently, we support only unique identifiers for volume creation on PowerMax from Ansible"

[csalitros]

I could use this as well... I usually just tag the volume with the SG Name is going into.

[anupamaloke]

@csalitros, this feature is going to be part of v2.1.0 that is going to come out later this week.

[csalitros]

I'm able to fill and StorageGroup without volume names in 2.1.0 ... but if I want to have some type of name, it has to be unique.

For example, this creates the SG with 3 volumes: ` - name: Creating Storage Group {{ storagegroup_name }} with LUNs dellemc.powermax.storagegroup: <<: *unisphere_vars sg_name: "{{ storagegroup_name }}" service_level: "{{ service_level }}" srp: "SRP_1" compression: True state: "{{ sgstate }}" volumes:

• vol_name: size: "{{ size }}" cap_unit: "{{ unit }}"
• vol_name: size: "{{ size }}" cap_unit: "{{ unit }}"
• vol_name: size: "{{ size }}" cap_unit: "{{ unit }}" vol_state: "present-in-group" This creates the SG with 1 volume: - name: Creating Storage Group {{ storagegroup_name }} with LUNs dellemc.powermax.storagegroup: <<: *unisphere_vars sg_name: "{{ storagegroup_name }}" service_level: "{{ service_level }}" srp: "SRP_1" compression: True state: "{{ sgstate }}" volumes:
• vol_name: "{{ storagegroup_name }}" size: "{{ size }}" cap_unit: "{{ unit }}"
• vol_name: "{{ storagegroup_name }}" size: "{{ size }}" cap_unit: "{{ unit }}"
• vol_name: "{{ storagegroup_name }}" size: "{{ size }}" cap_unit: "{{ unit }}" vol_state: "present-in-group" `

[rawstorage]

if you omit the vol_name you can add multiple volumes in one change without the volume id, this is not an idempotent task though if you need your task to be idempotent then you should always use the volume name or add volume id for existing volumes to ensure idempotency.

[anupamaloke]

Closing the issue as it has been fixed by #28