Skip to content

feat: bump go to 1.27 - #21721

Open
kruskall wants to merge 12 commits into
elastic:mainfrom
kruskall:feat/go1.27
Open

kruskall wants to merge 12 commits into
elastic:mainfrom
kruskall:feat/go1.27

Conversation

@kruskall

Copy link
Copy Markdown
Member

Motivation/summary

keep go version up to date
run make fmt
bump swiss lib to fix compile error

Checklist

For functional changes, consider:

  • Is it observable through the addition of either logging or metrics?
  • Is its use being published in telemetry to enable product improvement?
  • Have system tests been added to avoid regression?

How to test these changes

Related issues

@kruskall
kruskall requested a review from a team as a code owner August 24, 2026 04:57
@github-actions

Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify

mergify Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label. Could you fix it @kruskall? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8.19 is the label to automatically backport to the 8.19 branch.
  • backport-9./d is the label to automatically backport to the 9./d branch. /d is the digit.
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@carsonip carsonip added backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches backport-active-8 Automated backport with mergify to all the active 8.[0-9]+ branches labels Sep 7, 2026

@carsonip carsonip left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

could you look into CI failures first

@mergify

mergify Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

This pull request is now in conflicts. Could you fix it @kruskall? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b feat/go1.27 upstream/feat/go1.27
git merge upstream/main
git push upstream feat/go1.27

@carsonip carsonip changed the title feat: bump go to 1.27.0 feat: bump go to 1.27.1 Sep 12, 2026
@lanre-ade
lanre-ade requested a lite review from Copilot September 16, 2026 02:53

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.

🟢 Approval recommended

The reviewed changes are consistent Go 1.27.1 updates with no unresolved review comments.

Pull request overview

Updates the repository and Go submodules to Go 1.27.1 with related tooling and code modernizations.

Changes:

  • Bumps Go versions and Staticcheck dependencies.
  • Applies errors.AsType, slices.Backward, and typed atomic improvements.
  • Updates formatting and composite literals.
File summaries
File Changes
x-pack/apm-server/sampling/reservoir.go Modernizes embedded struct initialization and formatting.
x-pack/apm-server/sampling/processor_test.go Updates test literals and formatting.
x-pack/apm-server/sampling/groups_test.go Simplifies policy literals.
x-pack/apm-server/sampling/config_test.go Simplifies policy initialization.
x-pack/apm-server/main.go Modernizes policy construction.
systemtest/go.mod Updates the system-test Go version.
internal/sourcemap/sourcemap_fetcher_test.go Updates embedded-field literals.
internal/sourcemap/processor.go Uses slices.Backward.
internal/sourcemap/metadata_fetcher_test.go Updates metadata test literals.
internal/sourcemap/elasticsearch.go Uses errors.AsType.
internal/publish/acker.go Uses typed atomic counters.
internal/beater/waitready.go Uses errors.AsType.
internal/beater/middleware/middleware.go Uses slices.Backward.
internal/beater/api/intake/handler.go Uses errors.AsType.
integrationservertest/go.mod Updates the integration-test Go version.
go.sum Refreshes tooling checksums.
go.mod Updates Go and Staticcheck dependencies.
cmd/intake-receiver/go.mod Updates the intake receiver Go version.
Review details
  • Files reviewed: 17/18 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@carsonip

Copy link
Copy Markdown
Member

Blocked by #21924

@kruskall kruskall changed the title feat: bump go to 1.27.1 feat: bump go to 1.27.2 Oct 14, 2026
@mergify

mergify Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

This pull request is now in conflicts. Could you fix it @kruskall? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b feat/go1.27 upstream/feat/go1.27
git merge upstream/main
git push upstream feat/go1.27

@kruskall kruskall changed the title feat: bump go to 1.27.2 feat: bump go to 1.27 Oct 20, 2026
@mergify

mergify Bot commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

This pull request is now in conflicts. Could you fix it @kruskall? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b feat/go1.27 upstream/feat/go1.27
git merge upstream/main
git push upstream feat/go1.27

Copilot AI lite review requested due to automatic review settings November 6, 2026 16:47

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.

🔵 Needs a closer look

Approval should be deferred until the partial Go 1.27 x/net/http2 compatibility-tag routing is validated across direct builds, Make targets, and system tests for TLS and gRPC behavior.

No open issues 🎉

Review effort: Lite

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

History

@aelnahas aelnahas self-assigned this Nov 6, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-active-8 Automated backport with mergify to all the active 8.[0-9]+ branches backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants