Skip to content

Skip BVI/vxlan_tunnel interfaces in asyncIntfStateUpdate#1805

Open
rejithomas-arista wants to merge 1 commit intosonic-net:masterfrom
rejithomas-arista:master-fix-skip-bvi-vxlan
Open

Skip BVI/vxlan_tunnel interfaces in asyncIntfStateUpdate#1805
rejithomas-arista wants to merge 1 commit intosonic-net:masterfrom
rejithomas-arista:master-fix-skip-bvi-vxlan

Conversation

@rejithomas-arista
Copy link
Copy Markdown

VPP-internal interfaces (BVI, vxlan_tunnel) have no Linux TAP pair. Skip port state notification for these to prevent ERR logs that cause loganalyzer failures in VxLAN ECMP tests.
Also add prefix to error messages and guard against empty interface name.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Mar 23, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: rejithomas-arista / name: Reji Thomas (f81eb56)

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rejithomas-arista rejithomas-arista force-pushed the master-fix-skip-bvi-vxlan branch from 409f8e1 to 2ece673 Compare April 13, 2026 05:53
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

VPP-internal interfaces (BVI, vxlan_tunnel) have no Linux TAP pair.
Skip port state notification for these to prevent ERR logs that
cause loganalyzer failures in VxLAN ECMP tests.
Also add prefix to error messages and guard against empty interface name.

Signed-off-by: Reji Thomas <reji.thomas@arista.com>
@rejithomas-arista rejithomas-arista force-pushed the master-fix-skip-bvi-vxlan branch from 2ece673 to f81eb56 Compare April 13, 2026 06:30
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rejithomas-arista
Copy link
Copy Markdown
Author

@yue-fred-gao could you review this?

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.

2 participants