ntc-templates icon indicating copy to clipboard operation
ntc-templates copied to clipboard

Normalize CDP, LLDP, ISDP capture groups

Open mjbear opened this issue 1 year ago • 3 comments

Normalization across several "discovery protocols", largely CDP and LLDP (though I did find broadcom_icos had ISDP as well). This huge effort started as a comment on #1405 and includes numerous capture group changes so it is likely the community will notice the breaking changes this PR brings.

Based off of #1468 so capture group documentation would end up in data_model.md

To the community: If I broke something, point it out, if I have time chances are I'll take ownership and fix it, but if not please realize I had good intentions and submit fix(es) in PR(s) to ntc-templates.

Hopefully all of this normalization brings positive change and improved usability for the project in the long term.

NOTE: it does seem like example raw Cisco IOS LLDP output had some data seemingly swapped (whether that's the LLDP sending or receiving end), but we end up with the output we have.

  • https://github.com/networktocode/ntc-templates/blob/master/tests/cisco_ios/show_lldp_neighbors_detail/cisco_ios_show_lldp_neighbors_detail2.yml
  • https://github.com/networktocode/ntc-templates/blob/master/tests/cisco_ios/show_lldp_neighbors_detail/cisco_ios_show_lldp_neighbors_detail2.raw
  • https://github.com/networktocode/ntc-templates/blob/master/ntc_templates/templates/cisco_ios_show_lldp_neighbors_detail.textfsm

mjbear avatar Sep 04 '23 18:09 mjbear

I am fixing up merge conflicts and putting this on @jvanderaa 's list :sweat_smile:

mjbear avatar Jan 07 '24 04:01 mjbear

ironed out some merge conflicts

mjbear avatar Feb 21 '24 01:02 mjbear

rebasing off of current NTC master branch to help alleviate merge conflicts for the maintainers

mjbear avatar Feb 21 '24 01:02 mjbear

Is this close? Happy to help.

ryanmerolle avatar Jun 19 '24 03:06 ryanmerolle

Hello @ryanmerolle,

This was waiting to be merged. (It was ready.) I'll see if I can make time to rebase it off the current main branch so Josh doesn't have merge conflicts and so on.

Thank you

mjbear avatar Jun 20 '24 16:06 mjbear

Thanks @mjbear I thought that I had all of them from earlier.

jvanderaa avatar Jun 20 '24 16:06 jvanderaa

This has been rebased.

mjbear avatar Jun 21 '24 17:06 mjbear