fix empty network_interface_groups in the module info

[genalt]

SUMMARY

In the module info we can gather Fusion information using network_interface_groups. It should returns the same information as nigs, but instead it returns nothing. See code here

Fusion-Collection/plugins/modules/fusion_info.py

Line 910 in cdd8c2e

if "nigs" in subset or "all" in subset:

I fixed the problem and added information for network_interface_groups.

ISSUE TYPE

• Bugfix Pull Request
• Docs Pull Request

COMPONENT NAME

Module fusion_info.

ADDITIONAL INFORMATION

• changelog

bugfixes:
  - fusion_info - network_interface_groups subset returned nothing. Now it collects the same information
    as nigs subset (https://github.com/Pure-Storage-Ansible/Fusion-Collection/pull/46).
deprecated_features:
  - fusion_info - nigs subset is deprecated in favor of network_interface_groups and will be removed
    in the version 1.7.0 (https://github.com/Pure-Storage-Ansible/Fusion-Collection/pull/46).

[CLAassistant]

All committers have signed the CLA.

[sdodsley]

You can't really deprecate a parametr in Ansible without a lot of warning. Use alias instead.

[sdodsley]

Use alias

[sdodsley]

Please add a release note in changelog/fragments. Name the file <PR#>_<short description>.yaml
See https://docs.ansible.com/ansible/latest/community/development_process.html#creating-a-changelog-fragment

[genalt]

You can't really deprecate a parametr in Ansible without a lot of warning. Use alias instead.

Thank you for the review @sdodsley. I am not deprecating any argument (parameter). It is still gather_subset

argument_spec.update(
        dict(gather_subset=dict(default="minimum", type="list", elements="str"))
    )

We use aliases to give another name to the argument, so this is not the case here.

My initial intension was to not allow to use value nigs and use network_interface_groups instead.

I am OK, to keep both of them just to no introduce backward incompatible changes.

[sdodsley]

Please create a releaee note for this as well in changelog/fragments. There is a naming convention for this files... <PR#>_<short description>.yaml

[genalt]

Please create a releaee note for this as well in changelog/fragments. There is a naming convention for this files... <PR#>_<short description>.yaml

Added changelogs/fragments/PR46_network_interface_groups.yaml with information