Skip to content

Commit a145f2c

Browse files
authored
Set FEC to reed-solomon if the port speed is 100G (#1383)
* Set FEC to reed-solomon if the port speed is 100G
1 parent ecf5c8d commit a145f2c

File tree

4 files changed

+7
-2
lines changed

4 files changed

+7
-2
lines changed

src/sonic-config-engine/minigraph.py

+2
Original file line numberDiff line numberDiff line change
@@ -418,6 +418,8 @@ def parse_xml(filename, platform=None, port_config_file=None):
418418

419419
for port_name in port_speeds:
420420
ports.setdefault(port_name, {})['speed'] = port_speeds[port_name]
421+
if port_speeds[port_name] == '100000':
422+
ports.setdefault(port_name, {})['fec'] = 'rs'
421423
for port_name in port_descriptions:
422424
ports.setdefault(port_name, {})['description'] = port_descriptions[port_name]
423425

src/sonic-config-engine/tests/sample_output/ports.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
},
2323
{
2424
"PORT_TABLE:Ethernet12": {
25-
"speed": "1000000",
25+
"speed": "100000",
2626
"description": "Interface description"
2727
},
2828
"OP": "SET"

src/sonic-config-engine/tests/simple-sample-graph.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@
260260
<MultiPortsInterface>false</MultiPortsInterface>
261261
<PortName>0</PortName>
262262
<Priority>0</Priority>
263-
<Speed>1000000</Speed>
263+
<Speed>100000</Speed>
264264
<Description>Interface description</Description>
265265
</a:EthernetInterface>
266266
</EthernetInterfaces>

src/sonic-config-engine/tests/test_cfggen.py

+3
Original file line numberDiff line numberDiff line change
@@ -140,3 +140,6 @@ def test_minigraph_ethernet_interfaces(self):
140140
argument = '-m "' + self.sample_graph_simple + '" -p "' + self.port_config + '" -v "PORT[\'Ethernet8\']"'
141141
output = self.run_script(argument)
142142
self.assertEqual(output.strip(), "{'alias': 'fortyGigE0/8', 'lanes': '37,38,39,40', 'description': 'Interface description', 'speed': '40000'}")
143+
argument = '-m "' + self.sample_graph_simple + '" -p "' + self.port_config + '" -v "PORT[\'Ethernet12\']"'
144+
output = self.run_script(argument)
145+
self.assertEqual(output.strip(), "{'alias': 'fortyGigE0/12', 'lanes': '33,34,35,36', 'fec': 'rs', 'speed': '100000', 'description': 'Interface description'}")

0 commit comments

Comments
 (0)