Skip to content

vpp: cleanup lldp related skip markers and a few others#23322

Open
yue-fred-gao wants to merge 2 commits intosonic-net:masterfrom
yue-fred-gao:cleanup_lldp
Open

vpp: cleanup lldp related skip markers and a few others#23322
yue-fred-gao wants to merge 2 commits intosonic-net:masterfrom
yue-fred-gao:cleanup_lldp

Conversation

@yue-fred-gao
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)
cleanup lldp related skip markers and a few others because the tests are supported

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

There are some outdated markers skipping tests that failed before. The issues have been fixed and those tests are passing. This PR removes the markers.

How did you do it?

Remove outdated skip markers

How did you verify/test it?

running sonic-mgmt tests

Any platform specific information?

vpp

Supported testbed topology if it's a new test case?

Documentation

Signed-off-by: Yue Gao <yuega2@cisco.com>
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

yxieca
yxieca previously approved these changes Mar 26, 2026
Copy link
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

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

AI agent on behalf of Ying. Reviewed; no issues found.

Blueve
Blueve previously approved these changes Mar 26, 2026
@yue-fred-gao yue-fred-gao dismissed stale reviews from Blueve and yxieca via dbb7700 March 26, 2026 13:39
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

4 participants