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

Added support to get random asic in a testcase using 'enum_rand_one_asic_index' #3017

Merged
merged 1 commit into from Feb 26, 2021

Conversation

oxygen980
Copy link
Contributor

@oxygen980 oxygen980 commented Feb 22, 2021

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

  1. If a script has 'enum_rand_one_per_hwsku_frontend_hostname' and 'enum_asic_index' , enum_asic_index always returns asics from first dut. but It should be returning asics from the dut chosen by 'enum_rand_one_per_hwsku_frontend_hostname'.

  2. If a script needs to run test cases only on one asic, we need a way to get random asic index.

How did you do it?

Added enum_rand_one_asic_index to get random asic index.

How did you verify/test it?

Ran on muti-asic dut.

Any platform specific information?

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

Documentation

@oxygen980 oxygen980 requested a review from a team as a code owner February 22, 2021 19:46
@oxygen980 oxygen980 changed the title Added support to enum_rand_one_asic_index Added support to get random asic in a testcase using 'enum_rand_one_asic_index' Feb 22, 2021
@yxieca yxieca merged commit c0bd153 into sonic-net:master Feb 26, 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.

None yet

2 participants