Skip to content

[Platform][AS9716-32D] Fix sonic-mgmt pytest test_chassis.py::testChassisApi::test_sfps fail#186

Merged
kevin28-9219 merged 1 commit intoedge-core:202311.Xfrom
tigerfu000:202311.X_pytest_chassis
Sep 9, 2024
Merged

[Platform][AS9716-32D] Fix sonic-mgmt pytest test_chassis.py::testChassisApi::test_sfps fail#186
kevin28-9219 merged 1 commit intoedge-core:202311.Xfrom
tigerfu000:202311.X_pytest_chassis

Conversation

@tigerfu000
Copy link
Contributor

Why I did it

  • platform_tests/api/test_chassis.py::TestChassisApi::test_sfps
    • Failed: Number of sfps (34) does not match expected number (32)
      • Wrong ‘num_ports’ value = 34 in pddf-device.json.

How I did it

  • Change ‘num_ports’ value = 32 in pddf-device.json.

How to verify it

  • Run pytest case is pass.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@kevin28-9219 kevin28-9219 merged commit 9f77012 into edge-core:202311.X Sep 9, 2024
@tigerfu000 tigerfu000 deleted the 202311.X_pytest_chassis branch September 9, 2024 07:34
bryan1978 pushed a commit that referenced this pull request Jan 17, 2025
…lly (#18076)

#### Why I did it
src/sonic-gnmi
```
* d56712a - (HEAD -> master, origin/master, origin/HEAD) Update GNMI path schema (#197) (4 days ago) [ganglv]
* 758ec18 - Call flag.Parse() to parse global flags like -logtostderr (#198) (5 days ago) [Zain Budhwani]
* 736e3b4 - Add signal handler to stop gnmi server for when sigterm or sigquit is called (#189) (3 weeks ago) [Zain Budhwani]
* 5b59c57 - Fix sonic string in osversion/build (#190) (4 weeks ago) [Zain Budhwani]
* d8d15c7 - Enable unit tests and code coverage for telemetry.go (#186) (5 weeks ago) [Zain Budhwani]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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

Successfully merging this pull request may close these issues.

2 participants