Skip to content

Update golang version to 1.26.1 to avoid vulnerability alerts.#628

Open
liushilongbuaa wants to merge 1 commit intosonic-net:masterfrom
liushilongbuaa:fix-vul
Open

Update golang version to 1.26.1 to avoid vulnerability alerts.#628
liushilongbuaa wants to merge 1 commit intosonic-net:masterfrom
liushilongbuaa:fix-vul

Conversation

@liushilongbuaa
Copy link
Copy Markdown
Contributor

@liushilongbuaa liushilongbuaa commented Mar 23, 2026

Why I did it

I'm working to fix GO-2026-4337.
before go1.24.13, from go1.25.0-0 before go1.25.7, from go1.26.0-rc.1 before go1.26.0-rc.3

How I did it

How to verify it

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

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

Description for the changelog

Link to config_db schema for YANG module changes

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

Copilot AI review requested due to automatic review settings March 23, 2026 05:25
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Mar 23, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: liushilongbuaa / name: Liu Shilong (2a6e90e)

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@liushilongbuaa liushilongbuaa changed the title Fix vul Update golang version to 1.26.1 to avoid vulnerability alerts. Mar 23, 2026
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR appears to update the repository’s Go toolchain/versioning and dependency metadata (likely as part of a vulnerability/security hardening effort) and adjusts build tooling behavior.

Changes:

  • Update Go version in go.mod and Azure Pipelines.
  • Tidy go.sum entries for removed/updated transitive dependencies.
  • Add -buildvcs=false to go install invocations in the Makefile.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File Description
go.sum Removes stale sums for older transitive module versions.
go.mod Updates Go version declaration and adjusts dependency direct/indirect classification.
azure-pipelines.yml Bumps Go version and adds a Go install step for CI jobs.
Makefile Disables VCS stamping for select go install commands.

Comment thread go.mod Outdated
Comment thread azure-pipelines.yml Outdated
Comment thread go.mod Outdated
@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: shilongliu <shilongliu@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@hdwhdw
Copy link
Copy Markdown
Contributor

hdwhdw commented Mar 24, 2026

This will not work as the go in gnmi container is based on bookworm right now (<= go 1.19) or even if we migrate to triie, go can at most be 1.24 unless we create new build recipes.

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