Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation for ansible_network_provider #18

Open
jwoogee opened this issue Nov 26, 2018 · 3 comments
Open

Documentation for ansible_network_provider #18

jwoogee opened this issue Nov 26, 2018 · 3 comments

Comments

@jwoogee
Copy link

jwoogee commented Nov 26, 2018

Is the expected action for users to set their inventory with the following variable?
ansible_network_provider: ansible-network.cisco_ios (or whatever os they are working with)

If so, are these roles built just to interact with ansible_network roles? If so, should we just have the ansible_network_provider determined automatically from the ansible_network_os?

@IPvSean
Copy link
Contributor

IPvSean commented Nov 26, 2018

hey @jwoogee at this time yes, you would see something like this->

Networking Platform Ansible Network Provider Role
Arista EOS ansible-network.arista_eos
Cisco IOS / IOS-XE ansible-network.cisco_ios
Cisco IOS-XR ansible-network.cisco_iosxr
Cisco NX-OS ansible-network.cisco_nxos
Juniper Junos ansible-network.juniper_junos
VyOS ansible-network.vyos

we might try to make this easier (defaults) in the future, but the goal is to make this super pluggable for other operating systems that are not these 6... (e.g. Dell, Huawei, etc) so that they can add support for a role without porting any code into the Ansible project

@jwoogee
Copy link
Author

jwoogee commented Nov 26, 2018

Thank you for the clarification. Would it make sense to add this variable requirement to the main READ.me or is there some other place you'd prefer? I can update the documentation, I'm just not sure where to.

@samccann
Copy link

How about adding a section called "Role Variables" and put it below the Requirements section? Then list the table @IPvSean has above? Also - how would a user of this role realize other operating systems have been added? I don't necessarily want to set it up that every new provider has to come in and update that table, but maybe that's the only option?
(sorry - asking more questions than providing answers here at the moment..)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants