Skip to content

Conversation

@Mikeg2881
Copy link
Contributor

@Mikeg2881 Mikeg2881 commented Nov 4, 2025

resolves #2221

This PR adds a new TextFSM template for Arista EOS devices to parse the output of the show interfaces transceiver hardware command.

@mjbear mjbear changed the title Add Arista EOS show interfaces transceiver hardware template (issue #2221) Add Arista EOS show int transceiver hardware Nov 4, 2025
@mjbear mjbear added the changes_requested Waiting on user to address feedback label Nov 4, 2025
- Reduced the duplicate test data in the 01 raw file
- Regenerated structured data (yml) for 01 raw file
- Removed yaml file with extension that doesn't match project CI
- Update index to make the template available
- Update the template so it parses correctly
- Regenerate 01 yml file
arista_eos_show_interfaces_transceiver_hardware_01.yml
Modify index pattern for 'sibling' command sh int trans detail since it
would have the same pattern for several words.
@mjbear mjbear force-pushed the feature/arista_eos_show_interfaces_transceiver_hardware branch from 2e61b48 to 759ee99 Compare November 8, 2025 20:44
@mjbear
Copy link
Collaborator

mjbear commented Nov 8, 2025

I made the mistake of being on the wrong branch (for 2220) since we've been working on more than one Arista template. 😞

I've removed commits and added them back on the correct branch to push.

@mjbear mjbear removed the changes_requested Waiting on user to address feedback label Nov 8, 2025
Copy link
Collaborator

@mjbear mjbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I canceled the CI since I was removing a raw file in the wrong directory. (It would have been the same successful outcome as the previous CI test run.)

@Mikeg2881
Copy link
Contributor Author

Do i need to do something here.

@mjbear
Copy link
Collaborator

mjbear commented Nov 9, 2025

Do i need to do something here.

This PR is complete from my viewpoint.

There's some time for other maintainers to choose to review before it gets merged.

@jmcgill298 jmcgill298 merged commit b85654b into networktocode:master Nov 11, 2025
11 of 20 checks passed
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.

Suggestion on making (arista_eos_interfaces_transceiver_hardware)

3 participants