Skip to content
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

Move scraper helpers to its own package scraperhelper #2185

Merged
merged 1 commit into from
Nov 20, 2020

Conversation

bogdandrutu
Copy link
Member

Updates #2036

Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com

@bogdandrutu bogdandrutu requested a review from a team November 19, 2020 22:56
@codecov
Copy link

codecov bot commented Nov 19, 2020

Codecov Report

Merging #2185 (19d9090) into master (1f3f91f) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2185      +/-   ##
==========================================
- Coverage   92.01%   91.99%   -0.03%     
==========================================
  Files         270      270              
  Lines       15784    15756      -28     
==========================================
- Hits        14524    14494      -30     
- Misses        858      859       +1     
- Partials      402      403       +1     
Impacted Files Coverage Δ
receiver/scraperhelper/errors.go 100.00% <ø> (ø)
receiver/scraperhelper/scraper.go 100.00% <ø> (ø)
receiver/hostmetricsreceiver/factory.go 86.84% <100.00%> (ø)
...icsreceiver/internal/scraper/cpuscraper/factory.go 100.00% <100.00%> (ø)
...csreceiver/internal/scraper/diskscraper/factory.go 100.00% <100.00%> (ø)
...iver/internal/scraper/filesystemscraper/factory.go 88.88% <100.00%> (ø)
...al/scraper/filesystemscraper/filesystem_scraper.go 100.00% <100.00%> (ø)
...csreceiver/internal/scraper/loadscraper/factory.go 100.00% <100.00%> (ø)
...receiver/internal/scraper/memoryscraper/factory.go 100.00% <100.00%> (ø)
...eceiver/internal/scraper/networkscraper/factory.go 100.00% <100.00%> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f3f91f...19d9090. Read the comment docs.

Copy link
Member

@james-bebbington james-bebbington left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider if it would it make sense to nest scraperhelper under receiverhelper

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@bogdandrutu bogdandrutu merged commit e365c9b into open-telemetry:master Nov 20, 2020
@bogdandrutu bogdandrutu deleted the scraperhelper branch January 13, 2021 18:47
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
open-telemetry#2185)

Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.39.1 to 1.40.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.39.1...v1.40.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
…elemetry#2185)

- setting partitions_auto_discovery_interval to 1 causes a storm of
  CommandPartitionedTopicMetadata requests to brokers. The broker doesn't log these
  requests and it's hard to detect the issue.
  The default value in the client is 1 minute which is 60*10^9.
  Set the example value to the default.

- setting batching_max_publish_delay to 10 will make batching uneffective
  since the unit is nanoseconds. The default value in the client is 10 milliseconds
  which is 10^7. Set the example value to the default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants