cisco.nxos icon indicating copy to clipboard operation
cisco.nxos copied to clipboard

nxos_nxapi changes to support MDS

Open srbharadwaj opened this issue 3 years ago • 2 comments

SUMMARY

nxos_nxapi changes to support MDS

ISSUE TYPE
  • Bug Report
COMPONENT NAME

nxos_nxapi

These will be changes that will need to be incorporated Let me go by each parameter

state:

            Present – will send only `feature nxapi`

            Absent - will send only `no feature nxapi`

http:

            No – will send only `no nxapi http`

            Yes – if set to yes then `http_port` must be passed as well, if not an appropriate error must be thrown

http_port:

            port number for http, this should be accompanied with the parameter `http`, there will be no default port explicitly sent

https:

            No – will send only `no nxapi https`

            Yes – if set to yes then `https_port` must be passed as well, if not an appropriate error must be thrown

https_port:

            port number for https, this should be accompanied with the parameter `https`, there will be no default port explicitly sent

sandbox:

            No – will send only `no nxapi sandbox`

            Yes- will send only  `nxapi sandbox`, platform check will be done to send only to MDS and N7K series and for other platforms an appropriate error will be thrown

ssl_strong_ciphers:

            No – will send only `nxapi ssl ciphers weak`

            Yes – will send only `no nxapi ssl ciphers weak` , again there will be no default set for this option

New key ssl_protocols will be added which will be mutually exclusive with the existing parameters tlsv1_0, tlsv1_1, tlsv1_2, which will eventually be deprecated

srbharadwaj avatar Mar 03 '22 06:03 srbharadwaj

@NilashishC @mikewiebe Please add mds attribute to it

srbharadwaj avatar Mar 03 '22 07:03 srbharadwaj

Discussed with @NilashishC We will be implementing a resource module of nxapi instead of changing the logic for the existing nxapi module

srbharadwaj avatar Jul 11 '22 14:07 srbharadwaj