ansible-ise icon indicating copy to clipboard operation
ansible-ise copied to clipboard

Inaccurate documentation

Open RyanMBess opened this issue 2 years ago • 5 comments

Seems a number of links in the documentation doesn't have all parameters marked as required. Here's an example https://docs.ansible.com/ansible/latest/collections/cisco/ise/endpoint_module.html#ansible-collections-cisco-ise-endpoint-module

Per the above link only ise_hostname, ise_password, ise_username are the only parameters needed to create and endpoint. Please update so we know what parameters are required to use this module (please do the same for all modules)

RyanMBess avatar Aug 31 '22 19:08 RyanMBess

Hi @RyanMBess This problem will be mentioned to Cisco since they are the ones who provide us with the API documentation to generate the libraries, and it is expected to be solved in future versions.

bvargasre avatar Sep 01 '22 04:09 bvargasre

Thank you.

RyanMBess avatar Sep 01 '22 12:09 RyanMBess

Hi @bvargasre, How goes updating the documentation. In this example https://docs.ansible.com/ansible/latest/collections/cisco/ise/sgt_module.html#ansible-collections-cisco-ise-sgt-module you MUST have the value attribute populated or you will be unable to create the SGT. Per the link it's not a required field.

RyanMBess avatar Oct 18 '22 15:10 RyanMBess

Cisco has been informed about the required problem, but we have not been provided with new documentation yet.

For the moment I updated the documentation of the required fields for the creation of an sgt from the sgt module. The change can be seen here https://ciscoise.github.io/ansible-ise/main/plugins/sgt_module.html#ansible-collections-cisco-ise-sgt-module The changes will be reflected in docs.ansible.com when Ansible releases a new version.

bvargasre avatar Oct 26 '22 02:10 bvargasre

All, I ran into this issue, also. I found the Cisco developer website that lists all required fields. I specifically used this link https://developer.cisco.com/docs/identity-services-engine/v1/#!selfregportal to create a self registered portal. The clincher for me was the settings.portSettings.authencationMethod field, which is a required UUID and not a text name.

Hopefully this will help others. The parent link is https://developer.cisco.com/docs/identity-services-engine/v1/ with the API reference being the source of field information.

rlangus avatar Apr 20 '23 16:04 rlangus