Skip to content

[topology]: Fix t1-64-lag topology device link base indices#414

Merged
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:topo
Jan 5, 2018
Merged

[topology]: Fix t1-64-lag topology device link base indices#414
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:topo

Conversation

@stcheng
Copy link
Contributor

@stcheng stcheng commented Jan 5, 2018

This piece of information is used in the LLDP test.
Signed-off-by: Shu0T1an ChenG shuche@microsoft.com

Description of PR

Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)
TASK [test : Compare the LLDP neighbor name with minigraph neigbhor name (exclude the management port)] ***
Friday 05 January 2018  00:22:33 +0000 (0:00:00.166)       0:00:26.667 ********
ok: [str-s6100-acs-5] => (item=Ethernet0)
ok: [str-s6100-acs-5] => (item=Ethernet1)
ok: [str-s6100-acs-5] => (item=Ethernet4)
ok: [str-s6100-acs-5] => (item=Ethernet5)
skipping: [str-s6100-acs-5] => (item=eth0)
ok: [str-s6100-acs-5] => (item=Ethernet58)
ok: [str-s6100-acs-5] => (item=Ethernet50)
ok: [str-s6100-acs-5] => (item=Ethernet52)
ok: [str-s6100-acs-5] => (item=Ethernet53)
ok: [str-s6100-acs-5] => (item=Ethernet54)
ok: [str-s6100-acs-5] => (item=Ethernet55)
ok: [str-s6100-acs-5] => (item=Ethernet38)
ok: [str-s6100-acs-5] => (item=Ethernet39)
ok: [str-s6100-acs-5] => (item=Ethernet16)
ok: [str-s6100-acs-5] => (item=Ethernet17)
ok: [str-s6100-acs-5] => (item=Ethernet36)
ok: [str-s6100-acs-5] => (item=Ethernet37)
ok: [str-s6100-acs-5] => (item=Ethernet34)
ok: [str-s6100-acs-5] => (item=Ethernet47)
ok: [str-s6100-acs-5] => (item=Ethernet46)
ok: [str-s6100-acs-5] => (item=Ethernet45)
ok: [str-s6100-acs-5] => (item=Ethernet44)
ok: [str-s6100-acs-5] => (item=Ethernet42)
ok: [str-s6100-acs-5] => (item=Ethernet61)
ok: [str-s6100-acs-5] => (item=Ethernet60)
ok: [str-s6100-acs-5] => (item=Ethernet63)
ok: [str-s6100-acs-5] => (item=Ethernet62)
ok: [str-s6100-acs-5] => (item=Ethernet21)
ok: [str-s6100-acs-5] => (item=Ethernet20)

This piece of information is used in the LLDP test.
Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
<EndDevice>{{ inventory_hostname }}</EndDevice>
<EndPort>{{ port_alias[32+index*8+2] }}</EndPort>
<StartDevice>ARISTA{{ '%02d' % (index+1) }}T0</StartDevice>
<StartDevice>ARISTA{{ '%02d' % (index*5+1) }}T0</StartDevice>
Copy link
Contributor

Choose a reason for hiding this comment

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

why 5 magic number?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

in total there are 20 T0 devices connected. we have four groups of links and each of the group has five up links.

