-
Notifications
You must be signed in to change notification settings - Fork 933
feat(config): extract declarative configuration into opentelemetry-configuration package #5356
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
Merged
xrmx
merged 29 commits into
open-telemetry:main
from
MikeGoldsmith:mike/extract-sdk-configuration
Jul 10, 2026
Merged
Changes from 8 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
949fd69
extract declarative configuration into opentelemetry-sdk-configuratio…
MikeGoldsmith d7ba5da
rename changelog fragment to PR #5356
MikeGoldsmith a40d653
Merge remote-tracking branch 'upstream/main' into mike/extract-sdk-co…
MikeGoldsmith 3c90d9c
register opentelemetry-sdk-configuration in uv workspace
MikeGoldsmith 4f62d78
fix CI: regenerate workflows, mock new pkg in routing tests, silence …
MikeGoldsmith 9eba9f6
fix CI: per-package pylintrc for sdk-configuration, drop bad link fro…
MikeGoldsmith 3cde032
drop file-configuration extras: pyyaml and jsonschema are now require…
MikeGoldsmith 1b4f949
tighten 5356 changelog entry
MikeGoldsmith bf27ea4
merge upstream/main; prefix 5356 changelog entry with package name
MikeGoldsmith 7641371
drop opentelemetry/sdk/__init__.pyi to make opentelemetry.sdk a prope…
MikeGoldsmith ac255fe
rename opentelemetry-sdk-configuration → opentelemetry-configuration …
MikeGoldsmith 9b7795b
post-rename CI fixes: ruff import sort + disable no-member on dynamic…
MikeGoldsmith e76b325
use class-based fake module in routing tests to drop the no-member py…
MikeGoldsmith 5877b3e
silence pylint no-name-in-module on conditional opentelemetry.configu…
MikeGoldsmith 45eacc2
add opentelemetry-configuration to pyright include/exclude, drop unus…
MikeGoldsmith 4b2e2df
loosen pyyaml floor to >= 5.4 (matches our actual API surface)
MikeGoldsmith f002080
Merge branch 'main' into mike/extract-sdk-configuration
herin049 2f19ee6
Merge branch 'main' into mike/extract-sdk-configuration
MikeGoldsmith 074705c
Merge remote-tracking branch 'upstream/main' into mike/extract-sdk-co…
MikeGoldsmith 92fffcf
Merge remote-tracking branch 'upstream/main' into mike/extract-sdk-co…
MikeGoldsmith 6ca3e0d
Merge remote-tracking branch 'origin/mike/extract-sdk-configuration' …
MikeGoldsmith 7901696
Merge branch 'main' into mike/extract-sdk-configuration
MikeGoldsmith 44f70eb
Merge remote-tracking branch 'upstream/main' into mike/extract-sdk-co…
MikeGoldsmith 6e7dacb
Merge remote-tracking branch 'origin/mike/extract-sdk-configuration' …
MikeGoldsmith 2f90eac
Merge remote-tracking branch 'upstream/main' into mike/extract-sdk-co…
MikeGoldsmith 49422bb
Merge branch 'main' into mike/extract-sdk-configuration
MikeGoldsmith 10d51fe
Merge remote-tracking branch 'upstream/main' into mike/extract-sdk-co…
MikeGoldsmith 5389687
Merge remote-tracking branch 'origin/mike/extract-sdk-configuration' …
MikeGoldsmith 9f69fe7
address xrmx review: pin to 0.65b0.dev via prerelease block, keep fil…
MikeGoldsmith File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Declarative configuration moves from `opentelemetry.sdk._configuration` into the new public `opentelemetry-sdk-configuration` package (`opentelemetry.sdk.configuration` namespace), published experimentally and versioned independently of the SDK. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| # Pylint config for opentelemetry-sdk-configuration. Extends the repo root | ||
| # `.pylintrc` (which pylint discovers by walking upward from each file) and | ||
| # adds disables that only apply to this package. | ||
| # | ||
| # `opentelemetry-sdk-configuration` extends the `opentelemetry.sdk.*` | ||
| # namespace owned by `opentelemetry-sdk`. At runtime PEP 420 namespace | ||
| # package merging makes `opentelemetry.sdk.configuration.*` resolvable, but | ||
| # astroid (pylint's resolver) does not follow editable-install namespace | ||
| # extensions across separate distributions, so it reports false positive | ||
| # `import-error` / `no-name-in-module` on every import that crosses the | ||
| # package boundary. These two messages are silenced for this package only. | ||
|
|
||
| [MASTER] | ||
| # Skip the codegen'd models.py (datamodel-code-generator output). | ||
| ignore-patterns=^models\.py$ | ||
| # Run with multiple processes. | ||
| jobs=0 | ||
| # Match the global pylintrc's no-self-use extension. | ||
| load-plugins=pylint.extensions.no_self_use | ||
|
|
||
| [MESSAGES CONTROL] | ||
| disable=missing-docstring, | ||
| fixme, | ||
| too-few-public-methods, | ||
| too-many-instance-attributes, | ||
| too-many-arguments, | ||
| too-many-positional-arguments, | ||
| duplicate-code, | ||
| ungrouped-imports, | ||
| wrong-import-order, | ||
| line-too-long, | ||
| exec-used, | ||
| super-with-arguments, | ||
| isinstance-second-argument-not-valid-type, | ||
| raise-missing-from, | ||
| unused-argument, | ||
| redefined-builtin, | ||
| cyclic-import, | ||
| import-error, | ||
| no-name-in-module, |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.