chore: update OTel Collector libraries to v1.41.0/v0.135.0#46464
Conversation
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
|
This pull request is now in conflicts. Could you fix it? 🙏 |
Resolves dependency conflict on github.com/cilium/ebpf.
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
|
Will approve once we stabilize CI again, this failure is appearing in a few places. |
|
This pull request is now in conflicts. Could you fix it? 🙏 |
The property `flush_timeout` has been required since v0.134.0. I believe this is a result of fixing config validation in open-telemetry/opentelemetry-collector#13611.
|
This pull request is now in conflicts. Could you fix it? 🙏 |
@cmacknz can you take a look now that the CI is green? |
This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged.
This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged.
* feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com>
* feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com>
|
Any reason we are not backporting this to 8.19? I believe we wanted to keep the code as similar as possible between main and 8.19 |
* chore: update OTel Collector libraries to `v1.41.0`/`v0.135.0` * fix(auditbeat): update ebpfevents to `v0.8.0` Resolves dependency conflict on github.com/cilium/ebpf. * make notice * test: fix TestFilebeatOTelE2E integration test The property `flush_timeout` has been required since v0.134.0. I believe this is a result of fixing config validation in open-telemetry/opentelemetry-collector#13611. * test: fix unit tests (cherry picked from commit 16c4d9a) # Conflicts: # NOTICE.txt # go.mod # go.sum # x-pack/filebeat/tests/integration/otel_test.go # x-pack/metricbeat/tests/integration/otel_test.go
* chore: update OTel Collector libraries to `v1.41.0`/`v0.135.0` * fix(auditbeat): update ebpfevents to `v0.8.0` Resolves dependency conflict on github.com/cilium/ebpf. * make notice * test: fix TestFilebeatOTelE2E integration test The property `flush_timeout` has been required since v0.134.0. I believe this is a result of fixing config validation in open-telemetry/opentelemetry-collector#13611. * test: fix unit tests (cherry picked from commit 16c4d9a) # Conflicts: # NOTICE.txt # go.mod # go.sum
* feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com> (cherry picked from commit ecb0e06) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md
* feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com> (cherry picked from commit ecb0e06) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md # internal/pkg/otel/manager/manager_test.go
* feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com> (cherry picked from commit ecb0e06) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md
* feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com> (cherry picked from commit ecb0e06) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md
…35.0 (#10849) * feat: Update OTel Collector components to v0.135.0 (#9858) * feat: Update OTel Collector components to v0.135.0 * chore: update Elastic OTel components go get github.com/elastic/opentelemetry-collector-components/connector/elasticapmconnector go get github.com/elastic/opentelemetry-collector-components/extension/apikeyauthextension go get github.com/elastic/opentelemetry-collector-components/extension/apmconfigextension go get github.com/elastic/opentelemetry-collector-components/processor/elasticinframetricsprocessor github.com/elastic/opentelemetry-collector-components/processor/elastictraceprocessor go get github.com/elastic/opentelemetry-collector-components/receiver/elasticapmintakereceiver * test: fix test after upstream change Logging was changed in open-telemetry/opentelemetry-collector#13606 * test: update gotestsum to latest Not sure why it got downgraded to v1.7.0 * use Beats from branch to verify CI This commit must be reverted before merging this PR. It temporarily replaces Beats dependency with elastic/beats#46464 to see if CI is happy with it, and to fix any remaining failures without waiting for that PR to be merged. * test: fix unit test * revert "use Beats from branch to verify CI" This reverts 4813ce3. * update Beats to 16c4d9a1aa8c This Beats commit has the update to OTel v0.135.0. * fix collector's metric names Without this replacement, some of the collector's internal metric names are broken. This has been broken since: - v9.1.3 in v9.1.x - v8.19.3 in v8.19.x See open-telemetry/opentelemetry-go#7039. See https://cloud-native.slack.com/archives/C01N6P7KR6W/p1757442690145329. * update go-mod-replace allowlist * mage notice --------- Co-authored-by: Christos Markou <chrismarkou92@gmail.com> (cherry picked from commit ecb0e06) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md * Fix conflicts * Fix unit tests * Fix an api change in mockes * Revert "Fix unit tests" This reverts commit 81ea886. * Bump beats dependency and fix tests * mage notice --------- Co-authored-by: Andrzej Stencel <andrzej.stencel@elastic.co> Co-authored-by: Mikołaj Świątek <mail@mikolajswiatek.com>
Proposed commit message
Updates the OpenTelemetry Collector dependencies to the current latest versions v1.41.0/v0.135.0. See changelogs since
v1.41.0/v0.135.0for Core and Contrib.Why is it important?
We want to be on the latest version to keep up with latest features and bug fixes.
Checklist
[ ] My code follows the style guidelines of this project[ ] I have commented my code, particularly in hard-to-understand areas[ ] I have made corresponding changes to the documentation[ ] I have made corresponding change to the default configuration files[ ] I have added tests that prove my fix is effective or that my feature works[ ] I have added an entry inCHANGELOG.next.asciidocorCHANGELOG-developer.next.asciidoc.How to test this PR locally