Add Structlog Processor as Instrumentation by JWinermaSplunk · Pull Request #4286 · open-telemetry/opentelemetry-python-contrib

JWinermaSplunk · 2026-03-03T21:22:02Z

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes open-telemetry/opentelemetry-python#2993

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?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

test_structlog.py

Does This PR Require a Core Repo Change?

Yes. - Link to PR:
No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

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

tammy-baylis-swi · 2026-03-19T20:35:56Z

Thank you for undertaking this!

tammy-baylis-swi · 2026-03-25T21:45:09Z

I think a rebase is needed. This PR is now reverting some unrelated docker-test fixes (this other PR)

JWinermaSplunk · 2026-03-25T21:49:51Z

I think a rebase is needed. This PR is now reverting some unrelated docker-test fixes (this other PR)

Believe I went ahead and rebased, but feel free to let me know if it still looks off!

dodofarm · 2026-04-18T00:27:07Z

Hey team,
any timeline on this PR? This is a great addition that we've been anticipating for quite some time!

pmcollins

Hey Josh, this instrumentation seems to work fine, but I added some comments.

github-actions · 2026-05-07T04:25:56Z

This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment.
If you're still working on this, please add a comment or push new commits.

JWinermaSplunk · 2026-05-07T17:04:14Z

commenting

emdneto · 2026-05-12T23:15:21Z

Thanks for the PR!

Just a heads-up: we no longer update CHANGELOG.md directly. The changelog is now generated from changelog fragments using Towncrier.

Please add the appropriate changelog fragment for this change instead of editing CHANGELOG.md manually. You can find the instructions and expected format in CONTRIBUTING.md.

github-actions · 2026-05-27T04:45:06Z

This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment.
If you're still working on this, please add a comment or push new commits.

# Conflicts: # .github/workflows/test_1.yml # .github/workflows/test_2.yml # .github/workflows/test_3.yml

JWinermaSplunk · 2026-07-02T18:10:45Z

Hi @xrmx,
Precommit was failing on merge from master, so I went ahead and reran and pushed. Should be good to go now.
Thank you!

xrmx · 2026-07-03T08:37:46Z

Hi @xrmx, Precommit was failing on merge from master, so I went ahead and reran and pushed. Should be good to go now. Thank you!

Puzzled by this since CI did run fine on main, anyway checked that tox -e precommit gives the very same result as yours. Thanks!

JWinermaSplunk marked this pull request as ready for review March 13, 2026 17:14

JWinermaSplunk requested a review from a team as a code owner March 13, 2026 17:14

JWinermaSplunk changed the title ~~[WIP] Add Structlog Handler as Instrumentation~~ Add Structlog Handler as Instrumentation Mar 13, 2026

github-project-automation Bot added this to Python PR digest Mar 13, 2026

JWinermaSplunk force-pushed the structlog-handler branch from e0317fa to fab1022 Compare March 13, 2026 17:20

tammy-baylis-swi reviewed Mar 19, 2026

View reviewed changes

Comment thread ...ntelemetry-instrumentation-structlog/src/opentelemetry/instrumentation/structlog/__init__.py Outdated

tammy-baylis-swi reviewed Mar 19, 2026

View reviewed changes

Comment thread ...ntelemetry-instrumentation-structlog/src/opentelemetry/instrumentation/structlog/__init__.py Outdated

tammy-baylis-swi reviewed Mar 19, 2026

View reviewed changes

Comment thread ...ntelemetry-instrumentation-structlog/src/opentelemetry/instrumentation/structlog/__init__.py Outdated

pmcollins reviewed Mar 19, 2026

View reviewed changes

Comment thread opentelemetry-instrumentation/src/opentelemetry/instrumentation/bootstrap_gen.py Outdated

Comment thread ...ntelemetry-instrumentation-structlog/src/opentelemetry/instrumentation/structlog/__init__.py Outdated

JWinermaSplunk force-pushed the structlog-handler branch from fd07c37 to ddae8b5 Compare March 19, 2026 23:11

xrmx moved this to Reviewed PRs that need fixes in Python PR digest Mar 25, 2026

tammy-baylis-swi reviewed Mar 25, 2026

View reviewed changes

Comment thread ...ntelemetry-instrumentation-structlog/src/opentelemetry/instrumentation/structlog/__init__.py

JWinermaSplunk force-pushed the structlog-handler branch from ef6ad9a to ece1f64 Compare March 25, 2026 21:48

tammy-baylis-swi reviewed Mar 25, 2026

View reviewed changes

Comment thread CHANGELOG.md Outdated

tammy-baylis-swi approved these changes Mar 25, 2026

View reviewed changes

JWinermaSplunk force-pushed the structlog-handler branch from 2648150 to 26c68f5 Compare April 20, 2026 16:50

pmcollins reviewed Apr 22, 2026

View reviewed changes

github-actions Bot added the Stale label May 7, 2026

github-actions Bot removed the Stale label May 8, 2026

github-actions Bot added the Stale label May 27, 2026

JWinermaSplunk force-pushed the structlog-handler branch from 4c55894 to d3ac0a4 Compare May 28, 2026 18:38

JWinermaSplunk changed the title ~~Add Structlog Handler as Instrumentation~~ Add Structlog Processor as Instrumentation May 28, 2026

JWinermaSplunk added 17 commits June 30, 2026 14:06

fix checks

be4ae62

# Conflicts: # .github/workflows/test_1.yml # .github/workflows/test_2.yml # .github/workflows/test_3.yml

comments

0c8e91e

add log_utils

dcd3c72

tox

491fe07

internal comments

285f4e9

tox

b2f4fbd

rebase, tox

cf71b5d

update deps

083acf7

internal comments

e57723e

comments

e5b003e

CI failure fixes

89e5f28

CI failure fixes

44d22ef

internal comments

a14cc29

comments

7497098

workflow, lint

fb402a9

workflows

351f79f

comments, tests, rebase, version

e4849da

JWinermaSplunk force-pushed the structlog-handler branch from f9e1186 to e4849da Compare June 30, 2026 21:52

test: log utils

afe162e

xrmx approved these changes Jul 2, 2026

View reviewed changes

Merge branch 'main' into structlog-handler

3710446

xrmx enabled auto-merge July 2, 2026 15:51

precommit

bc7801a

auto-merge was automatically disabled July 2, 2026 18:01
Head branch was pushed to by a user without write access

xrmx added this pull request to the merge queue Jul 3, 2026

Merged via the queue into open-telemetry:main with commit 04a4953 Jul 3, 2026
761 checks passed

github-project-automation Bot moved this from Approved PRs to Done in Python PR digest Jul 3, 2026

Uh oh!

Conversation

JWinermaSplunk commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

How Has This Been Tested?

Does This PR Require a Core Repo Change?

Checklist:

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tammy-baylis-swi commented Mar 19, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tammy-baylis-swi commented Mar 25, 2026

Uh oh!

JWinermaSplunk commented Mar 25, 2026

Uh oh!

Uh oh!

dodofarm commented Apr 18, 2026

Uh oh!

pmcollins left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented May 7, 2026

Uh oh!

JWinermaSplunk commented May 7, 2026

Uh oh!

emdneto commented May 12, 2026

Uh oh!

github-actions Bot commented May 27, 2026

Uh oh!

JWinermaSplunk commented Jul 2, 2026

Uh oh!

xrmx commented Jul 3, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

JWinermaSplunk commented Mar 3, 2026 •

edited

Loading