Skip to content
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

[sonic-cfggen] sonic-cfggen should consider DPU ports when generating L2 mode configuration #20937

Open
echuawu opened this issue Nov 27, 2024 · 0 comments

Comments

@echuawu
Copy link

echuawu commented Nov 27, 2024

Description

When running L2 mode test on bobcat switch, the sonic-cfggen should consider the DPU ports.
Currently, when using command "sonic-cfggen -H -k Mellanox-SN4280-O28 -p --preset l2 > /tmp/init_l2_config.json", it would generate DPU ports with admin status UP, but the operation status would be DOWN.
The DPU ports configuration should be admin status DOWN, in order to make the L2 tests pass.

Steps to reproduce the issue:

  1. On bobcat switch, run command "sonic-cfggen -H -k Mellanox-SN4280-O28 -p --preset l2 > /tmp/init_l2_config.json" to generate L2 mode port configuration.
  2. Besides normal ports, the DPU ports would be admin status UP, and operation status DOWN.

Describe the results you received:

Ethernet224 224,225,226,227,228,229,230,231 400G 9100 N/A etp29 trunk down up DPU-NPU Data Port N/A
Ethernet232 232,233,234,235,236,237,238,239 400G 9100 N/A etp30 trunk down up DPU-NPU Data Port N/A
Ethernet240 240,241,242,243,244,245,246,247 400G 9100 N/A etp31 trunk down up DPU-NPU Data Port N/A
Ethernet248 248,249,250,251,252,253,254,255 400G 9100 N/A etp32 trunk down up DPU-NPU Data Port N/A

Describe the results you expected:

All the DPU ports configuration should be admin status DOWN.
Command "sonic-cfggen -H -k Mellanox-SN4280-O28 -p --preset l2" should handle the bobcat switch automatically.

Output of show version:

202405 image

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant