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

Modified FIB testcases to support T2 topology on multi-dut #2891

Closed
wants to merge 1 commit into from

Conversation

oxygen980
Copy link
Contributor

Description of PR

  • Modified FIB testcases to support T2 topology on Multi-dut

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • [x ] Test case(new/improvement)

Approach

What is the motivation for this PR?

Needed FIB testcases to support T2 topology for multi-dut

How did you do it?

To run traffic across multiple line cards in case of VOQ, needed access to T3 and T2 line cards so removed 'rand_one_dut_hostname' .
sending traffic from T1 to T3 line cards.

How did you verify/test it?

Ran testcases on VOQ testbed with T2 topology

Any platform specific information?

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

T2

Documentation

@wangxin
Copy link
Collaborator

wangxin commented Feb 7, 2021

We have updated the FIB test to support dualtor topology in PR #2740. Unfortunately that PR introduced merge conflicts. Could you please update the FIB test based on changes of PR #2740? Sorry for the trouble.

@oxygen980 oxygen980 marked this pull request as draft February 11, 2021 17:37
@oxygen980
Copy link
Contributor Author

Will create a new PR with the changes made for multi-dut in FIB.

@oxygen980 oxygen980 closed this Mar 12, 2021
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