Install pygnmi==0.8.15 for docker-sonic-mgmt by liang-nexthop · Pull Request #28339 · sonic-net/sonic-buildimage

liang-nexthop · 2026-07-09T19:14:22Z

Why I did it

The new native pygnmi-based gNMI client for sonic-mgmt (see sonic-net/sonic-mgmt#26015 and sonic-net/sonic-mgmt#26013) talks gRPC directly to the DUT's gNMI server from the sonic-mgmt orchestrator, replacing the gnmic CLI binary that is being removed from docker-ptf. That client depends on the pygnmi Python package, which is not currently installed in the docker-sonic-mgmt image.

This PR adds pygnmi==0.8.15 to the docker-sonic-mgmt image so the new client and the gNMI tests that use it (tests/common/pygnmi_client.py, tests/gnmi/test_pygnmi_client.py) can run.

Related to sonic-net/sonic-mgmt#26015
Companion PR: sonic-net/sonic-mgmt#26013

Work item tracking

Microsoft ADO (number only):

How I did it

Added "pygnmi==0.8.15" to the uv pip install package list in dockers/docker-sonic-mgmt/Dockerfile.j2. The version is pinned for reproducible builds and to match the version validated by the sonic-mgmt changes.

How to verify it

Build the docker-sonic-mgmt image:
make target/docker-sonic-mgmt.gz
Confirm the package is present and pinned inside the image:
docker run --rm docker-sonic-mgmt python3 -c "import pygnmi; print(pygnmi.version)" expects 0.8.15
Run the pygnmi client tests from sonic-mgmt (Add native pygnmi MgmtGnmic client and migrate gNMI tests sonic-mgmt#26013) inside the built container and confirm they no longer fail on import pygnmi.

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

Tested branch (Please provide the tested image version)

202505: PASS Manual cherry-pick PR: [202505] Install pygnmi==0.8.15 for docker-sonic-mgmt #28690
202511: PASS Manual cherry-pick PR: [202511] Install pygnmi==0.8.15 for docker-sonic-mgmt #28691
202605: PASS, included here [202605] Backport docker-sonic-mgmt and pipeline updates from master #28536

Description for the changelog

Install pygnmi==0.8.15 in docker-sonic-mgmt for the native gNMI test client

Link to config_db schema for YANG module changes

N/A - no YANG/config_db schema changes.

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

mssonicbld · 2026-07-09T19:14:31Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2026-07-09T19:14:46Z

Azure Pipelines successfully started running 1 pipeline(s).

ronan-nexthop · 2026-07-09T20:04:03Z

@hdwhdw @bradh352

hdwhdw · 2026-07-10T03:54:30Z

Can you push an empty commit to restart build?

mssonicbld · 2026-07-10T11:12:57Z

This PR has backport request for branch(es): 202505,202511,202605,202608.
Added label(s) for branch(es) 202505,202511,202605,msft-202608.

_{---Powered by SONiC BuildBot}

Signed-off-by: liang-nexthop <liang@nexthop.ai>

mssonicbld · 2026-07-10T19:04:32Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2026-07-10T19:04:57Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2026-07-11T11:12:43Z

The backport request for branch(es) msft-202608 has been removed from PR description. Removed label(s)....

_{---Powered by SONiC BuildBot}

Copilot

Pull request overview

This PR updates the docker-sonic-mgmt container build to include the pygnmi Python dependency required by the new native pygnmi-based gNMI client and related sonic-mgmt tests.

Changes:

Add pygnmi==0.8.15 to the uv pip install package list in the docker-sonic-mgmt image build.

mssonicbld · 2026-07-14T19:14:31Z

The change is not in 202505 yet. @liang-nexthop, please manually create the cherry pick PR for branch 202505.
You can ping the release branch owner(github account: yejianquan) to approve your cherry pick PR.
If this change is already in 202505, please comment "already in 202505". Thanks!

_{---Powered by SONiC BuildBot}

mssonicbld · 2026-07-14T19:14:32Z

The change is not in 202511 yet. @liang-nexthop, please manually create the cherry pick PR for branch 202511.
You can ping the release branch owner(github account: vmittal-msft) to approve your cherry pick PR.
If this change is already in 202511, please comment "already in 202511". Thanks!

_{---Powered by SONiC BuildBot}

What: Adds pygnmi==0.8.15 to dockers/docker-sonic-mgmt/Dockerfile.j2. Why: The new native pygnmi-based gNMI client for sonic-mgmt talks gRPC directly to the DUT's gNMI server, replacing the gnmic CLI binary being removed from docker-ptf; that client depends on pygnmi, which isn't currently installed in the docker-sonic-mgmt image. How: Installs the pygnmi Python package in the docker-sonic-mgmt Dockerfile. Testing: CI green (required checks passing); PR approved. Signed-off-by: liang-nexthop <liang@nexthop.ai>

rookie-who · 2026-07-24T17:01:53Z

Hi — this PR is flagged for cherry-pick to the 202605 branch (Request for 202605 Branch). To move forward with the cherry-pick, could you please:

Test this change on the 202605 branch and add the test results to the PR description
Add the label Tested for 202605 branch once testing is complete

This helps us track readiness and ensures we don't cherry-pick untested changes. Thanks!

Comment by @vaibhavhd via automated tooling

mssonicbld · 2026-07-24T18:13:58Z

This PR has backport request label(s) for branch(es): 202505,202511,202605, but is missing required test information. Please make sure you tick the tested branch(es) in the Tested branch section and provide test evidence (e.g., 202505: <test result>) in the Test result section as well in your PR description.

_{---Powered by SONiC BuildBot}

liang-nexthop · 2026-07-27T22:49:13Z

Hi — this PR is flagged for cherry-pick to the 202605 branch (Request for 202605 Branch). To move forward with the cherry-pick, could you please:

Test this change on the 202605 branch and add the test results to the PR description

Add the label Tested for 202605 branch once testing is complete

This helps us track readiness and ensures we don't cherry-pick untested changes. Thanks!

Comment by @vaibhavhd via automated tooling

working on it.

liang-nexthop · 2026-07-29T22:52:25Z

This PR has backport request label(s) for branch(es): 202505,202511,202605, but is missing required test information. Please make sure you tick the tested branch(es) in the Tested branch section and provide test evidence (e.g., 202505: ) in the Test result section as well in your PR description.

---Powered by SONiC BuildBot

done

liang-nexthop · 2026-07-29T22:56:20Z

Hi — this PR is flagged for cherry-pick to the 202605 branch (Request for 202605 Branch). To move forward with the cherry-pick, could you please:

Test this change on the 202605 branch and add the test results to the PR description

Add the label Tested for 202605 branch once testing is complete

This helps us track readiness and ensures we don't cherry-pick untested changes. Thanks!

Comment by @vaibhavhd via automated tooling

done.

What: Adds pygnmi==0.8.15 to dockers/docker-sonic-mgmt/Dockerfile.j2. Why: The new native pygnmi-based gNMI client for sonic-mgmt talks gRPC directly to the DUT's gNMI server, replacing the gnmic CLI binary being removed from docker-ptf; that client depends on pygnmi, which isn't currently installed in the docker-sonic-mgmt image. How: Installs the pygnmi Python package in the docker-sonic-mgmt Dockerfile. Testing: CI green (required checks passing); PR approved. Signed-off-by: liang-nexthop <liang@nexthop.ai> Signed-off-by: jackson-micas <jackson@micasnetworks.com>

rookie-who · 2026-08-18T07:39:40Z

@liang-nexthop — This PR is requested for the 202605 branch but is missing test evidence. To move forward with cherry-pick approval, please:

Test on the 202605 branch and add results (image version + test output) to the PR description
Check the 202605 box in the "Tested Branches" section of the PR description
Add the "Tested for 202605 branch" label once testing is complete

Also, to help us prioritize correctly, could you clarify: Is this PR a bug fix / regression fix, a new feature, a platform config change, or a dependency/infra update? Understanding the nature of the change helps us triage the cherry-pick queue.

Thank you!

Review by @vaibhavhd via automated tooling

liang-nexthop marked this pull request as ready for review July 9, 2026 19:25

liang-nexthop requested a review from lguohan as a code owner July 9, 2026 19:25

ronan-nexthop approved these changes Jul 9, 2026

View reviewed changes

mssonicbld added Request for 202505 Branch Request for 202511 Branch Request for 202605 Branch Request for msft-202608 Branch labels Jul 10, 2026

mssonicbld added Cherry Pick Conflict_202505 Cherry Pick Conflict_202511 labels Jul 10, 2026

Install pygnmi==0.8.15 for docker-sonic-mgmt

bb520fd

Signed-off-by: liang-nexthop <liang@nexthop.ai>

liang-nexthop force-pushed the install-pygnmi-0-8-15-docker-sonic-mgmt branch from 7790990 to bb520fd Compare July 10, 2026 18:40

mssonicbld removed the Request for msft-202608 Branch label Jul 11, 2026

hdwhdw requested review from Copilot and hdwhdw July 14, 2026 18:15

hdwhdw approved these changes Jul 14, 2026

View reviewed changes

Copilot started reviewing on behalf of hdwhdw July 14, 2026 18:16 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

hdwhdw mentioned this pull request Jul 14, 2026

[sonic-grpc]: Add gNOI client and generated stubs to sonic-py-common #28341

Merged

1 task

yxieca merged commit 86b830a into sonic-net:master Jul 14, 2026
29 checks passed

hdwhdw mentioned this pull request Jul 20, 2026

[gNMI] Migrate legacy tests to native managed gRPC suites sonic-net/sonic-mgmt#26384

Open

This was referenced Jul 21, 2026

[202605] Backport docker-sonic-mgmt and pipeline updates from master #28536

Merged

[202511] Backport docker-sonic-mgmt dependency and security updates from master #28537

Merged

This was referenced Jul 29, 2026

[202505] Install pygnmi==0.8.15 for docker-sonic-mgmt #28690

Open

[202511] Install pygnmi==0.8.15 for docker-sonic-mgmt #28691

Open

Conversation

liang-nexthop commented Jul 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why I did it

Work item tracking

How I did it

How to verify it

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Uh oh!

mssonicbld commented Jul 9, 2026

Uh oh!

azure-pipelines Bot commented Jul 9, 2026

Uh oh!

ronan-nexthop commented Jul 9, 2026

Uh oh!

hdwhdw commented Jul 10, 2026

Uh oh!

mssonicbld commented Jul 10, 2026

Uh oh!

mssonicbld commented Jul 10, 2026

Uh oh!

azure-pipelines Bot commented Jul 10, 2026

Uh oh!

mssonicbld commented Jul 11, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

mssonicbld commented Jul 14, 2026

Uh oh!

mssonicbld commented Jul 14, 2026

Uh oh!

rookie-who commented Jul 24, 2026

Uh oh!

mssonicbld commented Jul 24, 2026

Uh oh!

liang-nexthop commented Jul 27, 2026

Uh oh!

liang-nexthop commented Jul 29, 2026

Uh oh!

liang-nexthop commented Jul 29, 2026

Uh oh!

rookie-who commented Aug 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

liang-nexthop commented Jul 9, 2026 •

edited

Loading