Skip to content
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.

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

Updating template Huawei VRP: display vlan #2016

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

evilmonkey19
Copy link
Contributor

Adding support for AP. I accept suggestions for VLAN_ID and VLAN_TYPE

@evilmonkey19 evilmonkey19 marked this pull request as draft February 8, 2025 12:16
@evilmonkey19
Copy link
Contributor Author

This templates are for Huawei VRP platform, which is used in Huawei Switches and APs. The command display vlan is used in both platforms, but they have similar but not exactly the same values. Also I decided to use a more common approach on display vlan imitating show etherchannel brief from cisco IOS.

As well, I added the command display vlan brief which is used in Huawei VRP AP. I reused a template from display vlan.

@evilmonkey19 evilmonkey19 marked this pull request as ready for review February 8, 2025 15:35
@mjbear
Copy link
Contributor

mjbear commented Feb 8, 2025

This templates are for Huawei VRP platform, which is used in Huawei Switches and APs. The command display vlan is used in both platforms, but they have similar but not exactly the same values. Also I decided to use a more common approach on display vlan imitating show etherchannel brief from cisco IOS.

I'm curious, so I looked but didn't find a template for etherchannel brief, but there is one for summary.

Did I miss something?

Thank you.

As well, I added the command display vlan brief which is used in Huawei VRP AP. I reused a template from display vlan.

@jvanderaa
Copy link
Contributor

I'm working on getting some additional eyes on this one. This does look like a breaking change at the moment. Which then we need to do a major release here soon.

@itdependsnetworks
Copy link
Contributor

What information does bried have that hte regular does not? In general, I regret supporting things like "show cdp neighbors" and "show cdp neighbors details". It's extra work and you get everything from details, no exceptions. The compute cost of a few extra lines is so small, it's not worth the development effort and upkeep.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants