Skip to content
This repository has been archived by the owner on May 15, 2019. It is now read-only.

Add Basic Ethernet State parsing for NXOS and EOS #124

Merged
merged 2 commits into from
Feb 3, 2018

Conversation

tcaiazza
Copy link
Contributor

@tcaiazza tcaiazza commented Feb 1, 2018

There are some useful states in the oc-ethernet model, like crc errors, that aren't currently being pulled in by the oc-interface state parsing. These changes will add some of these useful ethernet stats/states.

Along with the basic parser changes I added a new jinja filter. With the nxos mac addresses I needed a way to convert cisco style mac addresses into unix style, so I added a jinja
filter to do that.

Also added a note to the writing profiles section of the docs about writing parsers for models that are called by other models not needing to execute commands.

There are some useful states in the oc-ethernet model, like crc errors,
that aren't currently being pulled in by the oc-interface state parsing.
These changes will add some of these useful ethernet stats/states

As part of the parsing of the nxos mac addresses I needed a way to
convert cisco style mac addresses into unix style, so I added a jinja
filter to do that.

Also added a note to the writing profiles section of the docs about
writing parsers  for models that are called by other models not needing
to execute commands.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 84.331% when pulling 756eb18 on tcaiazza:ethernet_state into b9bb62b on napalm-automation:develop.

@dbarrosop
Copy link
Member

Awesome work as usual :D

@dbarrosop dbarrosop merged commit 240630e into napalm-automation:develop Feb 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants