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

Configure Cisco NX-OS Vlan doc #49

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

justjais
Copy link
Member

Signed-off-by: Sumit Jaiswal [email protected]

  • Adding the doc for configuring VLANs on Cisco NX-OS device

Signed-off-by: Sumit Jaiswal <[email protected]>
@justjais justjais requested a review from trishnaguha November 22, 2018 09:28
Signed-off-by: Sumit Jaiswal <[email protected]>
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

Copy link

@samccann samccann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made an attempt to clarify a few things here. Not sure if I did actually clarify or not since I got confused part way through on the VLAN state, status, etc and what the final default values would be on the nxos device if the user didn't set anything here besides the VLAN ID.

# Configure VLANs on the device

The `configure_vlans` function can be used to set VLANs on Cisco NX-OS devices.
This function is only supported over `network_cli` connection type and
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This function is only supported over `network_cli` connection type and
This function is only supported over the `network_cli` connection type and

status: active
```

The above playbook will set the VLANs with ID, description, and address to particular
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The above playbook will set the VLANs with ID, description, and address to particular
The above playbook will set the VLANs with ID, description, and address to a particular

using either the `roles` directive or the `tasks` directive. If no other
options are provided, then all of the available facts will be collected for
the device.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## How to set VLANs using roles

This function is only supported over `network_cli` connection type and
requires the `ansible_network_os` value set to `nxos`.

## How to set VLANs on the device
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## How to set VLANs on the device

The above playbook will set the VLANs with ID, description, and address to particular
interface under the `nxos` top level key.

### Implement using tasks
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Implement using tasks
### How to set VLANs using tasks


### status

This sets the status for the VLAN Id for the Cisco NX-OS device.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This sets the status for the VLAN Id for the Cisco NX-OS device.
This is an optional parameter that sets the status for the VLAN ID for the Cisco NX-OS device. The valid values are `active` or `suspend`.


This sets the status for the VLAN Id for the Cisco NX-OS device.

The default value is `omit` which means even if the user doesn't pass the respective
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The default value is `omit` which means even if the user doesn't pass the respective
The default value is `omit` which means the role will continue to run, with the default status of `active`.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

making another guess here based on reading the nxos_vlan module docs...

This sets the status for the VLAN Id for the Cisco NX-OS device.

The default value is `omit` which means even if the user doesn't pass the respective
value the role will continue to run without any failure.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
value the role will continue to run without any failure.


### state

This sets the VLANs value to the Cisco NX-OS device and if the value of the state is changed
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This sets the VLANs value to the Cisco NX-OS device and if the value of the state is changed
This is an optional parameter that sets the VLAN state to the Cisco NX-OS device. The options are `present` or `absent`. If the value of the state is changed

This sets the VLANs value to the Cisco NX-OS device and if the value of the state is changed
to `absent`, the role will go ahead and try to delete the VLANs via the arguments passed.

The default value is `present` which means even if the user doesn't pass the respective
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The default value is `present` which means even if the user doesn't pass the respective
The default value is `present` which means even if the user doesn't pass the

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

Successfully merging this pull request may close these issues.

2 participants