Skip to content

Conversation

l0crian1
Copy link
Contributor

Change summary

VLAN-to-VNI interface statistics are not returned with any existing op-mode command. This would provide that useful information. The following commands would be added:

show interfaces vxlan <interface name> statistics
show interfaces vxlan <interface name>vlan <vlan-id> statistics

The output would like like this:

show interfaces vxlan vxlan0 statistics
Interface      VLAN    VNI    Rx Packets    Rx Bytes    Tx Packets    Tx Bytes
-----------  ------  -----  ------------  ----------  ------------  ----------
vxlan0           10   1001             0           0             0           0
vxlan0           20   1002             0           0             0           0
show interfaces vxlan vxlan0 vlan 10 statistics
Interface      VLAN    VNI    Rx Packets    Rx Bytes    Tx Packets    Tx Bytes
-----------  ------  -----  ------------  ----------  ------------  ----------
vxlan0           10   1001             0           0             0           0

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

https://vyos.dev/T7764

Related PR(s)

How to test / Smoketest result

Configure a vxlan interface with vlan-to-vni mappings and attach to bridge:

set interfaces vxlan vxlan1 parameters external
set interfaces vxlan vxlan1 port '4789'
set interfaces vxlan vxlan1 source-address '10.5.5.5'
set interfaces vxlan vxlan1 vlan-to-vni 10 description 'Cust-A'
set interfaces vxlan vxlan1 vlan-to-vni 10 vni '1001'
set interfaces vxlan vxlan1 vlan-to-vni 20 description 'Cust-B'
set interfaces vxlan vxlan1 vlan-to-vni 20 vni '1002'

set interfaces bridge br0 member interface vxlan1

Run commands:

vyos@PE2:~$ show interfaces vxlan vxlan1 vlan-to-vni statistics
Interface      VLAN    VNI    Rx Packets    Rx Bytes    Tx Packets    Tx Bytes
-----------  ------  -----  ------------  ----------  ------------  ----------
vxlan1           10   1001             0           0             0           0
vxlan1           20   1002             0           0             0           0

vyos@PE2:~$ show interfaces vxlan vxlan1 vlan-to-vni vlan 10 statistics
Interface      VLAN    VNI    Rx Packets    Rx Bytes    Tx Packets    Tx Bytes
-----------  ------  -----  ------------  ----------  ------------  ----------
vxlan1           10   1001             0           0             0           0

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

Copy link

github-actions bot commented Aug 29, 2025

👍
No issues in PR Title / Commit Title

@c-po c-po added the bp/circinus Create automatic backport for circinus label Aug 31, 2025
Copy link
Member

@c-po c-po left a comment

Choose a reason for hiding this comment

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

Other then my minor two changes its a nice and welcomed extension.

- Fix typo in 'vlan-to-vni statistics' help text
Copy link

github-actions bot commented Sep 2, 2025

CI integration 👍 passed!

Details

CI logs

  • CLI Smoketests (no interfaces) 👍 passed
  • CLI Smoketests VPP 👍 passed
  • CLI Smoketests (interfaces only) 👍 passed
  • Config tests 👍 passed
  • Config tests VPP 👍 passed
  • RAID1 tests 👍 passed
  • TPM tests 👍 passed

Copy link
Contributor

@jestabro jestabro left a comment

Choose a reason for hiding this comment

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

Works as expected; typo fixed.

@c-po c-po merged commit 1517192 into vyos:current Sep 9, 2025
17 of 18 checks passed
@vyosbot vyosbot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Sep 9, 2025
@l0crian1 l0crian1 deleted the vid-to-vni-stats branch September 11, 2025 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bp/circinus Create automatic backport for circinus current mirror-completed
Development

Successfully merging this pull request may close these issues.

4 participants