Skip to content

Adds asset tracking for explicit-path .md and .txt assets, and add cross-function coverage#9627

Merged
chalin merged 8 commits intoopen-telemetry:mainfrom
chalin:chalin-m24-md-pass-thu-ga-2026-0410
Apr 10, 2026
Merged

Adds asset tracking for explicit-path .md and .txt assets, and add cross-function coverage#9627
chalin merged 8 commits intoopen-telemetry:mainfrom
chalin:chalin-m24-md-pass-thu-ga-2026-0410

Conversation

@chalin
Copy link
Copy Markdown
Contributor

@chalin chalin commented Apr 10, 2026

  • Contributes to Address site observability and data access #9559
  • Adds asset-tracking edge function to cover explicit requests of .md and .txt assets
  • Tracks direct asset requests regardless of response status and records the returned status_code and content_type
  • Uses asset_group = text for explicit .txt assets such as llms.txt
  • Adds unit coverage for .txt tracking and for status-preserving GA payloads
  • Adds cross-function integration tests to ensure markdown negotiation and asset tracking queue only one GA event per request
  • Updates routing, READMEs, and the asset analytics plan to match the current implementation
  • Housekeeping: Renames gate-style unit tests to say returns false where they skip tracking rather than reject requests

@chalin chalin requested a review from a team as a code owner April 10, 2026 20:48
@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Apr 10, 2026
@otelbot-docs otelbot-docs bot added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 10, 2026
@chalin chalin changed the title Extend asset tracking to explicit .md and .txt assets, and add cross-function coverage Adds asset tracking for explicit-path .md and .txt assets, and add cross-function coverage Apr 10, 2026
@otelbot-docs otelbot-docs bot added ready-to-be-merged This PR is ready to be merged by a maintainer and removed missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer labels Apr 10, 2026
@chalin chalin merged commit e267ae0 into open-telemetry:main Apr 10, 2026
24 checks passed
@chalin chalin deleted the chalin-m24-md-pass-thu-ga-2026-0410 branch April 10, 2026 22:57
tiffany76 pushed a commit to tiffany76/opentelemetry.io that referenced this pull request Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file ready-to-be-merged This PR is ready to be merged by a maintainer

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants