David Ashpole
David Ashpole
Feedback from https://github.com/etcd-io/etcd/pull/12919#discussion_r628085064. Globals are nice for simpler examples, but it is often better for larger projects to avoid them so libraries are isolated from each other. My suggestion would...
### Problem Statement I'd like to be able to send metrics from OpenCensus and OpenTelemetry SDKs to the same prometheus endpoint. This will allow me to migrate my OpenCensus metrics...
**Description:** Part of https://github.com/open-telemetry/opentelemetry-collector/issues/5550. NewThrottleRetry is an error that is similar in kind to a permanent error: it informs callers how to deal with retries. I think it makes the...
**Description:** The intention of the check for no resource attributes was to only produce the `target_info` metric when it provides meaningful additional attributes. However, we currently (without this change) will...
**Description:** This is just a small additional piece of validation to make sure our interpretation of Prometheus histograms is correct. Otherwise, our resulting histogram will place points that are in...
**Description:** Prometheus histograms don't have a sum if the histogram includes negative observations. Now that the protocol supports a notion of "unset" sums, use this in prometheus components to correctly...
## Background Currently, resource detection for GKE and EKS is done in separate packages ([EKS](https://github.com/open-telemetry/opentelemetry-go-contrib/tree/main/detectors/aws/eks), [GKE](https://github.com/open-telemetry/opentelemetry-go-contrib/blob/main/detectors/gcp/gke.go)), and have diverged in what they detect (container id vs container name, pod name,...
Part of https://github.com/open-telemetry/opentelemetry-specification/issues/1175 Related to https://github.com/open-telemetry/oteps/pull/210 ## Changes 1. Add a MetricProducer interface to the metrics SDK. It is an optional argument when creating a MetricReader meant to support non-SDK...
Fixes https://github.com/open-telemetry/opentelemetry-specification/issues/2682 ## Changes Add the `short_name` scope attribute. Alternatively, it could have been added as `otel.short_name` to distinguish it from non-otel attributes, or `prometheus.namespace` to make it explicitly associated...
Related: https://github.com/open-telemetry/opentelemetry-specification/issues/1906 This is a second attempt at https://github.com/open-telemetry/opentelemetry-specification/pull/2422. Depends on https://github.com/open-telemetry/opentelemetry-specification/pull/2702 ## Changes 1. Prevent Prometheus naming collisions between instrumentation libraries by adding the short_name as a prefix to...