Skip to content

Add juniper junos show ethernet-switching interfaces #2263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mjbear merged 7 commits into from
Feb 4, 2026
Merged

Add juniper junos show ethernet-switching interfaces #2263

mjbear merged 7 commits into from
Feb 4, 2026
+155 −0

Conversation

@clement-ldr
Copy link
Contributor

@clement-ldr clement-ldr commented Dec 20, 2025

New template to add support for juniper junos show ethernet-switching interfaces 🤠

@mjbear
Fix index order for junos ethernet-sw interfaces
01a754d
@mjbear
Simplify junos show eth-sw interfaces template
da6b9ad
@mjbear
Shorten TAGGING pattern via regex
dcc5e81
@mjbear
Use VLAN_ID instead of TAG
1d93800 
Use the normalized VLAN_ID capture group name for the VLAN tag.
@mjbear
Update ethernet-switching pattern in index
e67d64b 
Make the ethernet-switching regex pattern match another existing
command (where there are optional characters for the shortened version.)
@mjbear
Copy link
Collaborator

mjbear commented Dec 23, 2025 •
edited
Loading

@clement-ldr
I made a couple of tweaks.

  • Fix the index order
  • Simplify the template to reduce some duplication (since the raw cli output is simple enough)
  • Shorten the TAGGING pattern via regex
  • Use the normalized VLAN_ID instead of TAG from data model
  • Adjust the ethernet-switching word pattern to match another instance (the "table")

Note to self:
Command output details via a Juniper ethernet-switching CLI reference doc.

@mjbear mjbear removed the question label Dec 31, 2025
@mjbear mjbear merged commit 28d75cd into networktocode:master Feb 4, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mjbear mjbear mjbear approved these changes

@itdependsnetworks itdependsnetworks Awaiting requested review from itdependsnetworks itdependsnetworks is a code owner

@jmcgill298 jmcgill298 Awaiting requested review from jmcgill298 jmcgill298 is a code owner

@jvanderaa jvanderaa Awaiting requested review from jvanderaa jvanderaa is a code owner

@matt852 matt852 Awaiting requested review from matt852 matt852 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@clement-ldr @mjbear