Skip to content

Add GA headers to HTTP responses, add live-check fixtures as site test pages#9630

Merged
chalin merged 13 commits intoopen-telemetry:mainfrom
chalin:chalin-m24-asset-fetch-2026-0411
Apr 11, 2026
Merged

Add GA headers to HTTP responses, add live-check fixtures as site test pages#9630
chalin merged 13 commits intoopen-telemetry:mainfrom
chalin:chalin-m24-asset-fetch-2026-0411

Conversation

@chalin
Copy link
Copy Markdown
Contributor

@chalin chalin commented Apr 11, 2026

  • Contributes to Address site observability and data access #9559
  • Adds X-Asset-Fetch-Ga-Info response headers to schema, negotiated Markdown, and direct asset responses
  • Updates the asset-fetch analytics plan and edit history to match the current implementation
  • Uses ga-event-candidate with config-present or config-missing to expose derived GA path and coarse emission state
  • Keeps non-candidate cases readable with none: <reason> header values
  • Extends unit and cross-function integration tests to validate the new header behavior
  • Splits deployed-host live checks so delivery assertions and header assertions stay separate
  • Adds shared edge-function test helpers and makes the test suites more DRY without changing coverage
  • Splits deployed-host live checks so delivery assertions and GA-info header assertions stay separate
  • Adds isolated /site/testing/tests/ fixtures, including a regular page and an HTML-only page, for low-impact live checks

All unit, integration, and live tests are passing.

New response header is being generated, e.g.:

$ curl -sI  -H 'Accept: text/markdown' https://deploy-preview-9630--opentelemetry.netlify.app/site/testing/tests/regular/
HTTP/2 200 
...
x-asset-fetch-ga-info: /site/testing/tests/regular/index.md;ga-event-candidate,config-missing
...

@chalin chalin added the CI/infra Repo CI & infrastructure label Apr 11, 2026
@chalin chalin requested a review from a team as a code owner April 11, 2026 08:02
@otelbot-docs otelbot-docs bot added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 11, 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 11, 2026
@chalin chalin merged commit c48fe2d into open-telemetry:main Apr 11, 2026
24 checks passed
@chalin chalin deleted the chalin-m24-asset-fetch-2026-0411 branch April 11, 2026 13:18
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

analytics+observability CI/infra Repo CI & infrastructure 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