-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Deprecate all Scraper helpers in scraperhelper #11732
Conversation
@@ -63,14 +57,8 @@ type baseScraper struct { | |||
ScrapeFunc | |||
} | |||
|
|||
// NewScraper creates a scraper.Metrics that calls ScrapeMetrics at the specified collection interval, | |||
// reports observability information, and passes the scraped metrics to the next consumer. | |||
func NewScraper(scrape ScrapeFunc, options ...ScraperOption) (scraper.Metrics, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not breaking change because not released yet.
f130499
to
db7e301
Compare
type ScrapeFunc func(context.Context) (pmetric.Metrics, error) | ||
|
||
func (sf ScrapeFunc) ScrapeMetrics(ctx context.Context) (pmetric.Metrics, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not released.
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
db7e301
to
c0fec8a
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #11732 +/- ##
==========================================
- Coverage 91.61% 91.54% -0.08%
==========================================
Files 442 443 +1
Lines 23764 23764
==========================================
- Hits 21772 21754 -18
- Misses 1620 1637 +17
- Partials 372 373 +1 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
No description provided.