-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Nxos become #39859
Nxos become #39859
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Do we want people to use
authorize
- This functionality is only in 2.5.3 (?) onwards. That needs to be clearer in the docs
@trishnaguha How come this is getting backported, Is it counted as a bug fix, rather than feature work?
@@ -25,8 +25,9 @@ Connections Available | |||
| | | | | | Requires ``transport: nxapi`` | | |||
| | | | | | in the ``provider`` dictionary | | |||
+---------------------------+-----------------------------------------------+-----------------------------------------+ | |||
| | **Enable Mode** | | not supported by NXOS | | not supported by NXOS | | |||
| | (Privilege Escalation) | | | | | |||
| | **Enable Mode** | | supported - use ``ansible_become: yes`` | | supported - use ``authorize: yes`` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note about 2.5.3 here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed by 119f45e286d59b54916965f7ce0aaec86bac57cf?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
@gundalow bugfix |
@@ -96,6 +100,8 @@ Example NX-API ``group_vars/nxos.yml`` | |||
nxapi: | |||
host: "{{ inventory_hostname }}" | |||
transport: nxapi | |||
authorize: yes | |||
auth_pass: !vault... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this also needs to be removed from nxapi provider spec.
Per discussion with the cisco nxos team, nxapi itself doesn't support enable mode - reflected in the docs. |
Please backport into |
(cherry picked from commit 64f8786)
cherry-picked to 2.5 #40221 |
SUMMARY
Updates network documentation to cover changes in #39355
ISSUE TYPE
COMPONENT NAME
docs.ansible.com
ANSIBLE VERSION
2.5
Will backport to
stable-2.5