Skip to content

Adding Celestica E1031 SKU and related changes to testcases (Phase 1)#1105

Merged
sumukhatv merged 8 commits intosonic-net:masterfrom
sumukhatv:m0-nightly
Sep 10, 2019
Merged

Adding Celestica E1031 SKU and related changes to testcases (Phase 1)#1105
sumukhatv merged 8 commits intosonic-net:masterfrom
sumukhatv:m0-nightly

Conversation

@sumukhatv
Copy link
Contributor

Description of PR

Summary:
Adding Celestica E1031 SKU
Changes related to t0-52 topology

Type of change

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

Approach

How did you do it?

Made changes to get the tests running on Celestica E1031 SKU

How did you verify/test it?

Any platform specific information?

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

t0-52

Documentation

Copy link
Contributor

@prsunny prsunny left a comment

Choose a reason for hiding this comment

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

Changes looks good to me.

@sumukhatv sumukhatv merged commit 8b2905c into sonic-net:master Sep 10, 2019
yxieca pushed a commit that referenced this pull request Sep 11, 2019
Adding Celestica E1031 SKU and related changes to testcases (Phase 1)
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
 update the environment variable in the teardown (sonic-net#1101)
 Fix for show interface portchannel now working on 201911 (sonic-net#1105)
 [201911]show interface counters for multi ASIC devices (sonic-net#1104)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
update the environment variable in the teardown (sonic-net#1101)
Fix for show interface portchannel now working on 201911 (sonic-net#1105)
Revert "Pfcstat (sonic-net#1097)"
Revert " [201911]show interface counters for multi ASIC devices
      (sonic-net#1104)"

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants