vmware.vmware_rest: Generate action_groups

[mariolenz]

SUMMARY

Fixes #75

Generate action_groups for vmware.vmware_rest.

edit: Tested in ansible-collections/vmware.vmware_rest#461

ISSUE TYPE

• Feature Pull Request

COMPONENT NAME

plugins/action/generate_cloud_modules.py

ADDITIONAL INFORMATION

Follow-up to #76 which I've closed accidentally and can't reopen again.

[mariolenz]

@GomathiselviS Could you please review this? I've refactored the code a bit because it wasn't really feasible for a unit test... I mean, it wasn't a self-contained unit but part of a larger block of code. I hope it's OK now.

cc @alinabuzachis @NilashishC

[GomathiselviS]

Looks good to me. I've recommended adding a log message for clarity.

[mariolenz]

Looks good to me. I've recommended adding a log message for clarity.

done

[mariolenz]

I think the aws collection could make use of this, too. Maybe changing:

ansible.content_builder/plugins/action/generate_cloud_modules.py

Lines 1238 to 1244 in e775adc

	yaml_dict = {
	"requires_ansible": """>=2.11.0""",
	"action_groups": {"aws": []},
	"plugin_routing": {"modules": {}},
	}
	for m in module_list:
	yaml_dict["action_groups"]["aws"].append(m)

to something like:

yaml_dict = generate_runtime_yml("2.11.0", "aws", module_list)
yaml_dict["plugin_routing"] = {"modules": {}}

or:

yaml_dict = generate_runtime_yml(args.get("requires_ansible"), "aws", module_list)
yaml_dict["plugin_routing"] = {"modules": {}}

The latter would mean that requires_ansible should be also valid for aws, though.

BTW: This is jus a suggestion from my side. I won't implement this, I don't have anything to do with this collection 😏

[Yannik]

@mariolenz Great, thanks!