Skip to content

Upgrade Go to 1.27.3 - #2984

Merged
Benjamin Elder (BenTheElder) merged 3 commits into
agent-substrate:mainfrom
BenTheElder:go-1.27.3
Oct 20, 2026
Merged

Benjamin Elder (BenTheElder) merged 3 commits into
agent-substrate:mainfrom
BenTheElder:go-1.27.3

Conversation

@BenTheElder

@BenTheElder Benjamin Elder (BenTheElder) commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Go has a critical security fix. Longterm we're going to automate these but dependabot can't handle it and we're undecided on solution, Grant McCloskey (@MushuEE) / Steven Shriver (@stshrive) are looking.

NOTE: we don't have to bump the dockerfiles for go bumps, but in this case TLS is vulnerable so the download mechanism for gotoolchain is impacted, so I'm being extra-comprehensive.

AI use: Yes, I reviewed everything. Trivial diff, and tests were run

  • Tests pass
  • Docs updated

@BenTheElder Benjamin Elder (BenTheElder) added the area/security Security related issue/pr label Oct 20, 2026
@BenTheElder Benjamin Elder (BenTheElder) changed the title Go 1.27.3 Upgrade Go to 1.27.3 Oct 20, 2026
@BenTheElder

Copy link
Copy Markdown
Collaborator Author

CI is going to take longer here, because go upgrade will bust the cache. We've clearly been saving a lot of time with the cache.

@BenTheElder
Benjamin Elder (BenTheElder) added this pull request to the merge queue Oct 20, 2026
Merged via the queue into agent-substrate:main with commit 3c9a3cd Oct 20, 2026
15 checks passed
Quentin Bisson (QuentinBisson) added a commit to giantswarm/substrate that referenced this pull request Oct 23, 2026
agent-substrate#2984 added toolchain go1.27.3 to every module, so
the Go pin commit of require-golang-org-x-net-v0-61 and pin-go-1-27-3 are
class 2 and drop at the re-pin onto a commit containing 3c9a3cd. The one-line
toolchain pull request drafted on 2026-10-20 is withdrawn; upstream's own
automation is agent-substrate#3013 (agent-substrate#3021, agent-substrate#3025).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/security Security related issue/pr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants