Skip to content

[Spytest] Adapt to 'show interfaces status' change#2035

Merged
lolyu merged 1 commit intosonic-net:masterfrom
lolyu:show_int_status
Aug 6, 2020
Merged

[Spytest] Adapt to 'show interfaces status' change#2035
lolyu merged 1 commit intosonic-net:masterfrom
lolyu:show_int_status

Conversation

@lolyu
Copy link
Copy Markdown
Collaborator

@lolyu lolyu commented Aug 5, 2020

Description of PR

A new column FEC is introduced to show interfaces status output,
adjust the template to adapt to the change.

Signed-off-by: Longxiang Lyu lolv@microsoft.com

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

To adapt to the new column FEC in the output from show int status:

  Interface            Lanes    Speed    MTU    FEC         Alias    Vlan    Oper    Admin             Type    Asym PFC
-----------  ---------------  -------  -----  -----  ------------  ------  ------  -------  ---------------  ----------
  Ethernet0      33,34,35,36     100G   9100     rs   Ethernet1/1  routed      up       up  QSFP28 or later         off
  Ethernet4      37,38,39,40     100G   9100     rs   Ethernet2/1  routed    down     down  QSFP28 or later         off
  Ethernet8      41,42,43,44     100G   9100     rs   Ethernet3/1  routed    down     down  QSFP28 or later         off
 Ethernet12      45,46,47,48     100G   9100     rs   Ethernet4/1  routed    down     down  QSFP28 or later         off
 Ethernet16      49,50,51,52     100G   9100     rs   Ethernet5/1  routed    down     down  QSFP28 or later         off
 Ethernet20      53,54,55,56     100G   9100     rs   Ethernet6/1  routed    down     down  QSFP28 or later         off
...

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

A new column `FEC` is introduced to `show interfaces status` output,
adjust the template to adapt to the change.

Signed-off-by: Longxiang Lyu <lolv@microsoft.com>
@lolyu lolyu requested review from a team and yxieca August 5, 2020 06:29
@lolyu lolyu merged commit 4f02bb5 into sonic-net:master Aug 6, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
sonic-swss

1aa40f7 Remove port serdes object before removing port (sonic-net#2152)
876d690 [doc] Updating Policer config in Configuration manual (sonic-net#2144)

sonic-utilities
dfed952 show_platfom_info not run for simx (sonic-net#2042)
71fdee7 [aclshow] fix aclshow when clear is called before counters are populated (sonic-net#2037)
a48a027 [sonic-package-manager] implement blocking feature state change (sonic-net#2035)
c51871d [ci] Fix python dependencies reference path. (sonic-net#2060)
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.

2 participants