diff options
| author | Christian Poessinger <christian@poessinger.com> | 2020-09-20 13:40:51 +0200 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2020-09-20 14:29:32 +0200 |
| commit | 0a21905786d31e759416dd335d87726f9ed46ffa (patch) | |
| tree | 3de5e79cdbcd6e547d3a1715a6ed624ae000b8c5 /interface-definitions | |
| parent | e64ab9ec34ce8cb221f3c82787d3641efa30aac0 (diff) | |
| download | vyos-1x-0a21905786d31e759416dd335d87726f9ed46ffa.tar.gz vyos-1x-0a21905786d31e759416dd335d87726f9ed46ffa.zip | |
vif-s: ifconfig: T2903: use explicit VLAN protocol over raw numbers
In the past we had to provide the ethertype value used for the VLAN
protocol (0x88A8 -> 802.1ad or 0x8100 -> 802.1q).
This should be changed to a more user friendly CLI node (protocol over
ethertype) and 802.1ad over it's raw value 0x88A8. There is no need in
presenting RAW information from the ethernet header to the user. Also iproute2
calls it protocol which makes way more sense over the "raw" value.
Diffstat (limited to 'interface-definitions')
| -rw-r--r-- | interface-definitions/include/vif-s.xml.i | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/interface-definitions/include/vif-s.xml.i b/interface-definitions/include/vif-s.xml.i index a6d7c81ce..cd0afe742 100644 --- a/interface-definitions/include/vif-s.xml.i +++ b/interface-definitions/include/vif-s.xml.i @@ -13,25 +13,26 @@ #include <include/dhcpv6-options.xml.i> #include <include/interface-disable-link-detect.xml.i> #include <include/interface-disable.xml.i> - <leafNode name="ethertype"> + <leafNode name="protocol"> <properties> - <help>Set Ethertype</help> + <help>Protocol used for service VLAN (default: 802.1ad)</help> <completionHelp> - <list>0x88A8 0x8100</list> + <list>802.1ad 802.1q</list> </completionHelp> <valueHelp> - <format>0x88A8</format> - <description>802.1ad</description> + <format>802.1ad</format> + <description>Provider Bridging (IEEE 802.1ad, Q-inQ), ethertype 0x88a8</description> </valueHelp> <valueHelp> - <format>0x8100</format> - <description>802.1q</description> + <format>802.1q</format> + <description>VLAN-tagged frame (IEEE 802.1q), ethertype 0x8100</description> </valueHelp> <constraint> - <regex>(0x88A8|0x8100)</regex> + <regex>(802.1q|802.1ad)</regex> </constraint> - <constraintErrorMessage>Ethertype must be 0x88A8 or 0x8100</constraintErrorMessage> + <constraintErrorMessage>Ethertype must be 802.1ad or 802.1q</constraintErrorMessage> </properties> + <defaultValue>802.1ad</defaultValue> </leafNode> <node name="ip"> <children> |