napalm-automation / napalm-nxos

Apache License 2.0
9 stars 21 forks source link

Feature request: Optional argument port #99

Closed pmusolino-rms closed 7 years ago

pmusolino-rms commented 7 years ago

Description of Issue/Question

It would be helpful to be able to specify the nxos port in addition to the transport in the cases where default of 80/443 is not used

Did you follow the steps from https://github.com/napalm-automation/napalm#faq

[X ] Yes [ ] No

Setup

napalm-nxos version

(Paste verbatim output from pip freeze | grep napalm-nxos between quotes below)

napalm-nxos==0.5.3

NX-OS version

(Paste verbatim output from show version | json between quotes below)

pipe to json broken on nexus 5548.  version 7.3(2)N1(1)

Steps to Reproduce the Issue

Error Traceback

(Paste the complete traceback of the exception between quotes below)

Its a feature request.  No errors.
ktbyers commented 7 years ago

@pmusolino-rms It already does this:

https://github.com/napalm-automation/napalm-nxos/blob/develop/napalm_nxos/nxos.py#L67-L70

If it doesn't work, just re-open the issue.