ansible-collection-netscaleradc icon indicating copy to clipboard operation
ansible-collection-netscaleradc copied to clipboard

[FEATURE REQUEST]: Align module params with the equivalent Terraform module and NITRO

Open gusmb opened this issue 1 year ago • 0 comments

Summary

I am trying to maintain a custom role where I aim to provide support for both Terraform and Ansible configuration backends. I find important to keep the module specs consistent between the 2 tools, so that it becomes trivial to auto-generate the corresponding code. Right now it is not the case for a couple of parameters which are not aligned with the NITRO expected key.

Module: servicegroup_servicegroupmember_binding

The “state” parameter only supports present or absent. This is OK to instruct Ansible to create/update or delete, but actually state should also support ENABLED or DISABLED, which is what NITRO expects the state param to be, to indicate the actual admin state of the binding. This is supported in Terraform, but not in Ansible. What do you think about extending the “state” param here to support the ENABLED or DISABLED binding?

Module: servicegroup_lbmonitor_binding

For this module, the NITRO attribute is “monitorname”, and this is also the case on the corresponding terraform module. However, for some reason the Ansible module expects the parameter to be “monitor_name”, with an underscore. This makes it difficult to work with as it is inconsistent between the 2 tools.

Module: gslbservicegroup_gslbservicegroupmember_binding

Same as the first, the “state” parameter only supports present or absent, but it also needs to support ENABLED and DISABLED to be able to idempotently configure a member binding with ENABLED/DISABLED state. ENABLED/DISABLED state should then assume that the resource should be present in the configuration, with that particular state.

Issue Type

Feature Idea

Component Name

servicegroup_servicegroupmember_binding, servicegroup_lbmonitor_binding, gslbservicegroup_gslbservicegroupmember_binding

Describe alternatives you've considered

No response

Additional Information


gusmb avatar Jun 14 '24 12:06 gusmb