forked from open-telemetry/opentelemetry-collector-contrib
-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
handling http semantic convention breaking changes in v1.23.0 #172
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…tAsDefaultHost` feature gate (open-telemetry#30985) Description: awsproxy extension adopts `component.UseLocalHostAsDefaultHost` feature gate Link to tracking Issue: open-telemetry#30702 Fixes open-telemetry#30984 Documentation: Updated docs. Co-authored-by: Pablo Baeyens <[email protected]>
**Description:** Similar to open-telemetry#30917 and open-telemetry#30925, the ID was empty before. **Link to tracking Issue:** This is needed to make contrib tests pass on open-telemetry/opentelemetry-collector#9414
…Ds (open-telemetry#30994) **Description:** - Makes the test on awsxray exporter use the `exportertest.NewNopCreateSettings().ID`. This is because this is the ID passed to `newTracesExporter`: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/2f5b0bc52cd160886b4e1ee03461704d6c440a8d/exporter/awsxrayexporter/awsxray_test.go#L109 Currently, this makes no difference, but it will once the ID on NewNopCreateSettings is fixed. - Also fixes the hostmetrics receiver ID, just for fun :) **Link to tracking Issue:** Needed for open-telemetry/opentelemetry-collector/pull/9414
…etry#29299) **Description:** This commit addresses the need to accept logs from Google Cloud Platform into an OpenTelemetry pipeline. The goal is to introduce the `cloud_logging` encoding option for the receiver, allowing users to seamlessly integrate Cloud Logging logs into their OpenTelemetry pipeline. **Link to tracking Issue:** open-telemetry#23184 **Testing:** Added tests for parsing LogEntries **Documentation:** Added the option to the README.md Co-authored-by: Kamal Al Marhubi <[email protected]>
…ectly (open-telemetry#30225) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> This PR sets the AppVersion property for the default fallthrough case if it exists on any spans in the request. This allows it to properly propagate through to Datadog. Currently, if using a library such as datadog/dd-trace-dotnet , the datadog service version property (dd_version) does not propagate through to datadog at all. The bug that this fixes is that the Datadog version resource attribute is currently not flowing through. **Link to tracking Issue:** <Issue number if applicable> Closes open-telemetry#30526 **Testing:** <Describe what testing was performed and which tests were added.> Local testing using delve to confirm expected behavior, added existing version tag to tests. Deployed patch internally on OpenTelemetry collector and observed desired behavior. **Documentation:** <Describe the documentation added.>
**Description:** godoc comments added to non-commented public interfaces in [pkg/ottl] packages **Link to tracking Issue:** open-telemetry#28893 **Testing:** <Describe what testing was performed and which tests were added.> **Documentation:** <Describe the documentation added.> --------- Co-authored-by: Evan Bradley <[email protected]> Co-authored-by: Curtis Robert <[email protected]> Co-authored-by: Tyler Helmuth <[email protected]>
…etry#30506) Relates to open-telemetry#27849 --------- Co-authored-by: Antoine Toulme <[email protected]> Co-authored-by: Tyler Helmuth <[email protected]>
**Description:** - Parallelize the integration tests like the unit/lint job. - Add more groups to spread out the load **Link to tracking Issue:** <Issue number if applicable> Related to open-telemetry#30880 **Testing:** <Describe what testing was performed and which tests were added.> **Documentation:** <Describe the documentation added.>
…st feature gate (open-telemetry#30992) **Description:** Fixes open-telemetry#30818, see open-telemetry#30818 (comment) for historical context. I think we should still do it. Also reorders the entries in the changelog to be in alphabetical order
…elemetry#30424) Relates to open-telemetry#27849 I have removed code blocking the creation of the extension as Config.Validate does the same checks. --------- Co-authored-by: Tyler Helmuth <[email protected]>
open-telemetry#30930) This PR is just a minor cleanup step broken out from open-telemetry#30784
…ry#30972) **Description:** add generated lifecycle tests to zipkinexporter **Link to tracking Issue:** Relates to open-telemetry#27849 Co-authored-by: Tyler Helmuth <[email protected]>
**Description:** This PR serves as a refactor for core logic of the connector, to further decouple the failover component from the pipeline_selector which will now be the exclusive component to deal with managing the index corresponding to the target consumers. **Testing:** Modified existing tests to align with changes.
Fixes open-telemetry#26649 Fixes open-telemetry#27803 --------- Co-authored-by: Curtis Robert <[email protected]>
…ure gate (open-telemetry#30987) Description: internal/aws/proxy adopts `component.UseLocalHostAsDefaultHost` feature gate Link to tracking Issue: open-telemetry#30702 Fixes open-telemetry#30986 Documentation: Updated docs. --------- Co-authored-by: Pablo Baeyens <[email protected]> Co-authored-by: Pablo Baeyens <[email protected]>
…pen-telemetry#31006) Do not use time.Sleep in tests. Use require.Eventually instead or channels.
**Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> This PR adds documentation notes on how to achieve fault tolerance on `filelog`'s receiver offset tracking. The need for this is obvious but was also explained at open-telemetry#20552 (comment). **Link to tracking Issue:** <Issue number if applicable> **Testing:** <Describe what testing was performed and which tests were added.> **Documentation:** <Describe the documentation added.> --------- Signed-off-by: ChrsMark <[email protected]>
Reorganizes job dependencies and parallelizes the k8s tests Related to open-telemetry#30880 **Testing** https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/7745609364
Fixes FailoverRecovery test issue from open-telemetry#31006 for both metrics and logs as well.
…-telemetry#30768) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> This change enables `goleak` checks for the [Azure Data Explorer Exporter](https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/azuredataexplorerexporter). This is a test only update, adding some `Close` calls that were missing and causing leaks. **Link to tracking Issue:** <Issue number if applicable> open-telemetry#30438 **Testing:** <Describe what testing was performed and which tests were added.> All tests are passing, including added goleak check.
…telemetry#30764) Fixes open-telemetry#30763 --------- Signed-off-by: Alex Boten <[email protected]>
…22 to 1.24 (open-telemetry#30927) **Description:** Docker Client API default has been upgraded from 1.22 to 1.24. The details of the requirements to this change has been given in the accompanying issue with docker upstream PRs. **Link to tracking Issue:** Fixes open-telemetry#30900 **Testing:** Updated unit tests accordingly and ran integration tests to make sure everything works as it should.
This is no longer needed in many places as census isn't loaded anymore Signed-off-by: Alex Boten <[email protected]>
…pen-telemetry#30709) **Description:** As noted in the related issues, a lot of the logic in the sqlquery receiver can be moved to a central package to be used by other receivers. I realize this appears to be a large change, but it's purely a re-organization PR, there's no functional changes. GitHub is not recognizing moved files, so please refer to new file names and their deleted counterparts. A lot of members are now public in the internal package that were private in the receiver which is likely what's causing GitHub to miss that they're just moved files. This doesn't require a changelog (in my mind) because all things moving were originally private to the receiver. They're now still purely internal, so they won't be published. I believe `internal` is the best destination for now as it allows us to solidify the usage and interface before moving it to `pkg` to be publicly available. There are still a lot of unknowns as far as how it can be used by other receivers, so I fully expect "breaking changes" to this. If we keep it internal we can update all usages with the changing interface, so the changes won't be breaking. **Link to tracking Issue:** open-telemetry#30297, open-telemetry#13546
…n-telemetry#30475) Relates to open-telemetry#27849 --------- Signed-off-by: Jared Tan <[email protected]> Co-authored-by: Antoine Toulme <[email protected]>
== This is continuation of open-telemetry#19341 == **Description:** Adding AWS SecretsManager Support. To be able to inject secrets via AWS Lambda Layer **Link to tracking Issue:** [19368](open-telemetry#19368) **Testing:** Test fetching via SecretsManager emulating AWS API **Documentation:** README.md --------- Co-authored-by: Luís Duarte <[email protected]>
…emetry#31302) **Description:** <Describe what has changed.> * Follow up to open-telemetry#31081 to refactor using shared code between Stanza and OTTL for parsing CSV. **Testing:** * Existing unit tests cover this refactor
…telemetry#31301) **Description:** We are looking to deprecate component.ErrNilNextConsumer and have pipelines check it rather than set it the expectation on every component that the next component may be nil. See open-telemetry/opentelemetry-collector#9526 for context.
…ion (open-telemetry#30981) **Description:** Deprecate the httpforwarder extension Introduce the httpforwarderextension extension. **Link to tracking Issue:** Relates to open-telemetry#24171
**Description:** We are looking to deprecate component.ErrNilNextConsumer and have pipelines check it rather than set it the expectation on every component that the next component may be nil. See open-telemetry/opentelemetry-collector#9526 for context.
…n-telemetry#31283) This clarifies potential confusion about how to access metadata using the `from_context` configuration argument. A few months ago I submitted open-telemetry#27465 to remove an obsolete warning, but then a user [asked](open-telemetry@d50e094#commitcomment-131957576) about why their `from_context` isn't working properly with a batch processor. I agree that it's not obvious, so I hope this PR clarifies it enough. I did not try running the update example, so I hope it works :) cc @jpkrohling
…pen-telemetry#31293) **Description:** Release unused resources after some time This PR is: * upgrading the used library from v0.17.0 to v0.18.0 * introducing new configuration option - `buffer.purge_older_than` **Link to tracking Issue:** open-telemetry#31292 **Testing:** Issue has been in the underlying library - scalyr/dataset-go#75 - where I have fixed the issue. ![Screenshot 2024-02-16 at 12 01 40](https://github.com/open-telemetry/opentelemetry-collector-contrib/assets/122797378/f1c80807-69de-49c4-aa62-7edd211e3b34) **Documentation:** I have added documentation to the newly added configuration option - `buffer.purge_older_than`. Fixes open-telemetry#31292
This fixes github issue templates, broken because we have a duplicate in the dropdowns right now. Fixes open-telemetry#31306
Adds a new internal, _experimental_ package `metrics/identity` which implements identity types for resource, scope, metric and stream. This is closely related to work being done in open-telemetry#30707 and open-telemetry#30827. The package is specifically experimental, as it shall be treated as an internal component to above processors which may change at any moment as long as those are under active initial development. /cc @jpkrohling @djaglowski @RichieSams
**Description:** Add @cparkins to codeowners of azure event hub receiver. **Link to tracking Issue:** Fixes open-telemetry#31269
… [chore] (open-telemetry#31271) **Description:** amend the loadbalancingexporter README to explain how topology changes can influence decisions around retry configuration, and how they can result in data loss. <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> **Link to tracking Issue:** 31209 **Testing:** N/A **Documentation:** README.ms
Prepare for v0.95.0 --------- Signed-off-by: Juraci Paixão Kröhling <[email protected]>
The following commands were run to prepare this release: - make chlog-update VERSION=v0.95.0 - sed -i.bak s/0.94.0/0.95.0/g versions.yaml - make multimod-prerelease - make multimod-sync
Resolves open-telemetry#30836 Also removes a dependency on `observIQ/nanojack` by simplifying rotation tests. We still test both move/create and copy/truncate strategies, but the unit tests are managing rotated files manually. We are no longer testing rotation based on different naming strategies since this is effectively just a matching problem, which is well tested elsewhere.
**Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Fixes flaky failover recovery unit test bug and expands on failover recovery test cases to test for more complicated failover scenarios. **Link to tracking Issue:** Resolves open-telemetry#31005 **Testing:** <Describe what testing was performed and which tests were added.> Additional tests created for traces, metrics, logs. Tests added for pipeline_selector component @djaglowski Is there anything specific we want to do before including the component in the distro and/or putting it in alpha? **Documentation:** <Describe the documentation added.>
This PR removes the sample testing config that was added accidentally with open-telemetry#30914. cc: @djaglowski Signed-off-by: ChrsMark <[email protected]>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [k8s.io/api](https://togithub.com/kubernetes/api) | `v0.29.1` -> `v0.29.2` | [![age](https://developer.mend.io/api/mc/badges/age/go/k8s.io%2fapi/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/k8s.io%2fapi/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/k8s.io%2fapi/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/k8s.io%2fapi/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | | [k8s.io/apimachinery](https://togithub.com/kubernetes/apimachinery) | `v0.29.1` -> `v0.29.2` | [![age](https://developer.mend.io/api/mc/badges/age/go/k8s.io%2fapimachinery/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/k8s.io%2fapimachinery/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/k8s.io%2fapimachinery/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/k8s.io%2fapimachinery/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | | [k8s.io/client-go](https://togithub.com/kubernetes/client-go) | `v0.29.1` -> `v0.29.2` | [![age](https://developer.mend.io/api/mc/badges/age/go/k8s.io%2fclient-go/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/k8s.io%2fclient-go/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/k8s.io%2fclient-go/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/k8s.io%2fclient-go/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | | [k8s.io/kubelet](https://togithub.com/kubernetes/kubelet) | `v0.29.1` -> `v0.29.2` | [![age](https://developer.mend.io/api/mc/badges/age/go/k8s.io%2fkubelet/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/k8s.io%2fkubelet/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/k8s.io%2fkubelet/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/k8s.io%2fkubelet/v0.29.1/v0.29.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>kubernetes/api (k8s.io/api)</summary> ### [`v0.29.2`](https://togithub.com/kubernetes/api/compare/v0.29.1...v0.29.2) [Compare Source](https://togithub.com/kubernetes/api/compare/v0.29.1...v0.29.2) </details> <details> <summary>kubernetes/apimachinery (k8s.io/apimachinery)</summary> ### [`v0.29.2`](https://togithub.com/kubernetes/apimachinery/compare/v0.29.1...v0.29.2) [Compare Source](https://togithub.com/kubernetes/apimachinery/compare/v0.29.1...v0.29.2) </details> <details> <summary>kubernetes/client-go (k8s.io/client-go)</summary> ### [`v0.29.2`](https://togithub.com/kubernetes/client-go/compare/v0.29.1...v0.29.2) [Compare Source](https://togithub.com/kubernetes/client-go/compare/v0.29.1...v0.29.2) </details> <details> <summary>kubernetes/kubelet (k8s.io/kubelet)</summary> ### [`v0.29.2`](https://togithub.com/kubernetes/kubelet/compare/v0.29.1...v0.29.2) [Compare Source](https://togithub.com/kubernetes/kubelet/compare/v0.29.1...v0.29.2) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMDAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> Co-authored-by: Tyler Helmuth <[email protected]> Co-authored-by: Ziqi Zhao <[email protected]>
open-telemetry#31280) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> This PR adds some clarifications in the documentation of the `serviceAccount` auth type of the `kubeletstats` receiver. This is an outcome of the discussions at open-telemetry/opentelemetry-helm-charts#1034 (comment). Signed-off-by: ChrsMark <[email protected]>
…onment (open-telemetry#31250) **Description:** Adding TLS configuration to telemetrygen made it so that providing root CA information is mandatory. This is for the case when you are generating CAs and using them to sign certificates. In the case where the CA that signed the certificate is a known trusted CA, we shouldn't have to provide its CA certificate. This PR fixes this problem by allowing the code to pull its CA pool from the host environment when CAs are not explicitly supplied. **Link to tracking Issue:** [github.com/open-telemetry/issues/31191](open-telemetry#31191) **Testing:** Generate metrics, logs, traces against http and grpc endpoints with legitimate certificates **Documentation:** <Describe the documentation added.>
) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/expr-lang/expr](https://togithub.com/expr-lang/expr) | `v1.16.0` -> `v1.16.1` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fexpr-lang%2fexpr/v1.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fexpr-lang%2fexpr/v1.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fexpr-lang%2fexpr/v1.16.0/v1.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fexpr-lang%2fexpr/v1.16.0/v1.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>expr-lang/expr (github.com/expr-lang/expr)</summary> ### [`v1.16.1`](https://togithub.com/expr-lang/expr/releases/tag/v1.16.1) [Compare Source](https://togithub.com/expr-lang/expr/compare/v1.16.0...v1.16.1) **Expr** is a Go-centric expression language designed to deliver dynamic configurations with unparalleled accuracy, safety, and speed. ```go program, err := expr.Compile(`now() - created_at < duration("24h")`) output, err := expr.Run(program, env) ``` **In this release**: - Fixed recursive operator overloading bug ([#&open-telemetry#8203;548](https://togithub.com/expr-lang/expr/issues/548), [#&open-telemetry#8203;554](https://togithub.com/expr-lang/expr/issues/554)) - Fixed AST printing of `?.[` operator ([`ffaec9e`](https://togithub.com/expr-lang/expr/commit/ffaec9e)) - Fixed string literal slicing error ([#&open-telemetry#8203;543](https://togithub.com/expr-lang/expr/issues/543)) - Compilation performance improvements ([`95084fb`](https://togithub.com/expr-lang/expr/commit/95084fb)) - Improved functions call performance ([`ed89647`](https://togithub.com/expr-lang/expr/commit/ed89647)) - Refactored `groupBy()` and `sortBy()` builtins ([#&open-telemetry#8203;562](https://togithub.com/expr-lang/expr/issues/562)) - Added operator overloading with `expr.Function()` ([#&open-telemetry#8203;408](https://togithub.com/expr-lang/expr/issues/408)) - Added `concat()` builtin ([#&open-telemetry#8203;565](https://togithub.com/expr-lang/expr/issues/565)) - Added `reverse()` builtin ([#&open-telemetry#8203;553](https://togithub.com/expr-lang/expr/issues/553)) - Added `program.Locations()` method ([`e53cefe`](https://togithub.com/expr-lang/expr/commit/e53cefe)) - Added optional array access `array?.[42]` ([#&open-telemetry#8203;550](https://togithub.com/expr-lang/expr/issues/550)) - Added compiler optimization for jump opcodes ([#&open-telemetry#8203;545](https://togithub.com/expr-lang/expr/issues/545)) - Added compiler optimization for boolean operations with `all`, `any`, `one`, `none` functions ([#&open-telemetry#8203;555](https://togithub.com/expr-lang/expr/issues/555)) **Expr Editor** The [Expr Editor](https://expr-lang.org/editor) is an embeddable code editor written in JavaScript with full support of Expr language. **Expr Pro** [Expr Pro](https://expr-lang.org/expr-pro) is a set of extensions for Expr for advanced use cases. It includes expressions explanation, performance profiling, and more. </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMDAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]>
…ry#31348) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [sigs.k8s.io/controller-runtime](https://togithub.com/kubernetes-sigs/controller-runtime) | `v0.16.3` -> `v0.17.2` | [![age](https://developer.mend.io/api/mc/badges/age/go/sigs.k8s.io%2fcontroller-runtime/v0.17.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/sigs.k8s.io%2fcontroller-runtime/v0.17.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/sigs.k8s.io%2fcontroller-runtime/v0.16.3/v0.17.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/sigs.k8s.io%2fcontroller-runtime/v0.16.3/v0.17.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>kubernetes-sigs/controller-runtime (sigs.k8s.io/controller-runtime)</summary> ### [`v0.17.2`](https://togithub.com/kubernetes-sigs/controller-runtime/releases/tag/v0.17.2) [Compare Source](https://togithub.com/kubernetes-sigs/controller-runtime/compare/v0.17.1...v0.17.2) #### What's Changed - ⚠ 🐛 Fakeclient: Do not consider an apply patch to be a strategic merge patch by [@&open-telemetry#8203;k8s-infra-cherrypick-robot](https://togithub.com/k8s-infra-cherrypick-robot) in [https://github.com/kubernetes-sigs/controller-runtime/pull/2681](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2681) - 🐛 Fix lazy rest mapper cache invalidation by [@&open-telemetry#8203;k8s-infra-cherrypick-robot](https://togithub.com/k8s-infra-cherrypick-robot) in [https://github.com/kubernetes-sigs/controller-runtime/pull/2688](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2688) **Full Changelog**: kubernetes-sigs/controller-runtime@v0.17.1...v0.17.2 ### [`v0.17.1`](https://togithub.com/kubernetes-sigs/controller-runtime/releases/tag/v0.17.1) [Compare Source](https://togithub.com/kubernetes-sigs/controller-runtime/compare/v0.17.0...v0.17.1) ##### What's Changed - 🐛 Clean restmapper cache if a version is notFound by [@&open-telemetry#8203;k8s-infra-cherrypick-robot](https://togithub.com/k8s-infra-cherrypick-robot) in [https://github.com/kubernetes-sigs/controller-runtime/pull/2676](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2676) **Full Changelog**: kubernetes-sigs/controller-runtime@v0.17.0...v0.17.1 ### [`v0.17.0`](https://togithub.com/kubernetes-sigs/controller-runtime/releases/tag/v0.17.0) [Compare Source](https://togithub.com/kubernetes-sigs/controller-runtime/compare/v0.16.5...v0.17.0) ####⚠️ Breaking Changes - Fake client: Only set TypeMeta for unstructured ([#&open-telemetry#8203;2633](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2633)) - Update k8s.io/\* deps to 1.29 ([#&open-telemetry#8203;2457](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2457) [#&open-telemetry#8203;2500](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2500) [#&open-telemetry#8203;2553](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2553) [#&open-telemetry#8203;2588](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2588) [#&open-telemetry#8203;2612](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2612) [#&open-telemetry#8203;2615](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2615) [#&open-telemetry#8203;2621](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2621)) - Remove apiutil.NewDiscoveryRESTMapper, use DynamicRESTMapper by default for cache ([#&open-telemetry#8203;2611](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2611)) - RESTMapper: don't treat non-existing GroupVersions as errors ([#&open-telemetry#8203;2571](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2571)) - Webhooks: Deprecate admission.Validator and admission.Defaulter ([#&open-telemetry#8203;2639](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2639)) - Webhooks: Limit size of AdmissionRequests to 7MB and size of TokenReviews to 1MB ([#&open-telemetry#8203;2598](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2598)) #### ✨ New Features - Cache: Allow defining options that apply to all namespaces without explicit config ([#&open-telemetry#8203;2528](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2528)) - Cache: Allow to remove informers ([#&open-telemetry#8203;2285](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2285)) - Cache: Add option to set WatchErrorHandler on informers ([#&open-telemetry#8203;2494](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2494)) - Client: client.MatchingFields now supports multiple indexes ([#&open-telemetry#8203;2512](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2512)) - Fake client: Add flowcontrol api v1 ([#&open-telemetry#8203;2623](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2623)) - Reconciler: Add reconcile.ObjectReconciler ([#&open-telemetry#8203;2592](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2592)) #### 🐛 Bug Fixes - Add corev1, coordinationv1 scheme for leader election when LeaderElection manager option is true ([#&open-telemetry#8203;2461](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2461)) - Cache: Default namespace only for namespaced object ([#&open-telemetry#8203;2480](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2480)) - Client: Don't truncate large int64s in MergePatch ([#&open-telemetry#8203;2650](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2650)) - controllerutil: Add `RemoveControllerReference` and `HasControllerReference` func ([#&open-telemetry#8203;2509](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2509) [#&open-telemetry#8203;2595](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2595)) - controllerutil: Add `RemoveOwnerReference` func ([#&open-telemetry#8203;2462](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2462)) - Dependencies: Refactor tests to drop hard otel dependency ([#&open-telemetry#8203;2460](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2460)) - Envtest: CRDs that aren't convertible should unset spec.conversion ([#&open-telemetry#8203;2525](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2525)) - Fake client: Correctly identify if patch call was made on status ([#&open-telemetry#8203;2508](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2508)) - Fake client: Do not update anything but status when using subresource client ([#&open-telemetry#8203;2479](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2479)) - Fake client: Fix returning object after status update ([#&open-telemetry#8203;2489](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2489)) - Fake client: Fix status subresource getting updated on Update when it is empty ([#&open-telemetry#8203;2484](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2484)) - Fake client: Handle unstructured status update with fake client ([#&open-telemetry#8203;2495](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2495)) - Manager: Fix goroutine leak ([#&open-telemetry#8203;2527](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2527)) - Manager: Use HTTP client from leaderElectionConfig for leader election recorder provider ([#&open-telemetry#8203;2464](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2464)) - RESTMapper: Return NoResourceMatchError when appropriate for backwards compatibility. ([#&open-telemetry#8203;2472](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2472)) - Webhook: Handle http.NoBody ([#&open-telemetry#8203;2605](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2605)) #### 🌱 Others - Client: Make client.MatchingLabels faster ([#&open-telemetry#8203;2529](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2529)) - Envtest: Allow to ignore scheme.Convertible check for CRDs ([#&open-telemetry#8203;2555](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2555)) - Envtest: Improve process cleanup ([#&open-telemetry#8203;2560](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2560)) - Metrics: Add 100/1000s buckets for Prometheus workqueue histograms ([#&open-telemetry#8203;2638](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2638)) - Metrics: Avoid dependency on apiserver/options if metrics/filters are used ([#&open-telemetry#8203;2645](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2645)) - Replace k8s.io/utils/pointer with k8s.io/utils/ptr ([#&open-telemetry#8203;2488](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2488)) - Webhooks: Cleanup webhook variable assignment ([#&open-telemetry#8203;2604](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2604)) #### 🌱 Dependencies - Bump github.com/emicklei/go-restful/v3 from v3.9.0 to 3.11.0 ([#&open-telemetry#8203;2507](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2507)) - Bump github.com/evanphx/json-patch from 5.6.0+incompatible to 4.12.0+incompatible ([#&open-telemetry#8203;2501](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2501) [#&open-telemetry#8203;2643](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2643)) - Note: We "downgraded" to v4.12.0+incompatible because it is actually newer than v5.6.0+incompatible and to make sure the code in fake client that is using this dependency behaves exactly the same as the corresponding upstream code. We recommend users to either use github.com/evanphx/[email protected]+incompatible or to upgrade to github.com/evanphx/json-patch/v5@&open-telemetry#8203;v5.8.0. More details can be fou[https://github.com/kubernetes-sigs/controller-runtime/pull/2643#discussion_r1446189655](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2643#discussion_r1446189655)189655. - Bump github.com/evanphx/json-patch/v5 from 5.6.0 to 5.8.0 ([#&open-telemetry#8203;2499](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2499) [#&open-telemetry#8203;2650](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2650)) - Bump github.com/fsnotify/fsnotify from 1.6.0 to 1.7.0 ([#&open-telemetry#8203;2561](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2561)) - Bump github.com/go-logr/logr from 1.2.4 to 1.4.1 ([#&open-telemetry#8203;2566](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2566) [#&open-telemetry#8203;2630](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2630)) - Bump github.com/go-logr/zapr from 1.2.4 to 1.3.0 ([#&open-telemetry#8203;2574](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2574)) - Bump github.com/google/go-cmp from 0.5.9 to 0.6.0 ([#&open-telemetry#8203;2545](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2545)) - Bump github.com/onsi/ginkgo/v2 from 2.11.0 to 2.14.0 ([#&open-telemetry#8203;2458](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2458) [#&open-telemetry#8203;2514](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2514) [#&open-telemetry#8203;2534](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2534) [#&open-telemetry#8203;2578](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2578) [#&open-telemetry#8203;2610](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2610) [#&open-telemetry#8203;2651](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2651)) - Bump github.com/onsi/gomega from 1.27.10 to 1.30.0 ([#&open-telemetry#8203;2521](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2521) [#&open-telemetry#8203;2562](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2562) [#&open-telemetry#8203;2568](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2568) [#&open-telemetry#8203;2579](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2579)) - Bump github.com/prometheus/client_golang from 1.16.0 to 1.18.0 ([#&open-telemetry#8203;2520](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2520) [#&open-telemetry#8203;2632](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2632)) - Bump github.com/prometheus/client_model from 0.4.1-0.20230718164431-9a2bf3000d16 to 0.5.0 ([#&open-telemetry#8203;2535](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2535)) - Bump go.uber.org/goleak from 1.2.1 to 1.3.0 ([#&open-telemetry#8203;2569](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2569)) - Bump go.uber.org/zap from 1.25.0 to 1.26.0 ([#&open-telemetry#8203;2502](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2502)) - Bump golang.org/x/net to 0.17.0 ([#&open-telemetry#8203;2540](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2540)) - Bump golang.org/x/sys from 0.11.0 to 0.16.0 ([#&open-telemetry#8203;2470](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2470) [#&open-telemetry#8203;2533](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2533) [#&open-telemetry#8203;2572](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2572) [#&open-telemetry#8203;2597](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2597) [#&open-telemetry#8203;2642](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2642)) - Bump k8s.io/klog/v2 from 2.100.1 to 2.110.1 ([#&open-telemetry#8203;2573](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2573)) - Bump sigs.k8s.io/yaml from 1.3.0 to 1.4.0 ([#&open-telemetry#8203;2567](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2567)) - Bump controller-tools from v0.13.0 to v0.14.0 ([#&open-telemetry#8203;2649](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2649)) - Bump go-apidiff from v0.5.0 to v0.8.2 ([#&open-telemetry#8203;2517](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2517) [#&open-telemetry#8203;2655](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2655)) - Upgrade dependencies for hack/tools and tools/ ([#&open-telemetry#8203;2517](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2517)) - Bump kubernetes-sigs/kubebuilder-release-tools from 0.3.0 to 0.4.3 ([#&open-telemetry#8203;2532](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2532) [#&open-telemetry#8203;2575](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2575) [#&open-telemetry#8203;2618](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2618)) - Bump golang version in GH actions ([#&open-telemetry#8203;2609](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2609)) - Bump actions/checkout from 3 to 4 ([#&open-telemetry#8203;2469](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2469)) - Bump actions/setup-go from 4 to 5 ([#&open-telemetry#8203;2617](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2617)) - Bump envtest to 1.28.0 ([#&open-telemetry#8203;2531](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2531)) :book: Additionally, there have been 5 contributions to our documentation. ([#&open-telemetry#8203;2497](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2497), [#&open-telemetry#8203;2498](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2498), [#&open-telemetry#8203;2538](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2538), [#&open-telemetry#8203;2599](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2599), [#&open-telemetry#8203;2602](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2602), [#&open-telemetry#8203;2635](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2635) [#&open-telemetry#8203;2647](https://togithub.com/kubernetes-sigs/controller-runtime/issues/2647)) *Thanks to all our contributors!* 😊 ### [`v0.16.5`](https://togithub.com/kubernetes-sigs/controller-runtime/releases/tag/v0.16.5) [Compare Source](https://togithub.com/kubernetes-sigs/controller-runtime/compare/v0.16.4...v0.16.5) #### What's Changed - 🐛 Fix lazy rest mapper cache invalidation by [@&open-telemetry#8203;g-gaston](https://togithub.com/g-gaston) in [https://github.com/kubernetes-sigs/controller-runtime/pull/2689](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2689) **Full Changelog**: kubernetes-sigs/controller-runtime@v0.16.4...v0.16.5 ### [`v0.16.4`](https://togithub.com/kubernetes-sigs/controller-runtime/releases/tag/v0.16.4) [Compare Source](https://togithub.com/kubernetes-sigs/controller-runtime/compare/v0.16.3...v0.16.4) #### What's Changed - 🏃 Make client.MatchingLabels faster by [@&open-telemetry#8203;samuelvl](https://togithub.com/samuelvl) in [https://github.com/kubernetes-sigs/controller-runtime/pull/2613](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2613) - 🐛 Avoid extra calls for not found resource by [@&open-telemetry#8203;g-gaston](https://togithub.com/g-gaston) in [https://github.com/kubernetes-sigs/controller-runtime/pull/2677](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2677) #### New Contributors - [@&open-telemetry#8203;samuelvl](https://togithub.com/samuelvl) made their first contribution in [https://github.com/kubernetes-sigs/controller-runtime/pull/2613](https://togithub.com/kubernetes-sigs/controller-runtime/pull/2613) **Full Changelog**: kubernetes-sigs/controller-runtime@v0.16.3...v0.16.4 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMDAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]>
…ry#31225) **Description:** Adding interface and other related files to serve as placeholder for the Ack extension proposed in open-telemetry#26376. Implementation will be added after this PR gets merged. **Link to tracking Issue:** open-telemetry#26376
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
It is a cherry-pick from opentelemetry upstream open-telemetry#31120
awsxrayexporter
is able to handle both old and new versions of http span attributes, support translate new http attributes to xray segment fields by following http server span and client span spec.Link to tracking Issue:
open-telemetry#30935
Testing:
Documentation:
https://opentelemetry.io/blog/2023/http-conventions-declared-stable/