search

fortinet-ansible-dev / ansible-galaxy-fortios-sphinxdoc

https://ansible-galaxy-fortios-docs.readthedocs.io/en/latest/
2 stars 7 forks source link

fortios_configuration_fact.rst contains selectors which are not valid #3

Closed JonTheNiceGuy closed 3 years ago

JonTheNiceGuy commented 3 years ago

I have tried using execute__tree__reboot as the selector for fortinet.fortios.fortios_configuration_fact, but when I got a Traceback, and checked the code, I noticed that all the selectors containing the string __ (e.g. diagnose__tree___settings, or execute__tree__reboot) are not listed, so there is a mismatch between the selector list and the code.

chillancezen commented 3 years ago

hi @JonTheNiceGuy Jon,

do you remember what specific FortiOS version you saw execute__tree.reboot or diagnose__tree.settings in?

we generated collection based on following FortiOS major version: https://ansible-galaxy-fortios-docs.readthedocs.io/en/latest/release.html#release-targets

I did not search them in their corresponding schema files.

so I guess the selector you missed must be existing in earlier versions. I am not sure.

thanks, Link

JonTheNiceGuy commented 3 years ago

This relates to https://github.com/fortinet-ansible-dev/fortios-ansible-generator/pull/19 which has just been merged :)

chillancezen commented 3 years ago

fixed in latest document: https://ansible-galaxy-fortios-docs.readthedocs.io/en/latest/fortios_configuration_fact.html