Skip to content

Conversation

svenfoo
Copy link
Contributor

@svenfoo svenfoo commented May 20, 2025

…ntrack errors

Pass the correct failed count when calling errors.AddPartial().

@svenfoo svenfoo requested review from a team, braydonk and dmitryax as code owners May 20, 2025 14:54
Copy link

linux-foundation-easycla bot commented May 20, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: svenfoo / name: Sven Neumann (d8056f5)

@atoulme
Copy link
Contributor

atoulme commented May 20, 2025

Thank you for catching this bug. Please fill the CLA, and create a changelog entry by following the command make chlog-new from the root of your checkout. You can enter the PR number as issue number. Once you have performed those steps, please mark the PR as ready for review and we will trigger CI again.

…ntrack errors

Pass the correct failed count when calling errors.AddPartial().
@svenfoo svenfoo force-pushed the sven/network-scraper-failed-count branch from c4cb316 to d8056f5 Compare May 21, 2025 14:27
@svenfoo svenfoo marked this pull request as ready for review May 21, 2025 14:50
Copy link
Contributor

github-actions bot commented Jun 5, 2025

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Jun 5, 2025
@svenfoo
Copy link
Contributor Author

svenfoo commented Jun 5, 2025

Did I miss a particular step or why does this PR not get any further review?

Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Jun 20, 2025
@svenfoo
Copy link
Contributor Author

svenfoo commented Jun 20, 2025

@atoulme Is there anything else you'd like me to do? As far as I can see this PR is ready for review.

@braydonk braydonk added ready to merge Code review completed; ready to merge by maintainers and removed Stale waiting-for-code-owners labels Jun 20, 2025
@songy23 songy23 merged commit 90c0415 into open-telemetry:main Jun 20, 2025
214 checks passed
@github-actions github-actions bot added this to the next release milestone Jun 20, 2025
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
open-telemetry#40175)

…ntrack errors

Pass the correct failed count when calling `errors.AddPartial()`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/hostmetrics/internal/scraper/network

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants