Skip to content

feat: Add basic OpenTelemetry instrumentation #1255

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
merged 15 commits into from
Jul 11, 2025
Merged

feat: Add basic OpenTelemetry instrumentation #1255

merged 15 commits into from
Jul 11, 2025

Conversation

Pijukatel
Copy link
Collaborator

@Pijukatel Pijukatel commented Jun 17, 2025

Description

  • Add CrawlerInstrumentor that can be used to add instrumentation for tracing requests and also to auto-instrument other Crawlee classes.
  • Add documentation guide about OpenTelemetry.
  • Add test for the CrawlerInstrumentor.
  • Do a minor refactoring of ContextPipeline and middleware handling to make it easily observable.

Issues

@Pijukatel Pijukatel added enhancement New feature or request. t-tooling Issues with this label are in the ownership of the tooling team. labels Jun 17, 2025
@github-actions github-actions bot added this to the 117th sprint - Tooling team milestone Jun 17, 2025
@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label Jun 17, 2025
@Pijukatel Pijukatel requested review from vdusek and janbuchar June 17, 2025 13:42
@Pijukatel Pijukatel marked this pull request as ready for review June 17, 2025 13:42
@Pijukatel Pijukatel requested a review from vdusek July 8, 2025 10:59
Pijukatel added 2 commits July 8, 2025 15:21
Even more engaging title and some spelling fixes
Pijukatel and others added 2 commits July 11, 2025 09:29
Co-authored-by: Vlada Dusek <v.dusek96@gmail.com>
@Pijukatel Pijukatel requested a review from vdusek July 11, 2025 08:00
Copy link
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Pijukatel Pijukatel merged commit a92d8b3 into master Jul 11, 2025
19 checks passed
@Pijukatel Pijukatel deleted the otel-test branch July 11, 2025 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add OpenTelemetry instrumentation
2 participants