From cd70a84da683239bc50cad974528e16c92209103 Mon Sep 17 00:00:00 2001 From: alex boten <223565+codeboten@users.noreply.github.com> Date: Tue, 10 Feb 2026 15:21:44 -0800 Subject: [PATCH 1/8] bump minimum version of go to 1.25 Follows the normal procedure with the release of go 1.26 Signed-off-by: alex boten <223565+codeboten@users.noreply.github.com> --- CONTRIBUTING.md | 4 ++-- Makefile.Common | 2 +- client/go.mod | 2 +- cmd/builder/go.mod | 2 +- cmd/builder/internal/builder/main.go | 2 +- cmd/builder/internal/builder/templates/go.mod.tmpl | 2 +- cmd/builder/internal/init/templates/go.mod.tmpl | 2 +- cmd/mdatagen/go.mod | 2 +- cmd/otelcorecol/go.mod | 2 +- component/componentstatus/go.mod | 2 +- component/componenttest/go.mod | 2 +- component/go.mod | 2 +- config/configauth/go.mod | 2 +- config/configcompression/go.mod | 2 +- config/configgrpc/go.mod | 2 +- config/confighttp/go.mod | 2 +- config/confighttp/xconfighttp/go.mod | 2 +- config/configmiddleware/go.mod | 2 +- config/confignet/go.mod | 2 +- config/configopaque/go.mod | 2 +- config/configoptional/go.mod | 2 +- config/configretry/go.mod | 2 +- config/configtelemetry/go.mod | 2 +- config/configtls/go.mod | 2 +- confmap/go.mod | 2 +- confmap/internal/e2e/go.mod | 2 +- confmap/provider/envprovider/go.mod | 2 +- confmap/provider/fileprovider/go.mod | 2 +- confmap/provider/httpprovider/go.mod | 2 +- confmap/provider/httpsprovider/go.mod | 2 +- confmap/provider/yamlprovider/go.mod | 2 +- confmap/xconfmap/go.mod | 2 +- connector/connectortest/go.mod | 2 +- connector/forwardconnector/go.mod | 2 +- connector/go.mod | 2 +- connector/xconnector/go.mod | 2 +- consumer/consumererror/go.mod | 2 +- consumer/consumererror/xconsumererror/go.mod | 2 +- consumer/consumertest/go.mod | 2 +- consumer/go.mod | 2 +- consumer/xconsumer/go.mod | 2 +- exporter/debugexporter/go.mod | 2 +- exporter/exporterhelper/go.mod | 2 +- exporter/exporterhelper/xexporterhelper/go.mod | 2 +- exporter/exportertest/go.mod | 2 +- exporter/go.mod | 2 +- exporter/nopexporter/go.mod | 2 +- exporter/otlpexporter/go.mod | 2 +- exporter/otlphttpexporter/go.mod | 2 +- exporter/xexporter/go.mod | 2 +- extension/extensionauth/extensionauthtest/go.mod | 2 +- extension/extensionauth/go.mod | 2 +- extension/extensioncapabilities/go.mod | 2 +- extension/extensionmiddleware/extensionmiddlewaretest/go.mod | 2 +- extension/extensionmiddleware/go.mod | 2 +- extension/extensiontest/go.mod | 2 +- extension/go.mod | 2 +- extension/memorylimiterextension/go.mod | 2 +- extension/xextension/go.mod | 2 +- extension/zpagesextension/go.mod | 2 +- featuregate/go.mod | 2 +- filter/go.mod | 2 +- go.mod | 2 +- internal/cmd/pdatagen/go.mod | 2 +- internal/componentalias/go.mod | 2 +- internal/e2e/go.mod | 2 +- internal/fanoutconsumer/go.mod | 2 +- internal/memorylimiter/go.mod | 2 +- internal/sharedcomponent/go.mod | 2 +- internal/telemetry/go.mod | 2 +- internal/testutil/go.mod | 2 +- internal/tools/go.mod | 2 +- otelcol/go.mod | 2 +- otelcol/otelcoltest/go.mod | 2 +- pdata/go.mod | 2 +- pdata/pprofile/go.mod | 2 +- pdata/testdata/go.mod | 2 +- pdata/xpdata/go.mod | 2 +- pipeline/go.mod | 2 +- pipeline/xpipeline/go.mod | 2 +- processor/batchprocessor/go.mod | 2 +- processor/go.mod | 2 +- processor/memorylimiterprocessor/go.mod | 2 +- processor/processorhelper/go.mod | 2 +- processor/processorhelper/xprocessorhelper/go.mod | 2 +- processor/processortest/go.mod | 2 +- processor/xprocessor/go.mod | 2 +- receiver/go.mod | 2 +- receiver/nopreceiver/go.mod | 2 +- receiver/otlpreceiver/go.mod | 2 +- receiver/receiverhelper/go.mod | 2 +- receiver/receivertest/go.mod | 2 +- receiver/xreceiver/go.mod | 2 +- renovate.json | 2 +- scraper/go.mod | 2 +- scraper/scraperhelper/go.mod | 2 +- scraper/scraperhelper/xscraperhelper/go.mod | 2 +- scraper/scrapertest/go.mod | 2 +- scraper/xscraper/go.mod | 2 +- service/go.mod | 2 +- service/hostcapabilities/go.mod | 2 +- service/telemetry/telemetrytest/go.mod | 2 +- 102 files changed, 103 insertions(+), 103 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 58f2f7c2bd9..d880a3161f2 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -202,7 +202,7 @@ section of the general project contributing guide. Working with the project sources requires the following tools: 1. [git](https://git-scm.com/) -2. [go](https://golang.org/) (version 1.24 and up) +2. [go](https://golang.org/) (version 1.25 and up) 3. [make](https://www.gnu.org/software/make/) 4. [docker](https://www.docker.com/) @@ -259,7 +259,7 @@ before merging (but see the above paragraph about writing good commit messages i ## General Notes -This project uses Go 1.24.* and [Github Actions.](https://github.com/features/actions) +This project uses Go 1.25.* and [Github Actions.](https://github.com/features/actions) It is recommended to run `make gofmt all` before submitting your PR. diff --git a/Makefile.Common b/Makefile.Common index 4837f816d83..26055eab6b5 100644 --- a/Makefile.Common +++ b/Makefile.Common @@ -62,7 +62,7 @@ modernize: .PHONY: tidy tidy: rm -fr go.sum - $(GOCMD) mod tidy -compat=1.24.0 + $(GOCMD) mod tidy -compat=1.25.0 .PHONY: lint lint: diff --git a/client/go.mod b/client/go.mod index 120b89bb5fd..c356f6057dd 100644 --- a/client/go.mod +++ b/client/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/client -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/cmd/builder/go.mod b/cmd/builder/go.mod index 20ee020a1b3..96084a269d0 100644 --- a/cmd/builder/go.mod +++ b/cmd/builder/go.mod @@ -3,7 +3,7 @@ module go.opentelemetry.io/collector/cmd/builder -go 1.24.0 +go 1.25.0 require ( github.com/knadh/koanf/parsers/yaml v1.1.0 diff --git a/cmd/builder/internal/builder/main.go b/cmd/builder/internal/builder/main.go index 4f8a44bd16c..5821f84c8fc 100644 --- a/cmd/builder/internal/builder/main.go +++ b/cmd/builder/internal/builder/main.go @@ -158,7 +158,7 @@ func GetModules(cfg *Config) error { return nil } - if _, err := runGoCommand(cfg, "mod", "tidy", "-compat=1.24"); err != nil { + if _, err := runGoCommand(cfg, "mod", "tidy", "-compat=1.25"); err != nil { return fmt.Errorf("failed to update go.mod: %w", err) } diff --git a/cmd/builder/internal/builder/templates/go.mod.tmpl b/cmd/builder/internal/builder/templates/go.mod.tmpl index 13ce9852a9a..2269c2dc44c 100644 --- a/cmd/builder/internal/builder/templates/go.mod.tmpl +++ b/cmd/builder/internal/builder/templates/go.mod.tmpl @@ -2,7 +2,7 @@ module {{.Distribution.Module}} -go 1.24 +go 1.25 require ( {{- range .ConfmapConverters}} diff --git a/cmd/builder/internal/init/templates/go.mod.tmpl b/cmd/builder/internal/init/templates/go.mod.tmpl index c6982ed9895..fadab362d31 100644 --- a/cmd/builder/internal/init/templates/go.mod.tmpl +++ b/cmd/builder/internal/init/templates/go.mod.tmpl @@ -1,5 +1,5 @@ module {{.Name}} -go 1.24 +go 1.25 tool go.opentelemetry.io/collector/cmd/builder diff --git a/cmd/mdatagen/go.mod b/cmd/mdatagen/go.mod index ccab6d12563..6659131df80 100644 --- a/cmd/mdatagen/go.mod +++ b/cmd/mdatagen/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/cmd/mdatagen -go 1.24.0 +go 1.25.0 require ( github.com/google/go-cmp v0.7.0 diff --git a/cmd/otelcorecol/go.mod b/cmd/otelcorecol/go.mod index eacffe5186d..fa4cbc454c5 100644 --- a/cmd/otelcorecol/go.mod +++ b/cmd/otelcorecol/go.mod @@ -2,7 +2,7 @@ module go.opentelemetry.io/collector/cmd/otelcorecol -go 1.24.0 +go 1.25.0 require ( go.opentelemetry.io/collector/component v1.51.0 diff --git a/component/componentstatus/go.mod b/component/componentstatus/go.mod index ae7e85afe86..78a9257d44c 100644 --- a/component/componentstatus/go.mod +++ b/component/componentstatus/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/component/componentstatus -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/component/componenttest/go.mod b/component/componenttest/go.mod index 38054fab180..0dee1ca6457 100644 --- a/component/componenttest/go.mod +++ b/component/componenttest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/component/componenttest -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/component/go.mod b/component/go.mod index f2aff4fd133..e2f8e9ceb15 100644 --- a/component/go.mod +++ b/component/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/component -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configauth/go.mod b/config/configauth/go.mod index e686e1d809b..cf03e4756f7 100644 --- a/config/configauth/go.mod +++ b/config/configauth/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configauth -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configcompression/go.mod b/config/configcompression/go.mod index 73791528737..bc56bf0cdb9 100644 --- a/config/configcompression/go.mod +++ b/config/configcompression/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configcompression -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configgrpc/go.mod b/config/configgrpc/go.mod index 127e5e774e0..2b573ba9267 100644 --- a/config/configgrpc/go.mod +++ b/config/configgrpc/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configgrpc -go 1.24.0 +go 1.25.0 require ( github.com/mostynb/go-grpc-compression v1.2.3 diff --git a/config/confighttp/go.mod b/config/confighttp/go.mod index 0b39ffb70c6..c60b885e33b 100644 --- a/config/confighttp/go.mod +++ b/config/confighttp/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/confighttp -go 1.24.0 +go 1.25.0 require ( github.com/golang/snappy v1.0.0 diff --git a/config/confighttp/xconfighttp/go.mod b/config/confighttp/xconfighttp/go.mod index 9ecfe2ce638..7aed2b71eca 100644 --- a/config/confighttp/xconfighttp/go.mod +++ b/config/confighttp/xconfighttp/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/confighttp/xconfighttp -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configmiddleware/go.mod b/config/configmiddleware/go.mod index 5a5b4d446d9..40127175629 100644 --- a/config/configmiddleware/go.mod +++ b/config/configmiddleware/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configmiddleware -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/confignet/go.mod b/config/confignet/go.mod index 9fd5f557643..42c371d5507 100644 --- a/config/confignet/go.mod +++ b/config/confignet/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/confignet -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configopaque/go.mod b/config/configopaque/go.mod index 1e1fce4faa9..6354df0c421 100644 --- a/config/configopaque/go.mod +++ b/config/configopaque/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configopaque -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configoptional/go.mod b/config/configoptional/go.mod index bc417d1bcc1..58af5eec247 100644 --- a/config/configoptional/go.mod +++ b/config/configoptional/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configoptional -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configretry/go.mod b/config/configretry/go.mod index 215a66c17a2..900d0bf06c8 100644 --- a/config/configretry/go.mod +++ b/config/configretry/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configretry -go 1.24.0 +go 1.25.0 require ( github.com/cenkalti/backoff/v5 v5.0.3 diff --git a/config/configtelemetry/go.mod b/config/configtelemetry/go.mod index d7cf1c4f0d8..c5c3750fea5 100644 --- a/config/configtelemetry/go.mod +++ b/config/configtelemetry/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configtelemetry -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/config/configtls/go.mod b/config/configtls/go.mod index e23722f1745..ac1fbdb2898 100644 --- a/config/configtls/go.mod +++ b/config/configtls/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/config/configtls -go 1.24.0 +go 1.25.0 require ( github.com/foxboron/go-tpm-keyfiles v0.0.0-20250903184740-5d135037bd4d diff --git a/confmap/go.mod b/confmap/go.mod index c96eee5edea..3cbce079941 100644 --- a/confmap/go.mod +++ b/confmap/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap -go 1.24.0 +go 1.25.0 require ( github.com/go-viper/mapstructure/v2 v2.5.0 diff --git a/confmap/internal/e2e/go.mod b/confmap/internal/e2e/go.mod index f00949b0d13..7cd95771d3c 100644 --- a/confmap/internal/e2e/go.mod +++ b/confmap/internal/e2e/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/internal/e2e -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/confmap/provider/envprovider/go.mod b/confmap/provider/envprovider/go.mod index 25d867fa83d..b62aaa662c4 100644 --- a/confmap/provider/envprovider/go.mod +++ b/confmap/provider/envprovider/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/provider/envprovider -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/confmap/provider/fileprovider/go.mod b/confmap/provider/fileprovider/go.mod index f4265c94a97..0240a6b4a56 100644 --- a/confmap/provider/fileprovider/go.mod +++ b/confmap/provider/fileprovider/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/provider/fileprovider -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/confmap/provider/httpprovider/go.mod b/confmap/provider/httpprovider/go.mod index 188fcbed2fa..03ade44747e 100644 --- a/confmap/provider/httpprovider/go.mod +++ b/confmap/provider/httpprovider/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/provider/httpprovider -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/confmap/provider/httpsprovider/go.mod b/confmap/provider/httpsprovider/go.mod index 3dd57003390..aef93e95a32 100644 --- a/confmap/provider/httpsprovider/go.mod +++ b/confmap/provider/httpsprovider/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/provider/httpsprovider -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/confmap/provider/yamlprovider/go.mod b/confmap/provider/yamlprovider/go.mod index b1d15dba971..fe4ebff9083 100644 --- a/confmap/provider/yamlprovider/go.mod +++ b/confmap/provider/yamlprovider/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/provider/yamlprovider -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/confmap/xconfmap/go.mod b/confmap/xconfmap/go.mod index 82dfcfdb31f..a638e577672 100644 --- a/confmap/xconfmap/go.mod +++ b/confmap/xconfmap/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/confmap/xconfmap -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/connector/connectortest/go.mod b/connector/connectortest/go.mod index be1ec814e67..2fb012bd528 100644 --- a/connector/connectortest/go.mod +++ b/connector/connectortest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/connector/connectortest -go 1.24.0 +go 1.25.0 require ( github.com/google/uuid v1.6.0 diff --git a/connector/forwardconnector/go.mod b/connector/forwardconnector/go.mod index 36b09438a80..4ab78ae7974 100644 --- a/connector/forwardconnector/go.mod +++ b/connector/forwardconnector/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/connector/forwardconnector -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/connector/go.mod b/connector/go.mod index bbebcfae3af..36fe65eb8ab 100644 --- a/connector/go.mod +++ b/connector/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/connector -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/connector/xconnector/go.mod b/connector/xconnector/go.mod index ecaa7200b19..fd976687bf0 100644 --- a/connector/xconnector/go.mod +++ b/connector/xconnector/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/connector/xconnector -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/consumer/consumererror/go.mod b/consumer/consumererror/go.mod index 9ded04e8777..4d3397ce34a 100644 --- a/consumer/consumererror/go.mod +++ b/consumer/consumererror/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/consumer/consumererror -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/consumer/consumererror/xconsumererror/go.mod b/consumer/consumererror/xconsumererror/go.mod index 42b121f0773..ac61e57ce13 100644 --- a/consumer/consumererror/xconsumererror/go.mod +++ b/consumer/consumererror/xconsumererror/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/consumer/consumererror/xconsumererror -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/consumer/consumertest/go.mod b/consumer/consumertest/go.mod index 2678543151d..7a05fd65758 100644 --- a/consumer/consumertest/go.mod +++ b/consumer/consumertest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/consumer/consumertest -go 1.24.0 +go 1.25.0 replace go.opentelemetry.io/collector/consumer => ../ diff --git a/consumer/go.mod b/consumer/go.mod index a6f6616f711..d7ddd385ca9 100644 --- a/consumer/go.mod +++ b/consumer/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/consumer -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/consumer/xconsumer/go.mod b/consumer/xconsumer/go.mod index 4dfad778442..97c9a5f3b69 100644 --- a/consumer/xconsumer/go.mod +++ b/consumer/xconsumer/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/consumer/xconsumer -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/debugexporter/go.mod b/exporter/debugexporter/go.mod index cdce8291857..a540065baf6 100644 --- a/exporter/debugexporter/go.mod +++ b/exporter/debugexporter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/debugexporter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/exporterhelper/go.mod b/exporter/exporterhelper/go.mod index 82a28e1c4b1..58133810291 100644 --- a/exporter/exporterhelper/go.mod +++ b/exporter/exporterhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/exporterhelper -go 1.24.0 +go 1.25.0 require ( github.com/cenkalti/backoff/v5 v5.0.3 diff --git a/exporter/exporterhelper/xexporterhelper/go.mod b/exporter/exporterhelper/xexporterhelper/go.mod index c07f25152a9..484dc8da1ed 100644 --- a/exporter/exporterhelper/xexporterhelper/go.mod +++ b/exporter/exporterhelper/xexporterhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/exporterhelper/xexporterhelper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/exportertest/go.mod b/exporter/exportertest/go.mod index f3bfa82350c..c04e53b0c6d 100644 --- a/exporter/exportertest/go.mod +++ b/exporter/exportertest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/exportertest -go 1.24.0 +go 1.25.0 require ( github.com/google/uuid v1.6.0 diff --git a/exporter/go.mod b/exporter/go.mod index 80bc6c16ab4..3fa5a9a1a03 100644 --- a/exporter/go.mod +++ b/exporter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/nopexporter/go.mod b/exporter/nopexporter/go.mod index 748a5b864c1..28dc7716ca5 100644 --- a/exporter/nopexporter/go.mod +++ b/exporter/nopexporter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/nopexporter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/otlpexporter/go.mod b/exporter/otlpexporter/go.mod index a78b6bdc847..1fd86de39c7 100644 --- a/exporter/otlpexporter/go.mod +++ b/exporter/otlpexporter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/otlpexporter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/otlphttpexporter/go.mod b/exporter/otlphttpexporter/go.mod index 03277e545dd..d89dd93de3d 100644 --- a/exporter/otlphttpexporter/go.mod +++ b/exporter/otlphttpexporter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/otlphttpexporter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/exporter/xexporter/go.mod b/exporter/xexporter/go.mod index cf815c60464..e4c838d22c6 100644 --- a/exporter/xexporter/go.mod +++ b/exporter/xexporter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/exporter/xexporter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/extensionauth/extensionauthtest/go.mod b/extension/extensionauth/extensionauthtest/go.mod index 6d67bbedd2a..8aff45f5aaf 100644 --- a/extension/extensionauth/extensionauthtest/go.mod +++ b/extension/extensionauth/extensionauthtest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/extensionauth/extensionauthtest -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/extensionauth/go.mod b/extension/extensionauth/go.mod index 4ec86605d4d..c15a5bc8757 100644 --- a/extension/extensionauth/go.mod +++ b/extension/extensionauth/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/extensionauth -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/extensioncapabilities/go.mod b/extension/extensioncapabilities/go.mod index fb6213e7f29..5c813408f2c 100644 --- a/extension/extensioncapabilities/go.mod +++ b/extension/extensioncapabilities/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/extensioncapabilities -go 1.24.0 +go 1.25.0 require ( go.opentelemetry.io/collector/component v1.51.0 diff --git a/extension/extensionmiddleware/extensionmiddlewaretest/go.mod b/extension/extensionmiddleware/extensionmiddlewaretest/go.mod index d20f4a3f46f..e6e1e090c25 100644 --- a/extension/extensionmiddleware/extensionmiddlewaretest/go.mod +++ b/extension/extensionmiddleware/extensionmiddlewaretest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/extensionmiddleware/extensionmiddlewaretest -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/extensionmiddleware/go.mod b/extension/extensionmiddleware/go.mod index 6008c67ad47..6d573f79e78 100644 --- a/extension/extensionmiddleware/go.mod +++ b/extension/extensionmiddleware/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/extensionmiddleware -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/extensiontest/go.mod b/extension/extensiontest/go.mod index 9a4cd40bed7..c46e3806ced 100644 --- a/extension/extensiontest/go.mod +++ b/extension/extensiontest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/extensiontest -go 1.24.0 +go 1.25.0 replace go.opentelemetry.io/collector/extension => .. diff --git a/extension/go.mod b/extension/go.mod index d82a29217b6..29f9b5d1d2a 100644 --- a/extension/go.mod +++ b/extension/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/memorylimiterextension/go.mod b/extension/memorylimiterextension/go.mod index 2eefeb64397..5f79b3aab8c 100644 --- a/extension/memorylimiterextension/go.mod +++ b/extension/memorylimiterextension/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/memorylimiterextension -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/xextension/go.mod b/extension/xextension/go.mod index e6bff101ea3..f1e681d52c4 100644 --- a/extension/xextension/go.mod +++ b/extension/xextension/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/xextension -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/extension/zpagesextension/go.mod b/extension/zpagesextension/go.mod index 41f8e090cd5..6f23fcf2f49 100644 --- a/extension/zpagesextension/go.mod +++ b/extension/zpagesextension/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/extension/zpagesextension -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/featuregate/go.mod b/featuregate/go.mod index e6131c87819..81e493bd33c 100644 --- a/featuregate/go.mod +++ b/featuregate/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/featuregate -go 1.24.0 +go 1.25.0 require ( github.com/hashicorp/go-version v1.8.0 diff --git a/filter/go.mod b/filter/go.mod index 2dfc3e6c0cd..1f398fe15f5 100644 --- a/filter/go.mod +++ b/filter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/filter -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/go.mod b/go.mod index fd9cd6c558c..7ed176bf2e3 100644 --- a/go.mod +++ b/go.mod @@ -8,7 +8,7 @@ module go.opentelemetry.io/collector // For the OpenTelemetry Collector Core distribution specifically, see // https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/internal/cmd/pdatagen/go.mod b/internal/cmd/pdatagen/go.mod index 8b36d25892b..a3864912424 100644 --- a/internal/cmd/pdatagen/go.mod +++ b/internal/cmd/pdatagen/go.mod @@ -1,5 +1,5 @@ module go.opentelemetry.io/collector/internal/cmd/pdatagen -go 1.24.0 +go 1.25.0 require github.com/ettle/strcase v0.2.0 diff --git a/internal/componentalias/go.mod b/internal/componentalias/go.mod index e6aeefe89a7..049d74cc6fa 100644 --- a/internal/componentalias/go.mod +++ b/internal/componentalias/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/componentalias -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/internal/e2e/go.mod b/internal/e2e/go.mod index 4343d81ddbe..a663297c80d 100644 --- a/internal/e2e/go.mod +++ b/internal/e2e/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/e2e -go 1.24.0 +go 1.25.0 require ( github.com/google/go-cmp v0.7.0 diff --git a/internal/fanoutconsumer/go.mod b/internal/fanoutconsumer/go.mod index d053a7bc822..c809ede181e 100644 --- a/internal/fanoutconsumer/go.mod +++ b/internal/fanoutconsumer/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/fanoutconsumer -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/internal/memorylimiter/go.mod b/internal/memorylimiter/go.mod index 7d6ed7180fc..e8510904a31 100644 --- a/internal/memorylimiter/go.mod +++ b/internal/memorylimiter/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/memorylimiter -go 1.24.0 +go 1.25.0 require ( github.com/shirou/gopsutil/v4 v4.26.1 diff --git a/internal/sharedcomponent/go.mod b/internal/sharedcomponent/go.mod index 741001f6759..52e9e351bb8 100644 --- a/internal/sharedcomponent/go.mod +++ b/internal/sharedcomponent/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/sharedcomponent -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/internal/telemetry/go.mod b/internal/telemetry/go.mod index e174a149029..214541e8f90 100644 --- a/internal/telemetry/go.mod +++ b/internal/telemetry/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/telemetry -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/internal/testutil/go.mod b/internal/testutil/go.mod index ed30c788425..99abd51e986 100644 --- a/internal/testutil/go.mod +++ b/internal/testutil/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/testutil -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/internal/tools/go.mod b/internal/tools/go.mod index d54ecd1852e..7b7b1705a19 100644 --- a/internal/tools/go.mod +++ b/internal/tools/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/internal/tools -go 1.24.0 +go 1.25.0 tool ( github.com/a8m/envsubst/cmd/envsubst diff --git a/otelcol/go.mod b/otelcol/go.mod index a53433bd9b1..3d110c0e807 100644 --- a/otelcol/go.mod +++ b/otelcol/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/otelcol -go 1.24.0 +go 1.25.0 require ( github.com/spf13/cobra v1.10.2 diff --git a/otelcol/otelcoltest/go.mod b/otelcol/otelcoltest/go.mod index c324a8576c2..1aa46879848 100644 --- a/otelcol/otelcoltest/go.mod +++ b/otelcol/otelcoltest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/otelcol/otelcoltest -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/pdata/go.mod b/pdata/go.mod index 67f8c41721f..69942c9f8e8 100644 --- a/pdata/go.mod +++ b/pdata/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/pdata -go 1.24.0 +go 1.25.0 require ( github.com/json-iterator/go v1.1.12 diff --git a/pdata/pprofile/go.mod b/pdata/pprofile/go.mod index 24ca7a0412b..fdd7b5baccc 100644 --- a/pdata/pprofile/go.mod +++ b/pdata/pprofile/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/pdata/pprofile -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/pdata/testdata/go.mod b/pdata/testdata/go.mod index 6764921a1d7..010ea13915a 100644 --- a/pdata/testdata/go.mod +++ b/pdata/testdata/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/pdata/testdata -go 1.24.0 +go 1.25.0 require ( go.opentelemetry.io/collector/pdata v1.51.0 diff --git a/pdata/xpdata/go.mod b/pdata/xpdata/go.mod index b8b274c03a3..9e0024edc76 100644 --- a/pdata/xpdata/go.mod +++ b/pdata/xpdata/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/pdata/xpdata -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/pipeline/go.mod b/pipeline/go.mod index 2adeeacf7b0..866e8ced317 100644 --- a/pipeline/go.mod +++ b/pipeline/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/pipeline -go 1.24.0 +go 1.25.0 require github.com/stretchr/testify v1.11.1 diff --git a/pipeline/xpipeline/go.mod b/pipeline/xpipeline/go.mod index 89dc05163db..4bab01c882e 100644 --- a/pipeline/xpipeline/go.mod +++ b/pipeline/xpipeline/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/pipeline/xpipeline -go 1.24.0 +go 1.25.0 require go.opentelemetry.io/collector/pipeline v1.51.0 diff --git a/processor/batchprocessor/go.mod b/processor/batchprocessor/go.mod index e1b971e94a8..b95f25fc7c0 100644 --- a/processor/batchprocessor/go.mod +++ b/processor/batchprocessor/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor/batchprocessor -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/processor/go.mod b/processor/go.mod index 59d0272a684..001ec8111a9 100644 --- a/processor/go.mod +++ b/processor/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/processor/memorylimiterprocessor/go.mod b/processor/memorylimiterprocessor/go.mod index 3e69e1d012e..4b66a697589 100644 --- a/processor/memorylimiterprocessor/go.mod +++ b/processor/memorylimiterprocessor/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor/memorylimiterprocessor -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/processor/processorhelper/go.mod b/processor/processorhelper/go.mod index 2b66e2d8e2b..890b6d8944f 100644 --- a/processor/processorhelper/go.mod +++ b/processor/processorhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor/processorhelper -go 1.24.0 +go 1.25.0 replace go.opentelemetry.io/collector/processor => ../ diff --git a/processor/processorhelper/xprocessorhelper/go.mod b/processor/processorhelper/xprocessorhelper/go.mod index 41553032381..c212a93b113 100644 --- a/processor/processorhelper/xprocessorhelper/go.mod +++ b/processor/processorhelper/xprocessorhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor/processorhelper/xprocessorhelper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/processor/processortest/go.mod b/processor/processortest/go.mod index c7c4bef9ea2..76063579f3b 100644 --- a/processor/processortest/go.mod +++ b/processor/processortest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor/processortest -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/processor/xprocessor/go.mod b/processor/xprocessor/go.mod index a77951bb17c..59e6897f0c2 100644 --- a/processor/xprocessor/go.mod +++ b/processor/xprocessor/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/processor/xprocessor -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/receiver/go.mod b/receiver/go.mod index 7627c3b91d6..46397b188ca 100644 --- a/receiver/go.mod +++ b/receiver/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/receiver -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/receiver/nopreceiver/go.mod b/receiver/nopreceiver/go.mod index c56278f62f2..5784ad15404 100644 --- a/receiver/nopreceiver/go.mod +++ b/receiver/nopreceiver/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/receiver/nopreceiver -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/receiver/otlpreceiver/go.mod b/receiver/otlpreceiver/go.mod index 11b014312b1..d899652c96f 100644 --- a/receiver/otlpreceiver/go.mod +++ b/receiver/otlpreceiver/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/receiver/otlpreceiver -go 1.24.0 +go 1.25.0 require ( github.com/klauspost/compress v1.18.4 diff --git a/receiver/receiverhelper/go.mod b/receiver/receiverhelper/go.mod index ed8275e289a..7b5665b1553 100644 --- a/receiver/receiverhelper/go.mod +++ b/receiver/receiverhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/receiver/receiverhelper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/receiver/receivertest/go.mod b/receiver/receivertest/go.mod index cb9087f72d0..339cd1b7ab7 100644 --- a/receiver/receivertest/go.mod +++ b/receiver/receivertest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/receiver/receivertest -go 1.24.0 +go 1.25.0 require ( github.com/google/uuid v1.6.0 diff --git a/receiver/xreceiver/go.mod b/receiver/xreceiver/go.mod index 3758bf472a0..5557b0d8bae 100644 --- a/receiver/xreceiver/go.mod +++ b/receiver/xreceiver/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/receiver/xreceiver -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/renovate.json b/renovate.json index 985a03ccf44..b5e51f72681 100644 --- a/renovate.json +++ b/renovate.json @@ -5,7 +5,7 @@ "dependencies" ], "constraints": { - "go": "1.24" + "go": "1.25" }, "extends": [ "config:recommended", diff --git a/scraper/go.mod b/scraper/go.mod index 5072297a8e2..acc61129469 100644 --- a/scraper/go.mod +++ b/scraper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/scraper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/scraper/scraperhelper/go.mod b/scraper/scraperhelper/go.mod index e3e6f4bd5e1..c27dcbf1ce3 100644 --- a/scraper/scraperhelper/go.mod +++ b/scraper/scraperhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/scraper/scraperhelper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/scraper/scraperhelper/xscraperhelper/go.mod b/scraper/scraperhelper/xscraperhelper/go.mod index c79344bfce4..5d90cb6341a 100644 --- a/scraper/scraperhelper/xscraperhelper/go.mod +++ b/scraper/scraperhelper/xscraperhelper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/scraper/scraperhelper/xscraperhelper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/scraper/scrapertest/go.mod b/scraper/scrapertest/go.mod index 65e223b3cf7..c1c025eda67 100644 --- a/scraper/scrapertest/go.mod +++ b/scraper/scrapertest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/scraper/scrapertest -go 1.24.0 +go 1.25.0 require ( github.com/google/uuid v1.6.0 diff --git a/scraper/xscraper/go.mod b/scraper/xscraper/go.mod index c3e327c21a6..7e2787153ed 100644 --- a/scraper/xscraper/go.mod +++ b/scraper/xscraper/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/scraper/xscraper -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 diff --git a/service/go.mod b/service/go.mod index 56251403385..3e920d03308 100644 --- a/service/go.mod +++ b/service/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/service -go 1.24.0 +go 1.25.0 require ( github.com/google/uuid v1.6.0 diff --git a/service/hostcapabilities/go.mod b/service/hostcapabilities/go.mod index b49372ad153..cc3e1c3843b 100644 --- a/service/hostcapabilities/go.mod +++ b/service/hostcapabilities/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/service/hostcapabilities -go 1.24.0 +go 1.25.0 require ( go.opentelemetry.io/collector/component v1.51.0 diff --git a/service/telemetry/telemetrytest/go.mod b/service/telemetry/telemetrytest/go.mod index 1e111a6550a..2e6250d986e 100644 --- a/service/telemetry/telemetrytest/go.mod +++ b/service/telemetry/telemetrytest/go.mod @@ -1,6 +1,6 @@ module go.opentelemetry.io/collector/service/telemetry/telemetrytest -go 1.24.0 +go 1.25.0 require ( github.com/stretchr/testify v1.11.1 From b2a35c4e43fd8df4a02160a938fe590a827a5e76 Mon Sep 17 00:00:00 2001 From: alex boten <223565+codeboten@users.noreply.github.com> Date: Tue, 10 Feb 2026 15:35:40 -0800 Subject: [PATCH 2/8] changelog Signed-off-by: alex boten <223565+codeboten@users.noreply.github.com> --- .chloggen/codeboten_1.25.yaml | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 .chloggen/codeboten_1.25.yaml diff --git a/.chloggen/codeboten_1.25.yaml b/.chloggen/codeboten_1.25.yaml new file mode 100644 index 00000000000..3d47b55e7ff --- /dev/null +++ b/.chloggen/codeboten_1.25.yaml @@ -0,0 +1,27 @@ +# Use this changelog template to create an entry for release notes. + +# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix' +change_type: breaking + +# The name of the component, or a single word describing the area of concern, (e.g. receiver/filelog) +component: all + +# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). +note: Increase minimum Go version to 1.25 + +# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists. +issues: [14567] + +# (Optional) One or more lines of additional information to render under the primary note. +# These lines will be padded with 2 spaces and then inserted directly into the document. +# Use pipe (|) for multiline entries. +subtext: + +# If your change doesn't affect end users or the exported elements of any package, +# you should instead start your pull request title with [chore] or use the "Skip Changelog" label. +# Optional: The change log or logs in which this entry should be included. +# e.g. '[user]' or '[user, api]' +# Include 'user' if the change is relevant to end users. +# Include 'api' if there is a change to a library API. +# Default: '[user]' +change_logs: [] From 504aab19d8b9696f92f2006b6dba4f9ecd3d48e7 Mon Sep 17 00:00:00 2001 From: Pablo Baeyens Date: Wed, 11 Feb 2026 11:26:08 +0100 Subject: [PATCH 3/8] Fix lint issue --- confmap/resolver_test.go | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/confmap/resolver_test.go b/confmap/resolver_test.go index 9cd1ed0856e..808a7b5019f 100644 --- a/confmap/resolver_test.go +++ b/confmap/resolver_test.go @@ -398,14 +398,12 @@ func TestResolverShutdownClosesWatch(t *testing.T) { require.NoError(t, errN) var watcherWG sync.WaitGroup - watcherWG.Add(1) - go func() { + watcherWG.Go(func() { errW, ok := <-resolver.Watch() // Channel is closed, no exception assert.NoError(t, errW) assert.False(t, ok) - watcherWG.Done() - }() + }) require.NoError(t, resolver.Shutdown(context.Background())) watcherWG.Wait() @@ -536,12 +534,10 @@ func newRaceDetectorProvider() ProviderFactory { } func (p *provider) Retrieve(_ context.Context, _ string, watcher WatcherFunc) (*Retrieved, error) { - p.wg.Add(1) - go func() { + p.wg.Go(func() { // mock a config change event and wait for goroutine to return. - defer p.wg.Done() watcher(&ChangeEvent{}) - }() + }) return NewRetrieved(map[string]any{}) } From 33bc55c14fa66d6dc6e5ea5afbe678e63474f36a Mon Sep 17 00:00:00 2001 From: Pablo Baeyens Date: Wed, 11 Feb 2026 11:33:51 +0100 Subject: [PATCH 4/8] Attempt to fix gotestsum failure --- Makefile.Common | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Makefile.Common b/Makefile.Common index 26055eab6b5..51d22cdbf6a 100644 --- a/Makefile.Common +++ b/Makefile.Common @@ -27,6 +27,8 @@ JUNIT_OUT_DIR ?= $(TOOLS_MOD_DIR)/testresults test: # GODEBUG=fips140=only is used to surface any FIPS-140-3 non-compliant cryptographic # calls into the Go standard library. See: https://go.dev/doc/security/fips140#fips-140-3-mode + # Ensure gotestsum is built before running with FIPS mode to avoid download failures + @$(GO_TOOL) gotestsum --version >/dev/null 2>&1 || true GODEBUG=fips140=only $(GO_TOOL) gotestsum --packages="./..." -- $(GOTEST_OPT) .PHONY: test-with-cover From b399da34db256dcb0de30f2da0b130c4cc3d66ea Mon Sep 17 00:00:00 2001 From: Pablo Baeyens Date: Wed, 11 Feb 2026 12:42:14 +0100 Subject: [PATCH 5/8] More fixes --- confmap/resolver_test.go | 2 +- .../internal/queue/async_queue_test.go | 20 ++++----- .../internal/queue/memory_queue_test.go | 42 +++++++------------ .../internal/queue/persistent_queue_test.go | 18 +++----- .../internal/queuebatch/multi_batcher.go | 6 +-- .../internal/queuebatch/partition_batcher.go | 6 +-- .../internal/queuebatch/queue_batch_test.go | 6 +-- 7 files changed, 34 insertions(+), 66 deletions(-) diff --git a/confmap/resolver_test.go b/confmap/resolver_test.go index 808a7b5019f..dcf77ef1e18 100644 --- a/confmap/resolver_test.go +++ b/confmap/resolver_test.go @@ -401,7 +401,7 @@ func TestResolverShutdownClosesWatch(t *testing.T) { watcherWG.Go(func() { errW, ok := <-resolver.Watch() // Channel is closed, no exception - assert.NoError(t, errW) + require.NoError(t, errW) assert.False(t, ok) }) diff --git a/exporter/exporterhelper/internal/queue/async_queue_test.go b/exporter/exporterhelper/internal/queue/async_queue_test.go index 17d12b3db18..3d92da5ce69 100644 --- a/exporter/exporterhelper/internal/queue/async_queue_test.go +++ b/exporter/exporterhelper/internal/queue/async_queue_test.go @@ -46,13 +46,11 @@ func TestAsyncMemoryQueueBlocking(t *testing.T) { require.NoError(t, ac.Start(context.Background(), componenttest.NewNopHost())) wg := &sync.WaitGroup{} for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 100_000 { assert.NoError(t, ac.Offer(context.Background(), 10)) } - }() + }) } wg.Wait() require.NoError(t, ac.Shutdown(context.Background())) @@ -72,13 +70,11 @@ func TestAsyncMemoryWaitForResultQueueBlocking(t *testing.T) { require.NoError(t, ac.Start(context.Background(), componenttest.NewNopHost())) wg := &sync.WaitGroup{} for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 100_000 { assert.NoError(t, ac.Offer(context.Background(), 10)) } - }() + }) } wg.Wait() require.NoError(t, ac.Shutdown(context.Background())) @@ -98,14 +94,12 @@ func TestAsyncMemoryQueueBlockingCancelled(t *testing.T) { ctx, cancel := context.WithCancel(context.Background()) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10 { - assert.NoError(t, ac.Offer(ctx, 1)) + require.NoError(t, ac.Offer(ctx, 1)) } assert.ErrorIs(t, ac.Offer(ctx, 3), context.Canceled) - }() + }) // Sleep some time so that the go-routine blocks, it doesn't have to but even if it // does things will work. <-time.After(1 * time.Second) diff --git a/exporter/exporterhelper/internal/queue/memory_queue_test.go b/exporter/exporterhelper/internal/queue/memory_queue_test.go index 50004a157af..f9b9cdd006b 100644 --- a/exporter/exporterhelper/internal/queue/memory_queue_test.go +++ b/exporter/exporterhelper/internal/queue/memory_queue_test.go @@ -58,11 +58,9 @@ func TestMemoryQueueBlockingCancelled(t *testing.T) { require.NoError(t, q.Offer(context.Background(), 3)) ctx, cancel := context.WithCancel(context.Background()) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.ErrorIs(t, q.Offer(ctx, 3), context.Canceled) - }() + }) cancel() wg.Wait() assert.EqualValues(t, 3, q.Size()) @@ -139,14 +137,12 @@ func TestMemoryQueueWaitForResultPassErrorBack(t *testing.T) { set.WaitForResult = true q := newMemoryQueue[intRequest](set) require.NoError(t, q.Start(context.Background(), componenttest.NewNopHost())) - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { _, req, done, ok := q.Read(context.Background()) assert.True(t, ok) assert.EqualValues(t, 1, req) done.OnDone(myErr) - }() + }) require.ErrorIs(t, q.Offer(context.Background(), intRequest(1)), myErr) require.NoError(t, q.Shutdown(context.Background())) wg.Wait() @@ -161,22 +157,18 @@ func TestMemoryQueueWaitForResultCancelIncomingRequest(t *testing.T) { require.NoError(t, q.Start(context.Background(), componenttest.NewNopHost())) // Consume async new data. - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { _, _, done, ok := q.Read(context.Background()) assert.True(t, ok) <-stop done.OnDone(nil) - }() + }) ctx, cancel := context.WithCancel(context.Background()) - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { <-time.After(time.Second) cancel() - }() + }) require.ErrorIs(t, q.Offer(ctx, intRequest(1)), context.Canceled) close(stop) require.NoError(t, q.Shutdown(context.Background())) @@ -192,22 +184,18 @@ func TestMemoryQueueWaitForResultSizeAndCapacity(t *testing.T) { require.NoError(t, q.Start(context.Background(), componenttest.NewNopHost())) // Consume async new data. - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { _, _, done, ok := q.Read(context.Background()) assert.True(t, ok) <-stop done.OnDone(nil) - }() + }) assert.EqualValues(t, 0, q.Size()) assert.EqualValues(t, 100, q.Capacity()) - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, q.Offer(context.Background(), intRequest(1))) - }() + }) assert.Eventually(t, func() bool { return q.Size() == 1 }, 1*time.Second, 10*time.Millisecond) assert.EqualValues(t, 100, q.Capacity()) close(stop) @@ -224,9 +212,7 @@ func BenchmarkMemoryQueueWaitForResult(b *testing.B) { require.NoError(b, q.Start(context.Background(), componenttest.NewNopHost())) // Consume async new data. - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for { _, req, done, ok := q.Read(context.Background()) if !ok { @@ -235,7 +221,7 @@ func BenchmarkMemoryQueueWaitForResult(b *testing.B) { consumed.Add(int64(req)) done.OnDone(nil) } - }() + }) b.ReportAllocs() for b.Loop() { diff --git a/exporter/exporterhelper/internal/queue/persistent_queue_test.go b/exporter/exporterhelper/internal/queue/persistent_queue_test.go index ab12b0f2d5e..546f8724032 100644 --- a/exporter/exporterhelper/internal/queue/persistent_queue_test.go +++ b/exporter/exporterhelper/internal/queue/persistent_queue_test.go @@ -432,13 +432,11 @@ func TestPersistentBlockingQueue(t *testing.T) { td := intRequest(10) wg := &sync.WaitGroup{} for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 100_000 { assert.NoError(t, pq.Offer(context.Background(), td)) } - }() + }) } wg.Wait() // Because the persistent queue is not draining after Shutdown, need to wait here for the drain. @@ -710,9 +708,7 @@ func TestPersistentQueueStartWithNonDispatchedConcurrent(t *testing.T) { proWg := sync.WaitGroup{} // Sending small amount of data as windows test can't handle the test fast enough for range 5 { - proWg.Add(1) - go func() { - defer proWg.Done() + proWg.Go(func() { // Put in items up to capacity for range 10 { for { @@ -723,18 +719,16 @@ func TestPersistentQueueStartWithNonDispatchedConcurrent(t *testing.T) { time.Sleep(50 * time.Nanosecond) } } - }() + }) } conWg := sync.WaitGroup{} for range 5 { - conWg.Add(1) - go func() { - defer conWg.Done() + conWg.Go(func() { for range 10 { assert.True(t, consume(pq, func(context.Context, intRequest) error { return nil })) } - }() + }) } conDone := make(chan struct{}) diff --git a/exporter/exporterhelper/internal/queuebatch/multi_batcher.go b/exporter/exporterhelper/internal/queuebatch/multi_batcher.go index 6fb9eeb79e9..3177bff2147 100644 --- a/exporter/exporterhelper/internal/queuebatch/multi_batcher.go +++ b/exporter/exporterhelper/internal/queuebatch/multi_batcher.go @@ -75,11 +75,9 @@ func (mb *multiBatcher) Consume(ctx context.Context, req request.Request, done q func (mb *multiBatcher) Shutdown(ctx context.Context) error { var wg sync.WaitGroup mb.shards.Range(func(_, shard any) bool { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { _ = shard.(*partitionBatcher).Shutdown(ctx) - }() + }) return true }) wg.Wait() diff --git a/exporter/exporterhelper/internal/queuebatch/partition_batcher.go b/exporter/exporterhelper/internal/queuebatch/partition_batcher.go index 895d5a53c82..d58e0d0830d 100644 --- a/exporter/exporterhelper/internal/queuebatch/partition_batcher.go +++ b/exporter/exporterhelper/internal/queuebatch/partition_batcher.go @@ -198,9 +198,7 @@ func (qb *partitionBatcher) Start(context.Context, component.Host) error { return nil } qb.timer = time.NewTimer(qb.cfg.FlushTimeout) - qb.stopWG.Add(1) - go func() { - defer qb.stopWG.Done() + qb.stopWG.Go(func() { for { select { case <-qb.shutdownCh: @@ -209,7 +207,7 @@ func (qb *partitionBatcher) Start(context.Context, component.Host) error { qb.flushCurrentBatchIfNecessary() } } - }() + }) return nil } diff --git a/exporter/exporterhelper/internal/queuebatch/queue_batch_test.go b/exporter/exporterhelper/internal/queuebatch/queue_batch_test.go index 2ec8914fb65..46a344d7155 100644 --- a/exporter/exporterhelper/internal/queuebatch/queue_batch_test.go +++ b/exporter/exporterhelper/internal/queuebatch/queue_batch_test.go @@ -486,11 +486,9 @@ func TestQueueBatch_BatchBlocking(t *testing.T) { // send 6 blockOnOverflow requests wg := sync.WaitGroup{} for range 6 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, qb.Send(context.Background(), &requesttest.FakeRequest{Items: 1, Delay: 10 * time.Millisecond})) - }() + }) } wg.Wait() From 031d5fae46c08e3d9b86e3a08c51969d360d0687 Mon Sep 17 00:00:00 2001 From: Pablo Baeyens Date: Wed, 11 Feb 2026 14:49:30 +0100 Subject: [PATCH 6/8] Even more fixes --- .../exporterhelper/internal/queue/async_queue.go | 6 ++---- internal/memorylimiter/memorylimiter.go | 7 ++----- processor/batchprocessor/batch_processor.go | 5 +---- receiver/otlpreceiver/otlp.go | 14 ++++---------- receiver/receivertest/contract_checker.go | 6 ++---- .../internal/controller/controller.go | 6 ++---- 6 files changed, 13 insertions(+), 31 deletions(-) diff --git a/exporter/exporterhelper/internal/queue/async_queue.go b/exporter/exporterhelper/internal/queue/async_queue.go index 99304dd3774..d5e8a1618e2 100644 --- a/exporter/exporterhelper/internal/queue/async_queue.go +++ b/exporter/exporterhelper/internal/queue/async_queue.go @@ -36,11 +36,9 @@ func (qc *asyncQueue[T]) Start(ctx context.Context, host component.Host) error { } var startWG sync.WaitGroup for i := 0; i < qc.numConsumers; i++ { - qc.stopWG.Add(1) startWG.Add(1) - go func() { //nolint:contextcheck + qc.stopWG.Go(func() { //nolint:contextcheck startWG.Done() - defer qc.stopWG.Done() for { ctx, req, done, ok := qc.Read(context.Background()) if !ok { @@ -51,7 +49,7 @@ func (qc *asyncQueue[T]) Start(ctx context.Context, host component.Host) error { qc.refCounter.Unref(req) } } - }() + }) } startWG.Wait() diff --git a/internal/memorylimiter/memorylimiter.go b/internal/memorylimiter/memorylimiter.go index 787f4708c23..f10e3360fba 100644 --- a/internal/memorylimiter/memorylimiter.go +++ b/internal/memorylimiter/memorylimiter.go @@ -94,10 +94,7 @@ func (ml *MemoryLimiter) Start(_ context.Context, _ component.Host) error { ml.refCounter++ if ml.refCounter == 1 { ml.closed = make(chan struct{}) - ml.waitGroup.Add(1) - go func() { - defer ml.waitGroup.Done() - + ml.waitGroup.Go(func() { for { select { case <-ml.ticker.C: @@ -106,7 +103,7 @@ func (ml *MemoryLimiter) Start(_ context.Context, _ component.Host) error { } ml.CheckMemLimits() } - }() + }) } return nil } diff --git a/processor/batchprocessor/batch_processor.go b/processor/batchprocessor/batch_processor.go index 0c43fb6f17b..0bc757a0a20 100644 --- a/processor/batchprocessor/batch_processor.go +++ b/processor/batchprocessor/batch_processor.go @@ -183,13 +183,10 @@ func (bp *batchProcessor[T]) Shutdown(context.Context) error { } func (b *shard[T]) start() { - b.processor.goroutines.Add(1) - go b.startLoop() + b.processor.goroutines.Go(b.startLoop) } func (b *shard[T]) startLoop() { - defer b.processor.goroutines.Done() - // timerCh ensures we only block when there is a // timer, since <- from a nil channel is blocking. var timerCh <-chan time.Time diff --git a/receiver/otlpreceiver/otlp.go b/receiver/otlpreceiver/otlp.go index 7481b5ec002..5b35b7576c8 100644 --- a/receiver/otlpreceiver/otlp.go +++ b/receiver/otlpreceiver/otlp.go @@ -119,14 +119,11 @@ func (r *otlpReceiver) startGRPCServer(ctx context.Context, host component.Host) } r.settings.Logger.Info("Starting GRPC server", zap.String("endpoint", gln.Addr().String())) - r.shutdownWG.Add(1) - go func() { - defer r.shutdownWG.Done() - + r.shutdownWG.Go(func() { if errGrpc := r.serverGRPC.Serve(gln); errGrpc != nil && !errors.Is(errGrpc, grpc.ErrServerStopped) { componentstatus.ReportStatus(host, componentstatus.NewFatalErrorEvent(errGrpc)) } - }() + }) return nil } @@ -177,14 +174,11 @@ func (r *otlpReceiver) startHTTPServer(ctx context.Context, host component.Host) } r.settings.Logger.Info("Starting HTTP server", zap.String("endpoint", hln.Addr().String())) - r.shutdownWG.Add(1) - go func() { - defer r.shutdownWG.Done() - + r.shutdownWG.Go(func() { if errHTTP := r.serverHTTP.Serve(hln); errHTTP != nil && !errors.Is(errHTTP, http.ErrServerClosed) { componentstatus.ReportStatus(host, componentstatus.NewFatalErrorEvent(errHTTP)) } - }() + }) return nil } diff --git a/receiver/receivertest/contract_checker.go b/receiver/receivertest/contract_checker.go index 048130f73c8..58c70c43672 100644 --- a/receiver/receivertest/contract_checker.go +++ b/receiver/receivertest/contract_checker.go @@ -146,9 +146,7 @@ func checkConsumeContractScenario(params CheckConsumeContractParams, decisionFun // The total number of generator calls will be equal to params.GenerateCount. for range concurrency { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for atomic.AddInt64(&generatedIndex, 1) <= int64(params.GenerateCount) { ids := params.Generator.Generate() require.NotEmpty(params.T, ids) @@ -161,7 +159,7 @@ func checkConsumeContractScenario(params CheckConsumeContractParams, decisionFun // generated data set. require.Empty(params.T, duplicates) } - }() + }) } // Wait until all generator goroutines are done. diff --git a/scraper/scraperhelper/internal/controller/controller.go b/scraper/scraperhelper/internal/controller/controller.go index 60f37cc9e2f..f9cc6175707 100644 --- a/scraper/scraperhelper/internal/controller/controller.go +++ b/scraper/scraperhelper/internal/controller/controller.go @@ -92,9 +92,7 @@ func (sc *Controller[T]) Shutdown(ctx context.Context) error { // startScraping initiates a ticker that calls Scrape based on the configured // collection interval. func (sc *Controller[T]) startScraping() { - sc.wg.Add(1) - go func() { - defer sc.wg.Done() + sc.wg.Go(func() { if sc.initialDelay > 0 { select { case <-time.After(sc.initialDelay): @@ -121,7 +119,7 @@ func (sc *Controller[T]) startScraping() { return } } - }() + }) } func GetSettings(sType component.Type, rSet receiver.Settings) scraper.Settings { From 49f569f51bb6c49c13a578c11fea43dc1529123b Mon Sep 17 00:00:00 2001 From: alex boten <223565+codeboten@users.noreply.github.com> Date: Wed, 11 Feb 2026 07:55:12 -0800 Subject: [PATCH 7/8] modernize Signed-off-by: alex boten <223565+codeboten@users.noreply.github.com> --- otelcol/collector_test.go | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/otelcol/collector_test.go b/otelcol/collector_test.go index eb4a381313b..d06672e3302 100644 --- a/otelcol/collector_test.go +++ b/otelcol/collector_test.go @@ -328,11 +328,9 @@ func TestCollectorFailedShutdown(t *testing.T) { require.NoError(t, err) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.EqualError(t, col.Run(context.Background()), "failed to shutdown collector telemetry: err1") - }() + }) assert.Eventually(t, func() bool { return StateRunning == col.GetState() @@ -582,11 +580,9 @@ func TestCollectorDryRun(t *testing.T) { func startCollector(ctx context.Context, t *testing.T, col *Collector) *sync.WaitGroup { wg := &sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, col.Run(ctx)) - }() + }) return wg } From d51e2c84135629fe376611f4841dae47f4eac00e Mon Sep 17 00:00:00 2001 From: alex boten <223565+codeboten@users.noreply.github.com> Date: Wed, 11 Feb 2026 08:07:08 -0800 Subject: [PATCH 8/8] modernize Signed-off-by: alex boten <223565+codeboten@users.noreply.github.com> --- pdata/plog/plogotlp/grpc_test.go | 12 ++++-------- pdata/pmetric/pmetricotlp/grpc_test.go | 12 ++++-------- pdata/pprofile/pprofileotlp/grpc_test.go | 12 ++++-------- pdata/ptrace/ptraceotlp/grpc_test.go | 12 ++++-------- processor/processorhelper/logs_test.go | 6 ++---- processor/processorhelper/metrics_test.go | 6 ++---- processor/processorhelper/traces_test.go | 6 ++---- .../xprocessorhelper/profiles_test.go | 6 ++---- scraper/logs_test.go | 6 ++---- scraper/metrics_test.go | 6 ++---- scraper/xscraper/profiles_test.go | 6 ++---- 11 files changed, 30 insertions(+), 60 deletions(-) diff --git a/pdata/plog/plogotlp/grpc_test.go b/pdata/plog/plogotlp/grpc_test.go index 7700888267c..16fe78f5ff7 100644 --- a/pdata/plog/plogotlp/grpc_test.go +++ b/pdata/plog/plogotlp/grpc_test.go @@ -27,11 +27,9 @@ func TestGrpc(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeLogsServer{t: t}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() @@ -60,11 +58,9 @@ func TestGrpcError(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeLogsServer{t: t, err: errors.New("my error")}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() diff --git a/pdata/pmetric/pmetricotlp/grpc_test.go b/pdata/pmetric/pmetricotlp/grpc_test.go index f31c5b1cf89..11321e4782c 100644 --- a/pdata/pmetric/pmetricotlp/grpc_test.go +++ b/pdata/pmetric/pmetricotlp/grpc_test.go @@ -27,11 +27,9 @@ func TestGrpc(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeMetricsServer{t: t}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() @@ -60,11 +58,9 @@ func TestGrpcError(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeMetricsServer{t: t, err: errors.New("my error")}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() diff --git a/pdata/pprofile/pprofileotlp/grpc_test.go b/pdata/pprofile/pprofileotlp/grpc_test.go index fb09ba2af7f..4a2bd3767b2 100644 --- a/pdata/pprofile/pprofileotlp/grpc_test.go +++ b/pdata/pprofile/pprofileotlp/grpc_test.go @@ -27,11 +27,9 @@ func TestGrpc(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeProfilesServer{t: t}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() @@ -60,11 +58,9 @@ func TestGrpcError(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeProfilesServer{t: t, err: errors.New("my error")}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() diff --git a/pdata/ptrace/ptraceotlp/grpc_test.go b/pdata/ptrace/ptraceotlp/grpc_test.go index 2172245de61..7c7b8addfbb 100644 --- a/pdata/ptrace/ptraceotlp/grpc_test.go +++ b/pdata/ptrace/ptraceotlp/grpc_test.go @@ -27,11 +27,9 @@ func TestGrpc(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeTracesServer{t: t}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() @@ -60,11 +58,9 @@ func TestGrpcError(t *testing.T) { s := grpc.NewServer() RegisterGRPCServer(s, &fakeTracesServer{t: t, err: errors.New("my error")}) wg := sync.WaitGroup{} - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { assert.NoError(t, s.Serve(lis)) - }() + }) t.Cleanup(func() { s.Stop() wg.Wait() diff --git a/processor/processorhelper/logs_test.go b/processor/processorhelper/logs_test.go index 82dbfd666a4..8ca9f24369a 100644 --- a/processor/processorhelper/logs_test.go +++ b/processor/processorhelper/logs_test.go @@ -93,13 +93,11 @@ func TestLogsConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { assert.NoError(t, lp.ConsumeLogs(context.Background(), incomingLogs)) } - }() + }) } wg.Wait() assert.NoError(t, lp.Shutdown(context.Background())) diff --git a/processor/processorhelper/metrics_test.go b/processor/processorhelper/metrics_test.go index 19d018d6291..2da25a537ae 100644 --- a/processor/processorhelper/metrics_test.go +++ b/processor/processorhelper/metrics_test.go @@ -91,13 +91,11 @@ func TestMetricsConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { assert.NoError(t, mp.ConsumeMetrics(context.Background(), incomingMetrics)) } - }() + }) } wg.Wait() assert.NoError(t, mp.Shutdown(context.Background())) diff --git a/processor/processorhelper/traces_test.go b/processor/processorhelper/traces_test.go index 8ad59e54f2d..6b0f6f8785e 100644 --- a/processor/processorhelper/traces_test.go +++ b/processor/processorhelper/traces_test.go @@ -93,13 +93,11 @@ func TestTracesConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { assert.NoError(t, mp.ConsumeTraces(context.Background(), incomingTraces)) } - }() + }) } wg.Wait() assert.NoError(t, mp.Shutdown(context.Background())) diff --git a/processor/processorhelper/xprocessorhelper/profiles_test.go b/processor/processorhelper/xprocessorhelper/profiles_test.go index c4cc53b8a7f..d30a018d439 100644 --- a/processor/processorhelper/xprocessorhelper/profiles_test.go +++ b/processor/processorhelper/xprocessorhelper/profiles_test.go @@ -89,13 +89,11 @@ func TestProfilesConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { assert.NoError(t, pp.ConsumeProfiles(context.Background(), incomingProfiles)) } - }() + }) } wg.Wait() assert.NoError(t, pp.Shutdown(context.Background())) diff --git a/scraper/logs_test.go b/scraper/logs_test.go index dc46e522c71..4fa04ca29b8 100644 --- a/scraper/logs_test.go +++ b/scraper/logs_test.go @@ -59,14 +59,12 @@ func TestLogsConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { _, errScrape := mp.ScrapeLogs(context.Background()) assert.NoError(t, errScrape) } - }() + }) } wg.Wait() require.NoError(t, mp.Shutdown(context.Background())) diff --git a/scraper/metrics_test.go b/scraper/metrics_test.go index bc9f9c52dd0..daf97401a52 100644 --- a/scraper/metrics_test.go +++ b/scraper/metrics_test.go @@ -66,14 +66,12 @@ func TestMetricsConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { _, errScrape := mp.ScrapeMetrics(context.Background()) assert.NoError(t, errScrape) } - }() + }) } wg.Wait() require.NoError(t, mp.Shutdown(context.Background())) diff --git a/scraper/xscraper/profiles_test.go b/scraper/xscraper/profiles_test.go index ac98158ad86..b97e978fa34 100644 --- a/scraper/xscraper/profiles_test.go +++ b/scraper/xscraper/profiles_test.go @@ -59,14 +59,12 @@ func TestProfilesConcurrency(t *testing.T) { var wg sync.WaitGroup for range 10 { - wg.Add(1) - go func() { - defer wg.Done() + wg.Go(func() { for range 10000 { _, errScrape := mp.ScrapeProfiles(context.Background()) assert.NoError(t, errScrape) } - }() + }) } wg.Wait() require.NoError(t, mp.Shutdown(context.Background()))