Aaron Abbott

Results 51 issues of Aaron Abbott

I realized today that possibly large swaths of the codebase are not being checked by pylint. It seems like it is not recursing into directories that don't have an `__init__.py`...

bug
build & infra

Protobuf 5 was released but it currently conflicts with `opentelemetry-proto` package https://github.com/open-telemetry/opentelemetry-python/blob/762bd8f2629a1babc0eee4361a4873433511e836/opentelemetry-proto/pyproject.toml#L27-L29 This is by design since protobuf generated code is supposed to match the runtime protobuf library version. When...

priority:p2
proto

[From the spec](https://github.com/open-telemetry/opentelemetry-specification/tree/v1.6.1/specification/resource/semantic_conventions#:~:text=%5B3%5D%3A%20MUST%20be,for%20more%20recommendations) > [`service.instance.id`] MUST be unique for each instance of the same `service.namespace,service.name` pair (in other words `service.namespace,service.name,service.instance.id` triplet MUST be globally unique). The ID helps to distinguish...

bug

As described [in the spec](https://github.com/open-telemetry/opentelemetry-specification/blob/v1.33.0/specification/compatibility/prometheus_and_openmetrics.md#metric-attributes) > Metrics from OpenTelemetry with unsupported Attribute names MUST replace invalid characters with the `_` character. Multiple consecutive `_` characters MUST be replaced with a...

bug
exporters
prometheus

This flag was added as an opt-out mechanism in https://github.com/open-telemetry/opentelemetry-python/pull/3924. It should be removed after a few releases to migrate everyone to the spec compliant behavior

exporters
prometheus

The [Cardinality Limits section](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk.md#cardinality-limits) of the Metrics SDK spec says: > An overflow attribute set is defined, containing a single attribute `otel.metric.overflow` having (boolean) value `true`, which is used to...

spec:metrics
triage:deciding:community-feedback

Quick concept of BatchSpanProcessor implemented with asyncio https://github.com/open-telemetry/opentelemetry-python/issues/3274. Not nearly as fleshed out as https://github.com/open-telemetry/opentelemetry-python/pull/3485 but could be offered in addition to that This would work as a drop in...

See https://github.com/GoogleCloudPlatform/opentelemetry-operations-e2e-testing/pull/58 for details

# Description Part 1/2 for https://github.com/open-telemetry/opentelemetry-python/issues/3260 This only migrates the auto formatting capabilities. Linting will be in a separate PR which could use more scrutiny. Most of these changes look...

do not merge

**What are you trying to achieve?** Capture metrics which may be abnormal around the time my service stops. When my application exits or I otherwise call `MeterProvider.shutdown()`, I want to...

spec:metrics
triage:accepted:ready-with-sponsor