[herin049]

Description

Fixes bug where threads used for resource detection can be leaked for long running resource detectors.

Fixes #5697

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Tox

Does This PR Require a Contrib Repo Change?

• Yes. - Link to PR:
• No.

Checklist:

• Followed the style guidelines of this project
• Changelogs have been updated
• Unit tests have been added
• Documentation has been updated

[opentelemetry-pr-dashboard]

Pull request dashboard status

Merged · refreshed 2026-10-06 13:55 UTC

Status above doesn't look right?

• Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

[Copilot]

Copilot review overview

🟡 Changes recommended

shutdown(wait=True) can make initialization hang indefinitely, removing the documented timeout bound.

Review effort: Lite
Findings: 1

Open (1)

• shutdown(wait=True) defeats timeout and can hang provider initialization · New

What changed in this PR

Fixes leaked threads from long-running resource detectors by joining executor workers during resource initialization.

Changes:

• Updates executor shutdown behavior.
• Adds regression coverage for worker cleanup.
• Adds a changelog entry.

File	Summary
opentelemetry-sdk/​tests/​resources/​test_resources.py	Tests detector worker-thread cleanup.
opentelemetry-sdk/​src/​opentelemetry/​sdk/​resources/​__init__.py	Joins detector executor threads.
.changelog/​5706.fixed	Documents the thread-leak fix.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.