Skip to content

Commit

Permalink
Revert "[sonic-cfggen] make minigraph parser fail when speed and lane…
Browse files Browse the repository at this point in the history
…s are not in PORT table (sonic-net#10228)"

This reverts commit cd330f0.
  • Loading branch information
wen587 committed Apr 26, 2022
1 parent cc30771 commit ff2647a
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions src/sonic-config-engine/sonic-cfggen
Original file line number Diff line number Diff line change
Expand Up @@ -247,19 +247,6 @@ def _get_jinja2_env(paths):

return env

def _must_field_by_yang(data, table, must_fields):
"""
Check if table contains must field based on yang definition
"""
if table not in data:
return

for must_field in must_fields:
for _, fields in data[table].items():
if must_field not in fields:
print(must_field, 'is a must field in', table, file=sys.stderr)
sys.exit(1)

def main():
parser=argparse.ArgumentParser(description="Render configuration file from minigraph data and jinja2 template.")
group = parser.add_mutually_exclusive_group()
Expand Down Expand Up @@ -354,8 +341,6 @@ def main():
deep_update(data, parse_xml(minigraph, platform, asic_name=asic_name))
else:
deep_update(data, parse_xml(minigraph, port_config_file=args.port_config, asic_name=asic_name, hwsku_config_file=args.hwsku_config))
# check if minigraph parser has speed and lanes in PORT table
_must_field_by_yang(data, 'PORT', ['speed', 'lanes'])

if args.device_description is not None:
deep_update(data, parse_device_desc_xml(args.device_description))
Expand Down

0 comments on commit ff2647a

Please sign in to comment.