Skip to content

fix(docker-ptf): update grpc-go to 1.82.1#28595

Open
auspham wants to merge 2 commits into
sonic-net:masterfrom
auspham:austinpham/update-grpcurl-grpc-1.82.1
Open

fix(docker-ptf): update grpc-go to 1.82.1#28595
auspham wants to merge 2 commits into
sonic-net:masterfrom
auspham:austinpham/update-grpcurl-grpc-1.82.1

Conversation

@auspham

@auspham auspham commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Why I did it

The grpcurl and gnmic binaries in docker-ptf include google.golang.org/grpc v1.79.3, which is affected by GHSA-hrxh-6v49-42gf. The first fixed release is v1.82.1.

Work item tracking
  • Microsoft ADO (number only): N/A

How I did it

Updated the grpc-go version used when building grpcurl and gnmic from source from v1.79.3 to v1.82.1.

How to verify it

Build target/docker-ptf.gz, load the generated image, and scan its language packages with Trivy:

Target                  Type       Vulnerabilities
usr/local/bin/gnmic     gobinary   0
usr/local/bin/grpcurl   gobinary   0

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511
  • 202512
  • 202605
  • 202608

No backport is requested.

Tracking issue/work item for backport/cherry-pick request (GitHub issue or Microsoft ADO): N/A
Failure type: other

Tested branch

  • master
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511
  • 202512
  • 202605
  • 202608
  • N/A

Test result

  • master: target/docker-ptf.gz built successfully. A Trivy image scan with HIGH and CRITICAL library findings enabled reported zero vulnerabilities for both usr/local/bin/gnmic and usr/local/bin/grpcurl.

Description for the changelog

Update grpcurl and gnmic to use grpc-go v1.82.1 and resolve GHSA-hrxh-6v49-42gf.

Link to config_db schema for YANG module changes

N/A

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

N/A

Update grpcurl to use the first grpc-go release that fixes GHSA-hrxh-6v49-42gf.

Copilot-Session: e838b61b-d284-4649-9a41-38824c1a742a
Signed-off-by: Austin (Ngoc Thang) Pham <austinpham@microsoft.com>
Copilot AI review requested due to automatic review settings July 23, 2026 05:10
@auspham
auspham requested a review from lguohan as a code owner July 23, 2026 05:10
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the docker-ptf image’s grpcurl source-build to pin google.golang.org/grpc to a vulnerability-fixed version, aligning the container’s embedded Go dependencies with the remediation noted in the PR description.

Changes:

  • Bump grpc-go used by grpcurl build from v1.79.3 to v1.82.1.

yijingyan2
yijingyan2 previously approved these changes Jul 23, 2026
Pin gnmic to the fixed grpc-go release so the docker-ptf image no longer reports GHSA-hrxh-6v49-42gf.

Copilot-Session: e838b61b-d284-4649-9a41-38824c1a742a
Signed-off-by: Austin (Ngoc Thang) Pham <austinpham@microsoft.com>
Copilot AI review requested due to automatic review settings July 23, 2026 05:53
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@auspham

auspham commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

/azpw retry

@mssonicbld

Copy link
Copy Markdown
Collaborator

Retrying failed(or canceled) jobs...

@mssonicbld

Copy link
Copy Markdown
Collaborator

Retrying failed(or canceled) stages in build 1172648:

✅Stage Test:

  • Job impacted-area-kvmtest-t1-lag-vpp by Elastictest: retried.

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.

4 participants