Skip to content

Conversation

@mirceaulinic
Copy link
Contributor

@mirceaulinic mirceaulinic commented Jul 12, 2018

What does this PR do?

Adds the initial version of two new modules: one Proxy and one Execution module for managing Cisco Nexus switches via via NX-API. Well, the NX-API is not the most reliable API in this world, but it's there and it can be used up to some extent.

The execution module is flexible enough to execute the commands on the (remote) device without necessarily requiring to be running inside the nxos_api Proxy Minion. This goes well with its SSH-based counterpart https://docs.saltstack.com/en/latest/ref/modules/all/salt.modules.nxos.html providing good coverage for NXOS switches.

The most important side of these modules is the fact that it has no external dependencies, as everything is HTTP-based. Not only that it's easier to get it up and running, but that also means it's straight forward to have it running under a regular Minion: on Nexus switches it is possible to install the Salt Minion under a container running in a what-is-called OAC (why keep it simple when can you make it awkward, right?). Anyway, the idea is that the user only has to execute yum install salt-minion in that OAC, then can start using: salt-call --local nxos_api.show 'show bgp sessions' without any further headache.
If, for whatever reasons, the user can't / is not allowed / the hardware does not support the OAC thing, can always fallback to using the nxos_api Proxy instead.

@mirceaulinic mirceaulinic force-pushed the nxos-api-mod branch 4 times, most recently from bdb7944 to 075cfcf Compare July 12, 2018 12:51
Copy link
Contributor

@gtmanfred gtmanfred left a comment

Choose a reason for hiding this comment

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

blind-review

@rallytime rallytime merged commit ff3ff80 into saltstack:develop Jul 16, 2018
@mirceaulinic mirceaulinic deleted the nxos-api-mod branch August 9, 2018 07:58
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.

3 participants