…elemetry#27418)

This reworks a recent addition (See open-telemetry#27064) to how we handle rotated
files.

The solution depends upon re-reading a file's fingerprint. This is
unchanged, but most of the implementation is now localized to the reader
struct.

…lemetry#27417)

Subset of open-telemetry#27416

…age (open-telemetry#27416)

Follows open-telemetry#27396 

This PR creates an internal `reader` package and moves directly related
structs into it.

I intend to clean up this codebase substantially from here. This is just
a first step that creates a crude boundary between concerns. There are
many exported fields which can later be abstracted, but currently the
codebase has many direct interactions. Tests remain in the
`fileconsumer` package for now but will be migrated once there are
cleaner interfaces to test.

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to
4.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/actions/checkout/releases">actions/checkout's
releases</a>.</em></p>
<blockquote>
<h2>v4.0.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Update default runtime to node20 by <a
href="https://github.com/takost"><code>@​takost</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1436">actions/checkout#1436</a></li>
<li>Support fetching without the --progress option by <a
href="https://github.com/simonbaird"><code>@​simonbaird</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1067">actions/checkout#1067</a></li>
<li>Release 4.0.0 by <a
href="https://github.com/takost"><code>@​takost</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1447">actions/checkout#1447</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/takost"><code>@​takost</code></a> made
their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1436">actions/checkout#1436</a></li>
<li><a
href="https://github.com/simonbaird"><code>@​simonbaird</code></a> made
their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1067">actions/checkout#1067</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/actions/checkout/compare/v3...v4.0.0">https://github.com/actions/checkout/compare/v3...v4.0.0</a></p>
<h2>v3.6.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Mark test scripts with Bash'isms to be run via Bash by <a
href="https://github.com/dscho"><code>@​dscho</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1377">actions/checkout#1377</a></li>
<li>Add option to fetch tags even if fetch-depth &gt; 0 by <a
href="https://github.com/RobertWieczoreck"><code>@​RobertWieczoreck</code></a>
in <a
href="https://redirect.github.com/actions/checkout/pull/579">actions/checkout#579</a></li>
<li>Release 3.6.0 by <a
href="https://github.com/luketomlinson"><code>@​luketomlinson</code></a>
in <a
href="https://redirect.github.com/actions/checkout/pull/1437">actions/checkout#1437</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a
href="https://github.com/RobertWieczoreck"><code>@​RobertWieczoreck</code></a>
made their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/579">actions/checkout#579</a></li>
<li><a
href="https://github.com/luketomlinson"><code>@​luketomlinson</code></a>
made their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1437">actions/checkout#1437</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/actions/checkout/compare/v3.5.3...v3.6.0">https://github.com/actions/checkout/compare/v3.5.3...v3.6.0</a></p>
<h2>v3.5.3</h2>
<h2>What's Changed</h2>
<ul>
<li>Fix: Checkout Issue in self hosted runner due to faulty submodule
check-ins by <a
href="https://github.com/megamanics"><code>@​megamanics</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1196">actions/checkout#1196</a></li>
<li>Fix typos found by codespell by <a
href="https://github.com/DimitriPapadopoulos"><code>@​DimitriPapadopoulos</code></a>
in <a
href="https://redirect.github.com/actions/checkout/pull/1287">actions/checkout#1287</a></li>
<li>Add support for sparse checkouts by <a
href="https://github.com/dscho"><code>@​dscho</code></a> and <a
href="https://github.com/dfdez"><code>@​dfdez</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1369">actions/checkout#1369</a></li>
<li>Release v3.5.3 by <a
href="https://github.com/TingluoHuang"><code>@​TingluoHuang</code></a>
in <a
href="https://redirect.github.com/actions/checkout/pull/1376">actions/checkout#1376</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a
href="https://github.com/megamanics"><code>@​megamanics</code></a> made
their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1196">actions/checkout#1196</a></li>
<li><a
href="https://github.com/DimitriPapadopoulos"><code>@​DimitriPapadopoulos</code></a>
made their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1287">actions/checkout#1287</a></li>
<li><a href="https://github.com/dfdez"><code>@​dfdez</code></a> made
their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1369">actions/checkout#1369</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/actions/checkout/compare/v3...v3.5.3">https://github.com/actions/checkout/compare/v3...v3.5.3</a></p>
<h2>v3.5.2</h2>
<h2>What's Changed</h2>
<ul>
<li>Fix: Use correct API url / endpoint in GHES by <a
href="https://github.com/fhammerl"><code>@​fhammerl</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1289">actions/checkout#1289</a>
based on <a
href="https://redirect.github.com/actions/checkout/issues/1286">#1286</a>
by <a href="https://github.com/1newsr"><code>@​1newsr</code></a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/actions/checkout/compare/v3.5.1...v3.5.2">https://github.com/actions/checkout/compare/v3.5.1...v3.5.2</a></p>
<h2>v3.5.1</h2>
<h2>What's Changed</h2>
<ul>
<li>Improve checkout performance on Windows runners by upgrading
<code>@​actions/github</code> dependency by <a
href="https://github.com/BrettDong"><code>@​BrettDong</code></a> in <a
href="https://redirect.github.com/actions/checkout/pull/1246">actions/checkout#1246</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/BrettDong"><code>@​BrettDong</code></a>
made their first contribution in <a
href="https://redirect.github.com/actions/checkout/pull/1246">actions/checkout#1246</a></li>
</ul>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/actions/checkout/blob/main/CHANGELOG.md">actions/checkout's
changelog</a>.</em></p>
<blockquote>
<h1>Changelog</h1>
<h2>v4.1.0</h2>
<ul>
<li><a href="https://redirect.github.com/actions/checkout/pull/1396">Add
support for partial checkout filters</a></li>
</ul>
<h2>v4.0.0</h2>
<ul>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1067">Support
fetching without the --progress option</a></li>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1436">Update to
node20</a></li>
</ul>
<h2>v3.6.0</h2>
<ul>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1377">Fix: Mark
test scripts with Bash'isms to be run via Bash</a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/579">Add
option to fetch tags even if fetch-depth &gt; 0</a></li>
</ul>
<h2>v3.5.3</h2>
<ul>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1196">Fix:
Checkout fail in self-hosted runners when faulty submodule are
checked-in</a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/1287">Fix
typos found by codespell</a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/1369">Add
support for sparse checkouts</a></li>
</ul>
<h2>v3.5.2</h2>
<ul>
<li><a href="https://redirect.github.com/actions/checkout/pull/1289">Fix
api endpoint for GHES</a></li>
</ul>
<h2>v3.5.1</h2>
<ul>
<li><a href="https://redirect.github.com/actions/checkout/pull/1246">Fix
slow checkout on Windows</a></li>
</ul>
<h2>v3.5.0</h2>
<ul>
<li><a href="https://redirect.github.com/actions/checkout/pull/1237">Add
new public key for known_hosts</a></li>
</ul>
<h2>v3.4.0</h2>
<ul>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1209">Upgrade
codeql actions to v2</a></li>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1210">Upgrade
dependencies</a></li>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1225">Upgrade
<code>@​actions/io</code></a></li>
</ul>
<h2>v3.3.0</h2>
<ul>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1045">Implement
branch list using callbacks from exec function</a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/1050">Add
in explicit reference to private checkout options</a></li>
<li>[Fix comment typos (that got added in <a
href="https://redirect.github.com/actions/checkout/issues/770">#770</a>)](<a
href="https://redirect.github.com/actions/checkout/pull/1057">actions/checkout#1057</a>)</li>
</ul>
<h2>v3.2.0</h2>
<ul>
<li><a href="https://redirect.github.com/actions/checkout/pull/942">Add
GitHub Action to perform release</a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/967">Fix
status badge</a></li>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1002">Replace
datadog/squid with ubuntu/squid Docker image</a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/964">Wrap
pipeline commands for submoduleForeach in quotes</a></li>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1029">Update
<code>@​actions/io</code> to 1.1.2</a></li>
<li><a
href="https://redirect.github.com/actions/checkout/pull/1039">Upgrading
version to 3.2.0</a></li>
</ul>
<h2>v3.1.0</h2>
<ul>
<li><a href="https://redirect.github.com/actions/checkout/pull/939">Use
<code>@​actions/core</code> <code>saveState</code> and
<code>getState</code></a></li>
<li><a href="https://redirect.github.com/actions/checkout/pull/922">Add
<code>github-server-url</code> input</a></li>
</ul>
<h2>v3.0.2</h2>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/actions/checkout/commit/8ade135a41bc03ea155e62e844d188df1ea18608"><code>8ade135</code></a>
Prepare 4.1.0 release (<a
href="https://redirect.github.com/actions/checkout/issues/1496">#1496</a>)</li>
<li><a
href="https://github.com/actions/checkout/commit/c533a0a4cfc4962971818edcfac47a2899e69799"><code>c533a0a</code></a>
Add support for partial checkout filters (<a
href="https://redirect.github.com/actions/checkout/issues/1396">#1396</a>)</li>
<li><a
href="https://github.com/actions/checkout/commit/72f2cec99f417b1a1c5e2e88945068983b7965f9"><code>72f2cec</code></a>
Update README.md for V4 (<a
href="https://redirect.github.com/actions/checkout/issues/1452">#1452</a>)</li>
<li><a
href="https://github.com/actions/checkout/commit/3df4ab11eba7bda6032a0b82a6bb43b11571feac"><code>3df4ab1</code></a>
Release 4.0.0 (<a
href="https://redirect.github.com/actions/checkout/issues/1447">#1447</a>)</li>
<li><a
href="https://github.com/actions/checkout/commit/8b5e8b768746b50394015010d25e690bfab9dfbc"><code>8b5e8b7</code></a>
Support fetching without the --progress option (<a
href="https://redirect.github.com/actions/checkout/issues/1067">#1067</a>)</li>
<li><a
href="https://github.com/actions/checkout/commit/97a652b80035363df47baee5031ec8670b8878ac"><code>97a652b</code></a>
Update default runtime to node20 (<a
href="https://redirect.github.com/actions/checkout/issues/1436">#1436</a>)</li>
<li>See full diff in <a
href="https://github.com/actions/checkout/compare/v3...v4">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=actions/checkout&package-manager=github_actions&previous-version=3&new-version=4)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…metry#27269)

**Description:**
Change entirely how dependabot update entries are generated, by using
the metadata.yaml status to find which components are most important in
the distribution.

The code now takes into account the distributions and the stability of
the component as a score to decide whether to push the component. Go
modules that don't have an associated metadata.yaml are not considered
and therefore not present in the module updates path.

**Link to tracking Issue:**

open-telemetry#19410

…elemetry#27253)

**Description:**
Move the filestats receiver to beta stability

**Link to tracking Issue:**
Fixes open-telemetry#27252

Prune histograms when the dimension cache evictions are removed

**Description:**
Prunes histograms when the dimension cache is pruned. This prevents
metric series from growing indefinitely

**Link to tracking Issue:**
 open-telemetry#27080

**Testing:**
I modified the the existing test to check `histograms` length instead of
dimensions cache length. This required simulating ticks to hit the
exportMetrics function

**Documentation:** <Describe the documentation added.>

Co-authored-by: Sean Marciniak <[email protected]>

…lemetry#27266)

Co-authored-by: Sean Marciniak <[email protected]>

…try#27421)

This test failed on windows
([here](https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/6400499092/job/17374210350#step:6:659)),
apparently because the storage file is still open when the test
framework attempts to clean up the directory.

This PR simplifies the flush function. The behavior should be the same
but the representation is simpler.

…ry#27422)

This will allow the load tests to produce consistent results. Note that
I removed the execution on every pull request as part of this change.
This is because the load tests running on every build would be too much
for the dedicated hardware to handle.

I'm not sure if we're getting much benefit from those tests running on
every build at this time, though we could always change this to only run
on main/tagged releases on hardware and otherwise on regular github
runners.

Waiting on open-telemetry/community#1708
before marking as ready for review

Signed-off-by: Alex Boten <[email protected]>

**Description:** 
Do not export function WithTracesMarshalers, add test for that and pass
checkapi

**Link to tracking Issue:** 

open-telemetry#26304

**Testing:** 
make chlog-validate
go run cmd/checkapi/main.go .
go test for pulsarexporter

**Documentation:**

**Description:** 
Do not export functions `WithLogsUnmarshalers`,
`WithMetricsUnmarshalers`, `WithTracesUnmarshalers`, add tests for that
and pass checkapi

**Link to tracking Issue:**

open-telemetry#26304

**Testing:** 
go run cmd/checkapi/main.go .
make chlog-validate
go test for pulsarreceiver

**Documentation:**

…ry#23947)

**Description:** <Describe what has changed.>
Adding Zipkin v2 encoding support for traces in kafkaexporter 

**Link to tracking Issue:** <Issue number if applicable>

open-telemetry#21102

**Testing:** <Describe what testing was performed and which tests were
added.>
Test for `tracesMarshalers` has been extended with zipkin JSON test
case.

…emetry#26564)

When doing a probabilistic sampling for logs, the probabilistic sampler
currently requires the source attribute to be of bytes value. This PR
changes that to allow any value type to be used.

Fixes open-telemetry#18222

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

---------

Signed-off-by: Juraci Paixão Kröhling <[email protected]>

…en-telemetry#27415)

**Description:**

Syslog receiver produces attribute (`structured_data`) which is a
stringified map. This PR fixes it. See Issue for more details

**Link to tracking Issue:**
open-telemetry#27414

**Testing:**

UT

**Documentation:**

In code comments

---------

Signed-off-by: Dominik Rosiek <[email protected]>

An error appears to be the load tests to fail, but it's not being
checked which makes it hard to debug. Helps with open-telemetry#27429

Signed-off-by: Alex Boten <[email protected]>
Co-authored-by: Bogdan Drutu <[email protected]>

Incremental step towards decoupling the `Reader` struct.

The TestTraceNoBackend10kSPS/MemoryLimit seems to hit about 75% on runs
on the dedicated host

Signed-off-by: Alex Boten <[email protected]>

…les (open-telemetry#27435)

Incremental step towards decoupling the Reader struct.

It is not clear from the example how the tags should be written down.

**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.-->

**Link to tracking Issue:** <Issue number if applicable>

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>

---------

Co-authored-by: Yang Song <[email protected]>
Co-authored-by: Mackenzie <[email protected]>

Had to update a Prometheus exporter test `TestCollectMetrics` to adapt
to the metric description String() output change made the recent Prom
library version prometheus/client_golang#1296.

Fixes
open-telemetry#27398

…defaults) (open-telemetry#27424)

**Description:**

- fix defaults
- fix some typos
- correctly handle data structures which are output of syslog receiver
- allow to not use tls and disable it by default

**Link to tracking Issue:** open-telemetry#25114 

**Testing:** 
Unit tests

**Documentation:** 
N/A

---------

Signed-off-by: Dominik Rosiek <[email protected]>

…7447)

The original logger is sampled by default now after
open-telemetry/opentelemetry-collector#8134

…lemetry#27448)

The original logger is now sampled by default after
open-telemetry/opentelemetry-collector#8134

This will result in more consistent benchmarks across different
environments.

Fixes open-telemetry#27429

Signed-off-by: Alex Boten <[email protected]>

Fixes a bug in the PRW example YAML, the `target_info` indentation.

This is a docs update.

Don't we all love YAML? ;)

)

**Description:** 
Added a line explaining to run collector as admin, since not all perf
counters are collected unless you do.
(Or I'm missing something, which should be documented)

**Testing:** No unit tests, but observed this with a simple collector
install on windows.

**Documentation:** See above.

…etry#27007)

**Description:**

- Refactored and moved `otlp-http-url-path` flag from the common flags
for each command
    - Added flag to the `traces` command with the default `/v1/traces`
    - Added flag to the `metrics` command with the default `/v1/metrics`
    - Flag was not used for the `logs` command so no longer exposed
- Handle changing the default endpoint based on the communication mode
(gRPC or HTTP)
- Flag `--otlp-endpoint` now defaults to empty string and targets a new
attribute `CustomEndpoint` on `common.Config`
- Added the `Endpoint()` getter function to the `common.Config` struct
for retrieving the correct endpoint
- When `CustomEndpoint` is empty then either `DefaultGRPCEndpoint` or
`DefaultHTTPEndpoint` are chosen based upon the value of
`config.UseHTTP`
- **Misc**: Split out the creation of trace/metric exporters into
standalone factory functions with docstrings available in `exporters.go`
in both modules
- **Misc**: Removed the "default value is" from the descriptions of some
flags as it was repeated information

**Link to tracking Issue:** open-telemetry#26999

**Testing:** 

Added new set of unit tests to cover the addition of the
`config.Endpoint()` getter.

Running `telemetrygen traces --otlp-http --otlp-insecure` now correctly
sends traces using HTTP to the default HTTP endpoint.

**Documentation:** 

No documentation added/changed but updated some command flag
descriptions.

---------

Co-authored-by: Alex Boten <[email protected]>

…ry#27453)

Subset of open-telemetry#27434

)

Subset of open-telemetry#27434

…auth_type` (open-telemetry#26665) (open-telemetry#27240)

**Description:**
Support specifying context for `kubeConfig` `auth_type` when
communicating with a Kubernetes cluster.

**Link to tracking Issue:** 26665

**Testing:** unit test added

**Documentation:** kubeletstatsreceiver README updated

Signed-off-by: JACQUES Francois <[email protected]>

…pen-telemetry#26486)

Context:
GoogleCloudPlatform/opentelemetry-operations-go#679

faas.id was removed from the semantic conventions:
open-telemetry/opentelemetry-specification#3188

We were previously using it improperly to store the instance id of the
faas, which should be mapped to faas.instance instead.

---------

Co-authored-by: Alex Boten <[email protected]>

…ry#27276)

**Description:** 
Adds new CPU utilization metrics with respect to pod/container CPU
limits and requests

**Link to tracking Issue:** <Issue number if applicable>
Closes
open-telemetry#24905

**Testing:** <Describe what testing was performed and which tests were
added.>
Added new unit tests and tested locally

…en-telemetry#27455)

This reduces the complexity of interacting with readers by letting them
delete their own file if configured to do so.

`ca_file`, `cert_file` and `key_file` should only be used if `insecure`
is set to `false`, otherwise client TLS is disabled:
https://github.com/open-telemetry/opentelemetry-collector/blob/7c5ecef11dff4ce5501c9683b277a25a61ea0f1a/config/configtls/README.md

Signed-off-by: Andreas Gerstmayr <[email protected]>

Ensures the removed deprecated struct members were not used.

---------

Signed-off-by: Alex Boten <[email protected]>

**Description:** <Describe what has changed.>
- Improve the unit-test to catch similar errors in the future. 
- Fix mongodb version collection. 

**Link to tracking Issue:** <Issue number if applicable>
open-telemetry#27441 
**Testing:** <Describe what testing was performed and which tests were
added.>
Added the expectation assert for the Mock. 
This is the log output from running the tests in the first catching the
bug
```
--- FAIL: TestScraperScrape (0.00s)
    --- FAIL: TestScraperScrape/Failed_to_fetch_database_names (0.00s)
        pathTo/opentelemetry-collector-contrib/receiver/mongodbreceiver/scraper_test.go:316: FAIL:	GetVersion(string)
            		at: [pathTo/opentelemetry-collector-contrib/receiver/mongodbreceiver/scraper_test.go:146 pathTo/opentelemetry-collector-contrib/receiver/mongodbreceiver/scraper_test.go:292]
        pathTo/opentelemetry-collector-contrib/receiver/mongodbreceiver/scraper_test.go:316: PASS:	ListDatabaseNames(string,string,string)
        pathTo/opentelemetry-collector-contrib/receiver/mongodbreceiver/scraper_test.go:316: FAIL: 1 out of 2 expectation(s) were met.
            	The code you are testing needs to make 1 more call(s).
            	at: [pathTo/opentelemetry-collector-contrib/receiver/mongodbreceiver/scraper_test.go:316]
```
**Documentation:** <Describe the documentation added.>

…pen-telemetry#27410)

Fixes a bug where the wrong function was called for the exponential
histogram type during the copy operation.

Fixes open-telemetry#27409

Added a test for `combine` that covers the failing code path.

…27478)

According to the recent design changes

…open-telemetry#27289)

**Description:** 
Move kafka configures authentication to internal pkg, make references
become clearer. Additionally, avoid to use export function
`kafkaexporter.ConfigureAuthentication` to pass checkapi.

**Link to tracking Issue:** 

open-telemetry#27093

To properly reflect change in
open-telemetry#27289

The
[regex_parser](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/v0.86.0/pkg/stanza/docs/operators/regex_parser.md#configuration-fields)'s
property name is `regex`, not `regexp`.

…7257)

**Description:**  allow adding resource_attribute warnings

**Link to tracking Issue:**
open-telemetry#19174

**Testing:**
- Generated unit tests
- Tested manually with some changes to k8sclusterreceiver:

```
  k8s.kubelet.version:
    description: The version of Kubelet running on the node.
    type: string
    enabled: false
    warnings:
      if_configured: This resource_attribute is deprecated and will be removed soon.

  opencensus.resourcetype:
    description: The OpenCensus resource type.
    type: string
    enabled: true
    warnings:
      if_enabled: This resource_attribute is deprecated and will be removed soon.
```

Gives the following output:
```
{"level":"warn","ts":1695985300.0413303,"caller":"metadata/generated_metrics.go:2201","msg":"[WARNING] `k8s.kubelet.version` should not be configured: This resource_attribute is deprecated and will be removed soon.","kind":"receiver","name":"k8s_cluster","data_type":"metrics"}     
{"level":"warn","ts":1695985300.041354,"caller":"metadata/generated_metrics.go:2207","msg":"[WARNING] `opencensus.resourcetype` should not be enabled: This resource_attribute is deprecated and will be removed soon.","kind":"receiver","name":"k8s_cluster","data_type":"metrics"}
 ```

…telemetry#27091)

**Description:** 

Change k8s.container.ready, k8s.pod.phase, k8s.pod.status_reason,
k8s.namespace.phase units to empty

**Link to tracking Issue:** open-telemetry#10553

…-telemetry#27579)

To unblock the CI until
open-telemetry#27577
is fixed

…7474)

**Description:**
Fix metrics dropped when logs and metrics used at the same time

**Link to tracking Issue:**
Fixes open-telemetry#27473 

**Testing:**
Unit tests.

…pen-telemetry#27575)

Pass the correct `expected, actual` variables to CompareLogRecord fn in
plogtest.

…open-telemetry#27586)

Change k8s.node.condition_* metrics units to be empty.

…tribute (open-telemetry#27584)

Manually tested, built and image and manually enabled and disabled this
resource attribute.

This is the warning we publish:

>
{"level":"warn","ts":1696935210.905353,"caller":"metadata/generated_metrics.go:2201","msg":"[WARNING]
`opencensus.resourcetype` should not be configured: This
resource_attribute is deprecated and will be removed
soon.","kind":"receiver","name":"k8s_cluster","data_type":"metrics"}

…ry#27591)

Bump cloud.google.com/go/spanner from 1.49.0 to 1.50.0 in
/receiver/googlecloudspannerreceiver
Bump github.com/ClickHouse/clickhouse-go/v2 from 2.14.1 to 2.14.2 in
/exporter/clickhouseexporter
Bump
github.com/GoogleCloudPlatform/opentelemetry-operations-go/detectors/gcp
from 1.19.1 to 1.20.0 in /exporter/datadogexporter
Bump
github.com/GoogleCloudPlatform/opentelemetry-operations-go/detectors/gcp
from 1.19.1 to 1.20.0 in /processor/resourcedetectionprocessor
Bump
github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/collector
from 0.43.1 to 0.44.0 in /exporter/googlecloudexporter
Bump
github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/collector
from 0.43.1 to 0.44.0 in /exporter/googlemanagedprometheusexporter
Bump
github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/collector/googlemanagedprometheus
from 0.43.1 to 0.44.0 in /exporter/googlemanagedprometheusexporter
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/exporter/awscloudwatchlogsexporter
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/exporter/awsemfexporter
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/exporter/awsxrayexporter
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/exporter/datadogexporter
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/extension/observer/ecsobserver
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/aws/awsutil
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/aws/cwlogs
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/aws/k8s
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/aws/proxy
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/aws/xray
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/aws/xray/testdata/sampleapp
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/internal/metadataproviders
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/processor/resourcedetectionprocessor
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/receiver/awscontainerinsightreceiver
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/receiver/awsecscontainermetricsreceiver
Bump github.com/aws/aws-sdk-go from 1.45.20 to 1.45.24 in
/receiver/awsxrayreceiver
Bump github.com/aws/aws-sdk-go-v2 from 1.21.0 to 1.21.1 in
/exporter/awskinesisexporter
Bump github.com/aws/aws-sdk-go-v2 from 1.21.0 to 1.21.1 in
/extension/sigv4authextension
Bump github.com/aws/aws-sdk-go-v2/config from 1.18.43 to 1.18.44 in
/exporter/awskinesisexporter
Bump github.com/aws/aws-sdk-go-v2/config from 1.18.43 to 1.18.44 in
/extension/sigv4authextension
Bump github.com/aws/aws-sdk-go-v2/credentials from 1.13.41 to 1.13.42 in
/exporter/awskinesisexporter
Bump github.com/aws/aws-sdk-go-v2/credentials from 1.13.41 to 1.13.42 in
/extension/sigv4authextension
Bump github.com/aws/aws-sdk-go-v2/service/kinesis from 1.19.0 to 1.19.1
in /exporter/awskinesisexporter
Bump github.com/aws/aws-sdk-go-v2/service/sts from 1.23.0 to 1.23.1 in
/extension/sigv4authextension
Bump github.com/getsentry/sentry-go from 0.24.1 to 0.25.0 in
/exporter/sentryexporter
Bump github.com/prometheus/prometheus from 0.47.0 to 0.47.1 in
/exporter/prometheusexporter
Bump github.com/prometheus/prometheus from 0.47.0 to 0.47.1 in
/exporter/prometheusremotewriteexporter
Bump github.com/prometheus/prometheus from 0.47.0 to 0.47.1 in
/pkg/translator/prometheusremotewrite
Bump github.com/prometheus/prometheus from 0.47.0 to 0.47.1 in
/receiver/prometheusreceiver
Bump github.com/prometheus/prometheus from 0.47.0 to 0.47.1 in
/receiver/simpleprometheusreceiver
Bump github.com/prometheus/prometheus from 0.47.0 to 0.47.1 in /testbed
Bump github.com/sijms/go-ora/v2 from 2.7.18 to 2.7.19 in
/receiver/sqlqueryreceiver
Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common
from 1.0.759 to 1.0.761 in /exporter/tencentcloudlogserviceexporter
Bump go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp from
0.44.0 to 0.45.0 in /examples/demo/client
Bump go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp from
0.44.0 to 0.45.0 in /examples/demo/server
Bump golang.org/x/crypto from 0.13.0 to 0.14.0 in
/receiver/podmanreceiver
Bump golang.org/x/mod from 0.12.0 to 0.13.0 in /cmd/configschema
Bump golang.org/x/net from 0.15.0 to 0.16.0 in
/exporter/azuremonitorexporter
Bump golang.org/x/net from 0.15.0 to 0.16.0 in /internal/aws/awsutil
Bump golang.org/x/net from 0.15.0 to 0.16.0 in /receiver/jaegerreceiver
Bump golang.org/x/oauth2 from 0.12.0 to 0.13.0 in
/exporter/f5cloudexporter
Bump golang.org/x/oauth2 from 0.12.0 to 0.13.0 in
/extension/oauth2clientauthextension
Bump golang.org/x/sys from 0.12.0 to 0.13.0 in
/exporter/signalfxexporter
Bump golang.org/x/sys from 0.12.0 to 0.13.0 in /pkg/stanza
Bump golang.org/x/sys from 0.12.0 to 0.13.0 in /pkg/winperfcounters
Bump golang.org/x/sys from 0.12.0 to 0.13.0 in
/receiver/hostmetricsreceiver
Bump golang.org/x/sys from 0.12.0 to 0.13.0 in
/receiver/windowseventlogreceiver
Bump golang.org/x/tools from 0.13.0 to 0.14.0 in /internal/tools
Bump google.golang.org/api from 0.143.0 to 0.146.0 in
/exporter/f5cloudexporter
Bump google.golang.org/api from 0.143.0 to 0.146.0 in
/exporter/googlecloudpubsubexporter
Bump google.golang.org/api from 0.143.0 to 0.146.0 in
/receiver/googlecloudpubsubreceiver
Bump google.golang.org/api from 0.143.0 to 0.146.0 in
/receiver/googlecloudspannerreceiver

…ry#27471)

**Description:** 
We noticed that if you pass in a condition that fails parsing that the
error message includes `drop() where ...` and it was jarring to see
`drop` as the function when using filterottl to generate a condition
that was not related to dropping data. While we wait for the ability to
parse conditions in isolation
(open-telemetry#13545),
this change makes `filterottl` a little friendlier.

**Testing:** <Describe what testing was performed and which tests were
added.>
Updated unit tests

…y#27598)

currently the charts are hard to read and the data file is 32mb in size.
100 data points per chart seem enough.

Signed-off-by: Alex Boten <[email protected]>

…ry#27600)

Bump github.com/DataDog/sketches-go from 1.4.2 to 1.4.3 in
/exporter/datadogexporter
Bump
github.com/GoogleCloudPlatform/opentelemetry-operations-go/detectors/gcp
from 1.19.1 to 1.20.0 in /exporter/datadogexporter
Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common
from 1.0.761 to 1.0.762 in /exporter/tencentcloudlogserviceexporter
Bump golang.org/x/net from 0.16.0 to 0.17.0 in
/exporter/azuremonitorexporter
Bump golang.org/x/net from 0.16.0 to 0.17.0 in /internal/aws/awsutil
Bump golang.org/x/net from 0.16.0 to 0.17.0 in /receiver/jaegerreceiver

Fixing a typo on the main README.md file:
please following the process -> please follow the process

The following commands were run to prepare this release:
- make chlog-update VERSION=v0.87.0
- sed -i.bak s/0.86.0/0.87.0/g versions.yaml
- make multimod-prerelease
- make multimod-sync

**Description:**
Remove the parquet exporter

**Link to tracking Issue:**
Fixes open-telemetry#27284

…etry#27235)

**Description:** Functions to modify matched text during replacement can
now be passed as optional arguments to the following Editors:
- replace_pattern
- replace_all_patterns
- replace_match
- replace_all_matches

**Documentation:**

https://github.com/rnishtala-sumo/opentelemetry-collector-contrib/blob/ottl-replace-pattern/pkg/ottl/ottlfuncs/README.md#replace_pattern

**Issue:**
Resolves
open-telemetry#22787

**Description:**

Add validation of protocol name and be case insensitive

**Link to tracking Issue:**

**Testing:**

Unit Tests

**Documentation:**

N/A

---------

Signed-off-by: Dominik Rosiek <[email protected]>

…ion (open-telemetry#27480)

open-telemetry#27481)

Linked to open-telemetry#25190

…r.Factory and pass checkapi (open-telemetry#27077)

Related to: open-telemetry#26304

…oudwatchlogs exporter (open-telemetry#26692)

Adds support to the to parallelism in the
awscloudwatchlogs exporter by leveraging the [exporter
helper](https://github.com/open-telemetry/opentelemetry-collector/blob/main/exporter/exporterhelper/README.md).

In this PR, we are adding support to the `num_consumers` configuration
in the `sending_queue`. This will allow users to specify the number of
consumers that will consume from the sending_queue in parallel.

It is possible and straightforward to use this approach because
CloudWatch logs [no longer requires that you use a token to control
access to the stream that you are writing
to](https://aws.amazon.com/about-aws/whats-new/2023/01/amazon-cloudwatch-logs-log-stream-transaction-quota-sequencetoken-requirement/).
You can write to the same stream in parallel.

To achieve this, this PR does the following:
* Create Pusher that is able to push to multiple streams at the same
time.
* Move lifecycle of the Pusher to the function that is used to consume
from the sending queue. This allows you to safely send to multiple
streams at the same time without any resource contention since each call
to consume logs will not share resources with others that are happening
in parallel (one exception is the creation of log streams).

Besides that I analyzed the code and removed other limitations:
* locks that were not necessary
* Limiter that was used to limit the number of requests per stream to 5
per second. [The TPS is much higher now and is per
account.](https://aws.amazon.com/about-aws/whats-new/2023/01/amazon-cloudwatch-logs-log-stream-transaction-quota-sequencetoken-requirement/)

** How to review this PR: **

The first 3 commits in this PR were used to refactor the code before
making the real changes. Please use the commits to simplify the review
process.

**Link to tracking Issue:** open-telemetry#26360

**Testing:**

- Unit tests were added.
- Tested locally sending logs to cloudwatch logs.

**Documentation:** Documentation was added describing the new
parameters.

---------

Signed-off-by: Raphael Silva <[email protected]>
Co-authored-by: Anthony Mirabella <[email protected]>

…pen-telemetry#23447)

Adding a feature: Making `maxBatchByteSize` a configurable parameter.
This would allow users to adjust it based on the capabilities of their
specific remote storage, offering more flexibility and potentially
improving performance.
Example:
```yaml
exporters:
  prometheusremotewrite:
    endpoint: "https://my-cortex:7900/api/v1/push"
    max_batch_byte_size: 5000000
```

Fixes open-telemetry#21911 

**Testing:** <Describe what testing was performed and which tests were
added.>
Added `MaxBatchByteSize` to `TestLoadConfig(t *testing.T)` in
`config_test.go`

**Documentation:** <Describe the documentation added.>
Added to `README.md`:
- `max_batch_byte_size` (default = `3000000` -> `~2.861 mb`): Maximum
size of a batch of
samples to be sent to the remote write endpoint. If the batch size is
larger
  than this value, it will be split into multiple batches.

---------

Co-authored-by: Alex Boten <[email protected]>

…pen-telemetry#27625)

**Description:** v0.87.0 `exporterhelper.queue_settings.validate()`
verifies the consumer setting for us. We no longer need to duplicate
that validation.

**Link to tracking Issue:** <Issue number if applicable> open-telemetry#27624

…#27434)

Incremental step towards decoupling the Reader struct.

This also clarifies and corrects the behavior of the `ValidateOrClose`
function.

This addresses https://pkg.go.dev/vuln/GO-2023-2102

Signed-off-by: Alex Boten <[email protected]>

open-telemetry#27616)

**Description:** 

This resource attribute was deprecated in prior release (
open-telemetry#27584)
so I am removing it.

**Link to tracking Issue:**

open-telemetry#26487

…elemetry#27569)

**Description:** Adds an e2e test for the k8sobjectsreceiver to github
workflow
<!--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:**
[18395](open-telemetry#18395)

…#27622)

**Description:**  Remove stale go mod replace for datadog-agent and upgrade to the released version.

**Link to tracking Issue:**  Fixes open-telemetry#25119.

…try#27634)

Remove translation dropping the `opencensus.resourcetype` resource
attribute as it's not emitted by k8s receivers anymore. The last one was
removed in
open-telemetry#27616

**Documentation:** <Describe the documentation added.>
Found a typo while reading the transform processor readme.

…open-telemetry#27272)

**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.-->
Added support for  Exporter Helper configuration. 
**Link to tracking Issue:** open-telemetry#24329 

**Testing:** <Describe what testing was performed and which tests were
added.> Added tests and manually tested with e2e scenarios

---------

Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Ziqi Zhao <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>

open-telemetry#27485)

**Description:** <Describe what has changed.>

add k8s.pod.qos_class optional resource attriute

**Link to tracking Issue:** <Issue number if applicable>

open-telemetry#27483

**Testing:** <Describe what testing was performed and which tests were
added.>
- updated unit tests

**Documentation:** <Describe the documentation added.>

- generated

**Description:** This component will export markers to be consumed by
the Honeycomb Markers API to highlight user events based initially on
preset configurations.

**Link to tracking Issue:** open-telemetry#26653 

**Testing:** Unit testing for factory and config

**Documentation:** README describing component usage

---------

Co-authored-by: Tyler Helmuth <[email protected]>

**Description:** 
It's not obvious from the readme that you won't get all metrics listed
in metadata by default.
Suggest small doc update to make it clearer. 

**Link to tracking Issue:** 

**Testing:** Observed with running collector.

**Documentation:** See above.

---------

Co-authored-by: Daniel Jaglowski <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>

…24183)

2nd step for the deprecation of `container.cpu.percent`

According to the deprecation plan in the
[docs](https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/v0.79.0/receiver/dockerstatsreceiver#transition-to-cpu-utilization-metric-name-aligned-with-opentelemetry-specification),
this PR disables the old metric by default, to be released in v0.83.0

tracking issue:
open-telemetry#21807

---------

Co-authored-by: Christian <[email protected]>

…pen-telemetry#27646)

**Description:**
Fix mask when multiple patterns exist

With following input:

```
attr: <secret1> <secret2>
```

and config:

```
redaction:
  blocked_values:
    - '<secret1>'
    - '<secret2>'
```

Output before fix:

```
attr: <secret1> ****
```

Output after fix:

```
attr: **** ****
```

**Link to tracking Issue:** <Issue number if applicable>

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>

GHSA-qppj-fm5r-hxr3

https://github.com/grpc/grpc-go/releases/tag/v1.58.3

---------

Signed-off-by: Pavol Loffay <[email protected]>

Resolves open-telemetry#27640

---------

Co-authored-by: Paulo Janotti <[email protected]>

…pen-telemetry#27459)

Description: Exposes bbolt fsync as a configuration option

Link to tracking Issue:
[20266](open-telemetry#20266)

Testing: Manual Testing, Updated unit tests for factory and client

Documentation: Added change-log and documentation comments in config.go

---------

Co-authored-by: Daniel Jaglowski <[email protected]>

…-telemetry#27658)

**Description:**

Cache the publisher event to:

1. Avoid logging the same error message every time one event from the
given source is logged.
2. Avoid opening and closing the event publisher for every single event.

**Link to tracking Issue:**

[Item 4 described on the
investigation](open-telemetry#21491 (comment))
for issue open-telemetry#21491.

**Testing:**

* Go tests for `pkg/stanza` and `receiver/windowseventlogreceiver` on
Windows box.
* Ran the contrib build locally to validate the change.
* Can't run the full make locally: misspell is failing on Windows
because the command line is too long.

**Documentation:**

Let me know if changing the severity of the log message requires a
changelog update.

**Description:**
Adding myself as owner of `windowseventlogreceiver` per invite
open-telemetry#27658 (comment)

cc @djaglowski

…tings.enabled flag (open-telemetry#27592)

**Description:** Previously the remote write exporter would incorrectly
retry if `retrySettings.enabled` was set to false.

**Testing:** Unit tests

…emetry#27678)

**Description:** 
The `system` detector extracts all the `cpu` info from the system even
if you disable the configs and I believe this is where the bug kicks in.
Disabling the settings will only stop it from setting the resource
attributes.

The
[library](https://github.com/shirou/gopsutil/blob/v3.23.9/cpu/cpu_windows.go#L113)
that we rely on doesn't extract some attributes for Windows OS (in this
case, the field `cpu.Model`) and it leaves this field empty. This
results in a bug when we try to parse an empty string.

The long-term fix will be to extract `cpu.Model` in `gopsutil` upstream
library.

**Link to tracking Issue:**
open-telemetry#27675

Related issue:
open-telemetry#20552

Tweak the mock-backend to do following:

- Receives data from the receiver.
- Returns errors randomly to our receiver, which attempts to resend/drop
the data.

This is helpful when we're required to test random behaviors of the
collector and ensure reliable data delivery.


This is my initial PR to expand the testbed. This will help my further
efforts to expand the testbed.
Myself and @omrozowicz-splunk plan on adding `sending_queue` support to
the testbed and expanding the testing capabilities.

---------

Co-authored-by: Daniel Jaglowski <[email protected]>

…elemetry#27464)

**Description:**

Adding integration tests for syslog exporter (and syslog receiver) and
fixing bugs which has been found during the process

**Link to tracking Issue:** open-telemetry#21245 

**Testing:**

Integration tests and more unit tests

**Documentation:**

N/A

---------

Signed-off-by: Dominik Rosiek <[email protected]>
Co-authored-by: Daniel Jaglowski <[email protected]>

…try#27684)

**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.-->
Logs and traces are not supported telemetry types for the file receiver,
and the collector will fail to start if it the receiver is in included
in a pipeline of either of these types. This change is to correct the
README to properly reflect this.

**Link to tracking Issue:** <Issue number if applicable>
Resolves open-telemetry#27682

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>

)

running `make gengithub` or `make update-codeowners` with an incorrect
token or without setting the token throws the same following error:
```https://api.github.com/orgs/open-telemetry/members?per_page=50: 401 Bad credentials []```

It makes sense to throw an error if the user has forgotten to set the GITHUB_TOKEN variable and it also distinguishes between the two cases (incorrect token and token not set)

This has been causing ambiguous imports all over the place for some
time.

Signed-off-by: Alex Boten <[email protected]>