-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Migrate Prometheus receiver metrics to obsreport package #617
Migrate Prometheus receiver metrics to obsreport package #617
Conversation
@@ -124,7 +124,8 @@ func (b *metricBuilder) AddDataPoint(ls labels.Labels, t int64, v float64) error | |||
return b.currentMf.Add(metricName, ls, t, v) | |||
} | |||
|
|||
// Build is to build an opencensus data.MetricsData based on all added data complexValue | |||
// Build is to build an opencensus data.MetricsData based on all added data complexValue. | |||
// The only error returned by this function is errNoDataToBuild. |
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.
Later we should look at returning a bool instead of an error but didn't want to make a bigger change now.
github.com/open-telemetry/opentelemetry-proto v0.0.0-20200211051721-ff5f19c6217d h1:hZcHR0at6tb3jBjaPHlfLr6yK7rTrA8xGCS6jlUSLcU= | ||
github.com/open-telemetry/opentelemetry-proto v0.0.0-20200211051721-ff5f19c6217d/go.mod h1:PMR5GI0F7BSpio+rBGFxNm6SLzg3FypDTcFuQZnO+F8= | ||
github.com/open-telemetry/opentelemetry-proto v0.0.0-20200227223449-a65b867510c0/go.mod h1:PMR5GI0F7BSpio+rBGFxNm6SLzg3FypDTcFuQZnO+F8= | ||
github.com/open-telemetry/opentelemetry-proto v0.0.0-20200308012146-674ae1c8703f h1:o21WlujGsrjoN3+n99AflASF/K0S6+SOLOj2O9nNplc= |
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.
This was updated on my env for a full build, took the chance to run go mod tidy
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.
Wouldn't it be better to send a separate PR for that?
This is the minimal change to migrate the Prometheus metrics to the obsreport package (capable of generating both new and legacy metrics).
4111f60
to
65bb96c
Compare
* othttp: add WithSpanFormatter option * plugin/othttp: add span formatter test * remove typo * preserve operation && change option name * nil check * fix comment typo Co-Authored-By: Rahul Patel <rghetia@yahoo.com> * add default formatter test case Co-authored-by: Rahul Patel <rghetia@yahoo.com>
…y#617) Bumps [boto3](https://github.com/boto/boto3) from 1.18.17 to 1.18.18. - [Release notes](https://github.com/boto/boto3/releases) - [Changelog](https://github.com/boto/boto3/blob/develop/CHANGELOG.rst) - [Commits](boto/boto3@1.18.17...1.18.18) --- updated-dependencies: - dependency-name: boto3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Description: This is a minimal change to migrate the Prometheus metrics to the obsreport package (capable of generating both new and legacy metrics).
Link to tracking Issue: Related to #141
Testing: Manual verification via docker-compose
Documentation: N/A