open-telemetry / open-telemetry/opentelemetry-python
Pull Request Dashboard
Open
Nobody has claimed this yet.
dashboard
- Dominant language
- Python
- Stars
- 2.6k
- Forks
- 1k
- Avg merge
- 4d 15h
- Merged PRs (30d)
- 19
Description
[!NOTE]
Open non-draft PRs grouped by who is expected to act next. Draft PRs are listed separately. The grouping is partly performed by an LLM (source) and could contain mistakes.Reviewers column: ✅ active approval · ✔️ active approval (non-code-owner) · ⏳ review pending · 💬 review thread needs author action · 📌 top-level feedback needs author action · 🔴 changes requested.
CI column: ✅ passing · ⏳ running · ❌ failing · 🔐 workflow action required.
Waiting on maintainers
| PR | Author | Reviewers | CI | Conflicts | Age |
|---|---|---|---|---|---|
| #5365 feat(config): wire top-level attribute_limits into per-signal providers | ocelotl | aabmass ⏳ herin049 ✅ rads-1996 ✔️ xrmx |
✅ | ✅ | 43d |
| #5448 opentelemetry-exporter-otlp-proto-grpc: tolerate invalid timeout env vars | Eason09053360 | herin049 ocelotl ✅ |
✅ | ✅ | 24d |
| #5618 Make processor queue capacity metrics observable | cijothomas | herin049 ✅ | ✅ | ✅ | 14d |
| #5616 Fix metrics collection timeout so slow observable callbacks raise MetricsTimeoutError | henry3260 | ocelotl ✅ | ✅ | ✅ | 14d |
| #5438 opentelemetry-sdk: unify logging force_flush timeout defaults to 30000ms | avinab-neogy | carlosalberto ✔️ lzchen ✅ pmcollins |
✅ | ✅ | 10d |
| #5643 build(deps-dev): bump gitpython from 3.1.58 to 3.1.59 | app/dependabot | herin049 ✅ xrmx ✅ |
✅ | ✅ | 10d |
| #5631 fix(sdk): raise error when named view specifies no instrument name (#2815) | dlowzzxx | herin049 ✅ | ✅ | ✅ | 5d |
| #5653 Add host.id to host resource attributes | jaydeluca | emdneto ✅ herin049 ✅ lzchen ✅ xrmx ⏳ |
✅ | ✅ | 5d |
| #5557 Strip only the leading uberctx- prefix from a Jaeger baggage key | serhiy-bzhezytskyy | carlosalberto ✔️ lzchen ✅ ocelotl ✅ xrmx ✅ |
✅ | ✅ | 4d |
| #5286 fix(config): align additional_properties typing | hariharan077 | lzchen ✅ MikeGoldsmith ⏳🔴 |
✅ | ✅ | 4d |
| #5613 Keep spans mutable during on_ending span processor callbacks | henry3260 | lzchen ✅ ocelotl ✅ |
✅ | ✅ | 3d |
| #5402 opentelemetry-codegen-json: support decoding enums from names | herin049 | DylanRussell ✅ | ✅ | ✅ | 3d |
| #5411 opentelemetry-exporter-prometheus: add preferred_aggregation parameter to PrometheusMetricReader | herin049 | tammy-baylis-swi ✅ | ✅ | ✅ | 3d |
| #5419 opentelemetry-sdk: add support for metric producers | herin049 | DylanRussell ✅ | ✅ | ✅ | 3d |
| #5668 opentelemetry-exporter-otlp-proto-grpc: fix incorrect default port for gRPC internal metrics | herin049 | lzchen ✅ | ✅ | ✅ | 2d |
| #5669 opentelemetry-configuration: add configuration support for OTLP JSON exporters | herin049 | lzchen ✅ | ✅ | ✅ | 2d |
| #5332 Add support for excluding attributes in metrics View | saurabh-saraswat | herin049 ⏳🔴 lzchen ✅ xrmx ⏳🔴 |
✅ | ✅ | 2d |
Waiting on reviewers
| PR | Author | Reviewers | CI | Conflicts | Age |
|---|---|---|---|---|---|
| #5510 build(deps): bump cryptography from 49.0.0 to 50.0.0 | app/dependabot | ✅ | ✅ | 45d | |
| #5538 docs: update pre-fork server guidance | AG0708 | ✅ | ✅ | 37d | |
| #5541 chore(deps): update all patch versions | app/renovate | ❌ | ✅ | 37d | |
| #5542 chore(docs): update docs deps | app/renovate | xrmx | ❌ | ❌ | 37d |
| #5549 chore(deps): update dependency asgiref to v3.12.1 | app/renovate | ❌ | ✅ | 35d | |
| #5554 Ignore reader preferred aggregations incompatible with async instruments | Eason09053360 | ✅ | ✅ | 32d | |
| #5561 chore(deps): refresh pip-compile outputs | app/renovate | ❌ | ✅ | 29d | |
| #5595 chore(deps): update dependency protobuf to v5 [security] | app/renovate | ❌ | ✅ | 22d | |
| #5569 Concurrent multi-processors never flush at interpreter exit | dwin-gharibi | ✅ | ✅ | 21d | |
| #5567 W3C Baggage propagator uses form encoding instead of percent encoding | dwin-gharibi | ✅ | ✅ | 21d | |
| #5571 record_min_max=False exports histogram min=+Infinity and max=-Infinity | dwin-gharibi | ✅ | ✅ | 21d | |
| #5598 Fix crashes from unguarded str()/repr() on invalid attribute keys/values | piyush182004 | ✅ | ✅ | 21d | |
| #5601 chore(deps): update dependency charset-normalizer to v3.5.1 | app/renovate | ✅ | ✅ | 19d | |
| #5603 chore(deps): update dependency decorator to v5.3.1 | app/renovate | ✅ | ✅ | 19d | |
| #5604 chore(deps): update dependency google-api-core to v2.38.0 | app/renovate | ❌ | ✅ | 19d | |
| #5183 sdk: tighten ReadableSpan.attributes to non-Optional Mapping (#4569) | WatchTree-19 | emdneto MikeGoldsmith 🔴 tekumara ✔️ |
✅ | ✅ | 18d |
| #5589 ci: restrict checkout credential persistence | opentelemetrybot | emdneto | ✅ | ✅ | 18d |
| #5611 build(deps): bump thrift from 0.13.0 to 0.24.0 in /docs/examples/fork-process-model/flask-gunicorn | app/dependabot | ✅ | ✅ | 17d | |
| #5612 chore(docs): update dependency thrift to v0.24.0 [security] | app/renovate | ✅ | ✅ | 17d | |
| #5615 build(deps): bump thrift from 0.13.0 to 0.24.0 in /docs/examples/fork-process-model/flask-uwsgi | app/dependabot | ✅ | ✅ | 16d | |
| #5630 Warn on conflicting metric identities between views matching the same instrument | Eason09053360 | ✅ | ✅ | 13d | |
| #5634 chore(deps): update dependency google-auth to v2.58.0 | app/renovate | ❌ | ✅ | 12d | |
| #5635 chore(deps): update dependency googleapis-common-protos to v1.75.3 | app/renovate | ❌ | ✅ | 12d | |
| #5632 fix(sdk): check default-view fallback stream for conflicting metric identities | somuai | ✅ | ✅ | 11d | |
| #5626 fix(sdk): catch and log unhandled exceptions in MultiSpanProcessor | somuai | ✅ | ✅ | 9d | |
| #5650 fix(sdk): add validation for export_timeout_millis <= 0 in BatchSpanP… | Dotify71 | ✅ | ✅ | 6d | |
| #5654 fix(exporter): warn when OTLP gRPC endpoint includes a path | Shriprasad-P | lzchen | ✅ | ✅ | 4d |
| #5444 opentelemetry-sdk: add support for advisory attributes on metric instruments | herin049 | tammy-baylis-swi | ✅ | ✅ | 3d |
| #5450 opentelemetry-api: implement metrics bind API | herin049 | ✅ | ✅ | 3d | |
| #5529 docs: clarify baggage current context behavior | goutamadwant | ✅ | ✅ | 2d | |
| #5673 fix(sdk): detect character pattern wildcards in View instrument_name (#5664) | somuai | ✅ | ✅ | 20h | |
| #5675 test(sdk): fix flaky BatchProcessor shutdown test (#5663) | Anurag-M1 | ✅ | ✅ | 18h | |
| #5674 fix(sdk): detect character pattern wildcards in View instrument_name (#5664) | Dotify71 | ✅ | ✅ | 6h |
Waiting on authors
| PR | Author | Reviewers | CI | Conflicts | Age |
|---|---|---|---|---|---|
| #4822 Fix Django autoinstrumentation not producing span in STDOUT output | BrianPetkovsek | ocelotl 💬📌 tammy-baylis-swi ⏳💬📌 xrmx 💬📌🔴 |
✅ | ✅ | 295d |
| #4995 fix(otlp-http): reset requests session after fork | dshivashankar1994 | DylanRussell emdneto 📌 MikeGoldsmith ⏳🔴 ocelotl 📌 tammy-baylis-swi ⏳ themavik 📌 xrmx ⏳ |
✅ | ❌ | 184d |
| #4767 fix(metrics): Add fork-safety to SynchronousMeasurementConsumer | dshivashankar1994 | emdneto 📌 herin049 MikeGoldsmith 📌✅ ocelotl 📌 xrmx |
⏳ | ❌ | 169d |
| #5085 fix(sdk): force_flush returns meaningful bool on MetricReader | ravitheja4531-cell | emdneto ⏳📌🔴 herin049 ⏳ IndiTheo ⏳📌 lzchen 📌 MikeGoldsmith ⏳ tammy-baylis-swi 📌 |
❌ | ✅ | 157d |
| #5342 docs: clarify view aggregation precedence | naveenvenk17 | aabmass 💬✅ lzchen 📌✅ |
✅ | ✅ | 88d |
| #5337 opentelemetry-process-context: implement Resource publishing (OTEP-4719) | herin049 | aabmass 📌 ivoanjo lzchen ocelotl 📌✅ xrmx 💬 |
✅ | ❌ | 87d |
| #5341 fix: support sequence metric attributes in view aggregation keys | naveenvenk17 | lzchen 📌 tammy-baylis-swi 📌 xrmx 📌 |
✅ | ❌ | 73d |
| #5115 feat(logs): add OTEL_LOG_LEVEL support | grvmishra788 | aabmass carlosalberto 💬 emdneto herin049 ⏳ lzchen ✅ MikeGoldsmith ⏳🔴 pmcollins ✅ tammy-baylis-swi |
❌ | ❌ | 69d |
| #5507 docs: fix grammar in README Python Version Support section | Sdnsoumy | tammy-baylis-swi 📌 | ❌ | ✅ | 45d |
| #5530 Fix: Address-family mismatch confirmed: the shared mock server in test_otlp_e | M001N | ❌ | ❌ | 40d | |
| #5486 Implement declarative config ConfigProvider / ConfigProperties API | ocelotl | herin049 ⏳ lzchen ⏳ xrmx ⏳📌 |
❌ | ✅ | 29d |
| #5565 Self-referential attribute value crashes the instrumented application with RecursionError | dwin-gharibi | DylanRussell 💬✅ | ✅ | ✅ | 25d |
| #5579 Samplers never receive the parent's tracestate, and the composite sampler erases it | dwin-gharibi | xrmx 💬 | ✅ | ✅ | 23d |
| #5596 opentelemetry-sdk: handle RuntimeError during LoggingHandler.flush | ansaftab-1 | ❌ | ✅ | 21d | |
| #5590 Use OS entropy in RandomIdGenerator so random.seed() cannot repeat trace/span IDs | geekflyer | ocelotl 📌 | ✅ | ✅ | 18d |
| #5323 Add OTEL_SEMCONV_EXCEPTION_SIGNAL_OPT_IN support to record_exception | RKest | aabmass 💬 DylanRussell ✅ lmolkova 💬 lzchen ⏳ |
✅ | ✅ | 18d |
| #5610 opentelemetry-api: honor OTEL_PROPAGATORS set after import | BetterAndBetterII | ocelotl 💬 | ✅ | ✅ | 17d |
| #5622 fix(sdk): handle unmapped aggregations defensively in MetricReaderStorage.collect | dlowzzxx | ocelotl 📌🔴 | ✅ | ✅ | 14d |
| #5623 Fix #5550: Pull Request Dashboard | alienvisitor8675-bit | ocelotl 📌 | ❌ | ✅ | 14d |
| #5575 PeriodicExportingMetricReader discards its shutdown timeout | dwin-gharibi | ocelotl 💬📌🔴 | ✅ | ✅ | 9d |
| #5625 ci: resolve contrib repo SHA once for hermetic CI | mwimpelberg28 | emdneto 💬 | ✅ | ✅ | 8d |
| #5639 Assert on the storage under test, not on process-global mock state, in test_metric_reader_storage.py | ocelotl | hectorhdzg 💬✅ herin049 ✅ |
✅ | ✅ | 7d |
| #5641 Fail any test that leaks a PeriodicExportingMetricReader ticker thread, and fix the four that do | ocelotl | hectorhdzg 💬 herin049 ✅ |
✅ | ✅ | 7d |
| #5660 [opentelemetry-sdk] Fix overwriting of the service.instance.id which has been populated through the resource detectors | rads-1996 | Copilot 💬 herin049 ⏳💬 Shriprasad-P 📌 |
✅ | ✅ | 3d |
| #5656 fix(sdk): add validation for export_timeout_millis <= 0 in PeriodicExportingMetricReader | Dotify71 | herin049 ✅ Shriprasad-P 📌 |
⏳ | ✅ | 3d |
| #5667 Feature: Include trace flags in spans exported by the OTLPSpanExporter | gavalur-sfdc | davidji99 💬 herin049 💬📌✅ lzchen 💬 |
❌ | ✅ | 2d |
| #5665 [issue-2688] Populate Resource schema URL from semantic convention version | Rajkaran-122 | herin049 💬📌 | ✅ | ❌ | 2d |
| #5662 Apply the specified status precedence in Span.set_status | WatchTree-19 | Copilot 💬 herin049 💬 |
✅ | ✅ | 9h |
| #5672 opentelemetry-sdk: align resource detector schema url conflict behavor with other languages | herin049 | Copilot 💬 JoeyPatricio ✔️ |
✅ | ✅ | 9h |
Draft pull requests
| PR | Author | Draft age |
|---|---|---|
| #5378 fix(sdk): correctly merge exponential histogram buckets that start empty | ocelotl | 79d |
| #5394 Trace continuation strategy prototype 2 | xrmx | 74d |
| #5464 Prototype for span type OTEP | lmolkova | 53d |
| #4765 Implement filtering logic for min_severity and trace_based parameters (Waiting for logs stabilization) | rads-1996 | 50d |
| #5504 Pure-Python otlp-proto-common encoding (no protobuf) [2/4 RFC] | ocelotl | 47d |
| #5505 Pure-Python otlp-proto-http over stdlib urllib (drop protobuf + requests) [3/4 RFC] | ocelotl | 47d |
| #5506 Pure-Python otlp-proto-grpc (drop protobuf + grpcio) [4/4 RFC] | ocelotl | 47d |
| #5503 Pure-Python opentelemetry-proto (drop the protobuf dependency) [1/4 RFC] | ocelotl | 29d |
| #5599 fix(trace): reject tracestate keys with vendor part longer than 13 chars | sidsri14 | 20d |
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the dashboard.py source linked in the issue and inspect the generated Pull Request Dashboard content. The issue does not state a requested change; PR #5623 is identified as fixing it, so determine the intended scope and current status there before considering any work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100