Skip to content

Interface assert: fixed vlan mismatch assert for mlag peerlinks #989

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

AndryNick98
Copy link
Collaborator

Related Issue

Fixes #987

Description

Due to an issue reported by an user, we noticed that when a user run an interface assert on a dataset containing mlag peerlinks, the assert may or may not pass randomly.

The problem is that when we evaluate the vlan mismatch for mlag peerlinks, we were looking just for indexPeer less than 0 or greater than 0, without never checking the entry at index 0. This was the cause of that random behavior.

This PR fix it.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Proposed Release Note Entry

Fixed interface assert vlan mismatch for mlag interfaces

Double Check

  • I have read the comments and followed the CONTRIBUTING.md.
  • I have explained my PR according to the information in the comments or in a linked issue.
  • My PR source branch is created from the develop branch.
  • My PR targets the develop branch.
  • All my commits have --signoff applied

Signed-off-by: AndryNick98 <andrea.nicoletti@stardustsystems.net>
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.

[Bug]: VLAN set mismatch error appears when running interface assertions
1 participant