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 iface_namingmode testcases to get spine ports for T2 topology #3758
Conversation
@@ -869,7 +869,7 @@ def spine_ports(self, setup): | |||
spine_ports['alias'] = list() | |||
|
|||
for key, value in minigraph_neighbors.items(): | |||
if (key in setup['physical_interfaces']) and ('T2' in value['name']): | |||
if key in setup['physical_interfaces'] and ('T2' in value['name'] or (tbinfo['topo']['type'] == 't2' and 'T3' in value['name'])): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you split the statement, prefer 80 chars, but 120 is fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oxygen980 is this addressed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rlhui yes this was done a long back.
dd15b3c
to
d174114
Compare
…sonic-net#3758 - oxygen980:spine_ports)
Description of PR
Modified iface_namingmode testcases to get spine ports for T2 topology
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
Spine ports function was not able to get spine ports for T2 topology, added condition for the same.
How did you do it?
Added condition to get spine [ports for T2 topolgy.
How did you verify/test it?
Tested on VOQ system.
Any platform specific information?
Supported testbed topology if it's a new test case?
T2
Documentation