-
Notifications
You must be signed in to change notification settings - Fork 711
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
[GCU] Undefined variable in test_ecn_config_updates
#8648
Comments
6 tasks
@isabelmsft Please check if the test case can run on Mellanox SPC3 platform, such as Mellanox-SN4600. Thanks |
6 tasks
Test case bug fixed in PR https://github.com/sonic-net/sonic-mgmt/pull/8652/files, requested review from scenario owner @neethajohn |
Fixed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If
is_valid_platform_and_version
returnsFalse
, we will see below exceptionIt's because the
output
variable is not defined.Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important:
The text was updated successfully, but these errors were encountered: