The following commands were run to prepare this release:
make chlog-update VERSION=v1.29.0/v0.123.0
make prepare-release PREVIOUS_VERSION=1.28.1 RELEASE_CANDIDATE=1.29.0
MODSET=stable
make prepare-release PREVIOUS_VERSION=0.122.1 RELEASE_CANDIDATE=0.123.0
MODSET=beta
#### Description
Fork of #12384 to showcase how component attributes can be injected into
scope attributes instead of log/metric/span attributes. See that PR for
more context.
To see the diff from the previous PR, filter changes starting from the
"Prototype using scope attributes" commit.
#### Link to tracking issue
Resolves#12217
Also incidentally resolves#12213 and resolves#12117
#### Testing
I updated the existing tests to check for scope attributes, and did some
manual testing with a debug exporter to check that the scope attributes
are added/removed properly.
---------
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
In this PR, we merge the functionality from "disabled_queue" within the
memory queue. For the moment this functionality is not available for the
persistent queue.
In the next PR, will merge the `internal.QueueConfig` with
`queuebatch.Config`.
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
This PR:
* Helps with
https://github.com/open-telemetry/opentelemetry-collector/issues/12709,
to avoid the situation where we need to add Unmarshal for a struct and
if used as embedded will not work.
* Helps with avoiding name conflicts between variables. Will allow for
example to have a "Timeout" member in both QueueConfig and ClientConfig,
even though that was a valid YAML config since `QueueConfig` is under
"seding_queue".
* This is a breaking change only if devs are created manually the OTLP
configs which should be very rare, and unfortunately this is hard to do
using deprecation steps.
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
<!-- Issue number if applicable -->
#### Link to tracking issue
Fixes #
<!--Describe what testing was performed and which tests were added.-->
#### Testing
<!--Describe the documentation added.-->
#### Documentation
<!--Please delete paragraphs that you did not use before submitting.-->
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Added the cspell to check spelling in .md, .yaml files.
<!-- Issue number if applicable -->
#### Link to tracking issue
Fixes#9287
<!--Describe what testing was performed and which tests were added.-->
#### Testing
<!--Describe the documentation added.-->
#### Documentation
<!--Please delete paragraphs that you did not use before submitting.-->
---------
Signed-off-by: Yuri Oliveira <yurimsa@gmail.com>
Add this as a separate config than exporterqueue.Config until the queue
and batcher are changed to support this new config. Once approved, will
merge and in sub-sequent PRs will change the components to accept this
new config and expose it to the users.
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
The following commands were run to prepare this release:
* make chlog-update VERSION=v1.28.1/v0.122.1
* make prepare-release PREVIOUS_VERSION=1[.]28[.]0
RELEASE_CANDIDATE=1.28.1 MODSET=stable
* make prepare-release PREVIOUS_VERSION=0[.]122[.]0
RELEASE_CANDIDATE=0.122.1 MODSET=beta
---------
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
The following commands were run to prepare this release:
- make chlog-update VERSION=v1.28.0/v0.122.0
- make prepare-release PREVIOUS_VERSION=1[.]27[.]0
RELEASE_CANDIDATE=1.28.0 MODSET=stable
- make prepare-release PREVIOUS_VERSION=0[.]121[.]0
RELEASE_CANDIDATE=0.122.0 MODSET=beta
Fixes
https://github.com/open-telemetry/opentelemetry-collector/issues/8950
The logic behind this, is that during merging and splitting we should
not care about the export func (see complicated logic removed in
FakeRequest of in the pdata requests implementations).
Also, this simplify what user should carry with the request.
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
#### Description
This PR adds support for iterators to Slice and Map types and their
autogenerated counterparts. Iterators were stabilized in Go 1.23.
The All method is analogous to [maps.All](https://pkg.go.dev/maps#All)
and [slices.All ](https://pkg.go.dev/slices#All) and is a more idiomatic
alternative to the more verbose `for i := 0; i < es.Len(); i++ { z :=
es.At(i) }` way of looping.
Code that is written like this today:
```go
for i := 0; i < ld.ResourceLogs().Len(); i++ {
rl := ld.ResourceLogs().At(i)
for j := 0; j < rl.ScopeLogs().Len(); j++ {
sl := rl.ScopeLogs().At(j)
for k := 0; k < sl.LogRecords().Len(); k++ {
lr := sl.LogRecords().At(k)
// use lr
}
}
}
```
Can now be written like this:
```go
for _, rl := range ld.ResourceLogs().All() {
for _, sl := range rl.ScopeLogs().All() {
for _, lr := range sl.LogRecords().All() {
// use lr
}
}
}
```
#### Link to tracking issue
Fixes
https://github.com/open-telemetry/opentelemetry-collector/issues/11982