Skip to content

Add Mappings for DNA Center - #519

Merged
itdependsnetworks merged 6 commits into
networktocode:developfrom
jdrew82:feat-add_dnac_map
May 16, 2024
Merged

Add Mappings for DNA Center#519
itdependsnetworks merged 6 commits into
networktocode:developfrom
jdrew82:feat-add_dnac_map

Conversation

@jdrew82

@jdrew82 jdrew82 commented May 6, 2024

Copy link
Copy Markdown
Contributor

This PR is to add the forward and reverse mappings for DNA Center. I'm a bit unsure how to handle IOS-XE for the reverse though as you can't have two cisco_ios keys so I changed the one for XE to be cisco_xe. I'd match it in the forward but the XE library mapping in Nautobot is missing some essential bits for IOS-XE.

@itdependsnetworks

Copy link
Copy Markdown
Contributor

We generally do the XE in 1 direction, but not the other

"cisco_xe": "iosxe", # no reverse

Comment thread netutils/lib_mapper.py Outdated
@itdependsnetworks

Copy link
Copy Markdown
Contributor

Sorry, should have mentioned (and we should add to the docs), need to add to

"LIB_MAPPER_TEMPLATE": {
and then run the development_script (if you run tests, it will tell you the command).

@jdrew82

jdrew82 commented May 14, 2024

Copy link
Copy Markdown
Contributor Author

Sorry, should have mentioned (and we should add to the docs), need to add to

"LIB_MAPPER_TEMPLATE": {
and then run the development_script (if you run tests, it will tell you the command).

Doh! I always forget to do that for this project...

@itdependsnetworks

Copy link
Copy Markdown
Contributor

In fairness, we should have it documented.

Comment thread development_scripts.py Outdated
@itdependsnetworks

Copy link
Copy Markdown
Contributor

now you have to re-run development script.

@jdrew82

jdrew82 commented May 15, 2024

Copy link
Copy Markdown
Contributor Author

now you have to re-run development script.

Sorry, meant to run that. Should be done now.

@itdependsnetworks
itdependsnetworks merged commit 11f5e88 into networktocode:develop May 16, 2024
@jdrew82
jdrew82 deleted the feat-add_dnac_map branch June 28, 2024 17:20
michalis1 pushed a commit to michalis1/netutils that referenced this pull request Feb 2, 2026
* feat: ✨ Add forward and reverse mapping for DNA Center
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