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

Subinterface doesn't inherit the speed of ancestors on kvm testbed. #19735

Open
yutongzhang-microsoft opened this issue Jul 30, 2024 · 1 comment · May be fixed by sonic-net/sonic-sairedis#1458
Assignees
Labels
MSFT Triaged this issue has been triaged

Comments

@yutongzhang-microsoft
Copy link
Contributor

Description

In test script sub_port_interfaces/test_show_subinterface.py, we want to create the subinterface and it should inherit the ancestor's speed. But from the rest result , we can see that, the ancestor's speed is 1G, which should be inherited by the subinterface. But actually, the speed of the new created subinterface is 40G. We suspect it is an image issue on kvm testbed.

Steps to reproduce the issue:

  1. Run the script sub_port_interfaces/test_show_subinterface.py on kvm testbed.

Describe the results you received:

>           pytest_assert(status.get("speed") == config["speed"],
                          "subinterface %s should have inherited speed as %s, actual speed %s"
                          % (subintf, config["speed"], status.get("speed")))
E           Failed: subinterface Ethernet4.20 should have inherited speed as 1G, actual speed 40G

Describe the results you expected:

Output of show version:

(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):

@ishidawataru
Copy link
Collaborator

I noticed the same problem.

The problem is that the vslib gets the oper speed from the veth interface, which doesn't reflect the configured speed.

I'm using the following SAI_VS_OPER_SPEED_IS_CONFIGURED_SPEED option to make the oper speed equal to the configured speed.

ishidawataru/sonic-sairedis@934fe25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MSFT Triaged this issue has been triaged
Projects
None yet
4 participants