@stcheng stcheng merged commit 10f362a into sonic-net:master Jan 5, 2018
@stcheng stcheng deleted the topo branch January 5, 2018 01:40
praveen-li pushed a commit to praveen-li/sonic-mgmt that referenced this pull request Jun 20, 2019
* msft_github/master:
  [PFCWD]: Add Add support for t0 toplogy and arista fanout testbed (sonic-net#424)
  add orchagent process check before each test and sanity after each test (sonic-net#431)
  Move mem_check.sh into helpers/ directory to conform with location of other helper scripts (sonic-net#435)
  fix tests for t0-116 topology (sonic-net#430)
  Updated labinfo file with Arista fanout sku (sonic-net#429)
  add missing snmp testcases to top level snmp.yml; fixed wrong test file for neighbour test; change arp test comment more readable (sonic-net#428)
  Fix sku-sensors-data for 7060 (sonic-net#427)
  Fix port alias mapping for Arista-7050QX-32S (sonic-net#426)
  [Lag 2] Allow lacp timing tests to retry limited times until succeeded (sonic-net#403)
  [hwsku]: Add Accton-AS7716-32X (sonic-net#405)
  add generate minigrah using testbed_name option (sonic-net#425)
  Update README.testbed.md
  Create README.testbed.Example.md
  Update README.testbed.Config.md
  add missing test case name (sonic-net#417)
  fix typo when assign dscp_mode for decap test (sonic-net#418)
  [repeat harness] rework repeat harness and introduce test case continuous reboot (sonic-net#416)
  allow upgrade sonic via onie or sonic-to-sonic upgrade method (sonic-net#413)
  [topology]: Fix t1-64-lag topology device link base indices (sonic-net#414)
  Run sudo command with log_analyzer (sonic-net#415)
  [sonic tests] use host time instead of ansible time (sonic-net#410)
  [upgrade_sonic]: Fix the hostname in the wait_for condition (sonic-net#411)
  [Test cases] clean up some test cases changes and enabling more tests (sonic-net#409)
  add restart_swss (sonic-net#412)
  [sonic test] introduce a repeat harness (sonic-net#402)
  Update README.test.md
  [bgp_speaker]: always clean up test environment after it finishes (sonic-net#406)
  [vlan]: add vlan test to test by testname (sonic-net#408)
  [bug]: convert the interface from unicode to string (sonic-net#407)
  [VLAN test] Add VLAN test (sonic-net#375)
  [vm]: change the default Front plane port to 4 instead of 8 (sonic-net#404)
  Update README.testbed.Setup.md
  [acl]: Adopt tests with acl-loader to t0 and t1-lag topo. (sonic-net#370)
  add connect_topo command for testbed-cli.sh (sonic-net#398)
  [FIB, BGP speaker] fix t0-116 topology source port list (sonic-net#396)
  [bgp_speaker]: Specify VLAN IP route in case LPM to other nexthop (sonic-net#394)
  [sensor]: Change sensor labels for Arista 7050 and 7260 (sonic-net#395)
  Delete Arista DPMs sensors data (sonic-net#393)
  [bug]: change maximum packet to 9114 to send in the mtu test (sonic-net#391)
sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Jun 30, 2025
…s crash (sonic-net#414)

<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->

Summary:
Workaround for sonic-net#19034
As changing port speed on 7060x6 will cause orchagent crash, resulting in subsequence test case failure. Skip it for now until the issue is fixed.

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

- [ ] 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

### Approach
#### What is the motivation for this PR?
Skip test case which cause orchagent crash on 7060x6 platform

#### How did you do it?
change from xfail to skip

#### How did you verify/test it?
=========================== short test summary info ============================
SKIPPED [2] iface_namingmode/test_iface_namingmode.py:678: Unsupported topology
SKIPPED [2] iface_namingmode/test_iface_namingmode.py:699: Unsupported topology
SKIPPED [2] iface_namingmode/test_iface_namingmode.py: Incorrect supported speeds for 7060X6 in STATE_DB, fix TBD
============ 54 passed, 6 skipped, 4 warnings in 1778.39s (0:29:38) ============

#### Any platform specific information?
Only on 7060x6-64pe platform

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

### Documentation
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
sonic-swss

* 32d3297 2019-01-24 | use sai_stat_id_t for new SAI header file (sonic-net#769)

sonic-sairedis

* fdfcefd 2019-04-24 | [SAI] Advance submodule to v1.4.1 (sonic-net#450) (HEAD -> 201811, origin/201811) [Marian Pritsak]
* 006df41 2019-02-27 | [vslib]: Add support of bitmap VNET (bmtor) SAI API (sonic-net#424) [Volodymyr Samotiy]
* ec5e187 2019-04-23 | [SAI]: Move SAI pointer to v1.4 (sonic-net#447) [Shuotian Cheng]
* 6b9c32e 2019-03-04 | [SAI] Update SAI pointer (sonic-net#431) [Stepan Blyshchak]
* f078998 2019-02-22 | Fix compilation issues in stretch docker with gcc-6.3 (sonic-net#426) [Stepan Blyshchak]
* 0540e59 2019-01-31 | Update SAI pointer (sonic-net#414) [Marian Pritsak]
* 9084920 2019-01-18 | Update SAI pointer to latest master (sonic-net#408) [Marian Pritsak]
* fbfe48b 2019-06-24 | Revert "[SAI header] upgrade SAI header to version v1.3.7 (sonic-net#445)" [Guohan Lu]

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants