Skip to content

chore: Take the Go 1.27 runtime defaults - #82

Merged
ecv merged 1 commit into
mainfrom
chore/go-1.27-runtime
Nov 3, 2026
Merged

ecv merged 1 commit into
mainfrom
chore/go-1.27-runtime

Conversation

@ecv

@ecv ecv commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Raises the go.mod go line from 1.26 to 1.27 so the binaries take Go 1.27's runtime defaults. Step 2 of https://github.com/datum-cloud/infra/issues/8340; step 1 already moved the toolchain to go1.27.3.

Setting Old New Effect on this service
tracebacklabels 0 1 Crash tracebacks and debug=2 pprof goroutine dumps include pprof goroutine labels. The repo sets no pprof labels, so no change.
x509sslcertoverrideplatform 0 1 macOS and Windows only: crypto/x509 honours SSL_CERT_FILE / SSL_CERT_DIR. No effect on Linux. The repo ships no macOS or Windows binaries (no goreleaser; Dockerfile only).

go mod tidy also dropped stale go.sum lines it no longer requires; go.mod changes only the go line.

No godebug lines kept.

Modules left alone: none. This is the only go.mod.

Test plan

  • Unit, e2e and lint pass
  • The service soaks in staging for a day with no new errors
  • Released to production on its own tag

Related to datum-cloud/infra#8340

Raise the go.mod go line to 1.27, so the binaries pick up Go 1.27's
default GODEBUG settings instead of 1.26's. Step 1 already builds on the
1.27 toolchain.

Key features/changes:
- tracebacklabels 0 -> 1: crash tracebacks and debug=2 pprof goroutine
  dumps now include pprof goroutine labels. The service sets no labels,
  so nothing changes in practice.
- x509sslcertoverrideplatform 0 -> 1: on macOS and Windows only,
  crypto/x509 honours SSL_CERT_FILE and SSL_CERT_DIR. No effect on
  Linux, where the service runs.
- go.sum drops stale entries that go mod tidy no longer requires.
@ecv
ecv marked this pull request as ready for review November 3, 2026 18:27
@ecv
ecv requested a review from kevwilliams November 3, 2026 18:28
@ecv
ecv enabled auto-merge November 3, 2026 18:28

@kevwilliams kevwilliams 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.

Verified the change is the go directive bump alone, with go.sum only dropping superseded x/ versions.

@ecv
ecv merged commit a5a8153 into main Nov 3, 2026
10 checks passed
@ecv
ecv deleted the chore/go-1.27-runtime branch November 3, 2026 19:39
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