Skip to content
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

Xfail this test for Cisco 8000 limitaion with ECMP hash with LAG hash #16578

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

rajendrat
Copy link
Contributor

Description of PR

Xfail this test for Cisco 8000 limitaion with ECMP hash with LAG hash.

Summary:
Fixes # (issue)
Xfail this test for Cisco 8000 limitaion with ECMP hash with LAG hash.

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

Xfail unsupported test

How did you do it?

Added conditional check

How did you verify/test it?

Running sonic-mgmt test

Any platform specific information?

Cisco-8000 specific

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rajendrat
Copy link
Contributor Author

rajendrat commented Jan 17, 2025

vix @wsycqyz @kevinskwang @bpar9

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@bpar9 bpar9 left a comment

Choose a reason for hiding this comment

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

lgtm

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.

3 participants