Skip to content

Conversation

theletterf
Copy link
Contributor

This PR contributes to https://github.com/elastic/opentelemetry-dev/issues/1044

It brings the EDOT narrative documentation and adds an automation script to update a key variable.

@theletterf theletterf requested review from a team as code owners September 15, 2025 07:59
@theletterf theletterf marked this pull request as draft September 15, 2025 08:00
@theletterf theletterf self-assigned this Sep 15, 2025
@theletterf theletterf added the documentation Improvements or additions to documentation label Sep 15, 2025
Copy link

github-actions bot commented Sep 15, 2025

🔍 Preview links for changed docs

More links …

@theletterf theletterf marked this pull request as ready for review September 15, 2025 13:16
@shainaraskas
Copy link
Collaborator

just did a quick peek in. where is the pathway for ECK here?

…ch/docker.md

Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
theletterf and others added 9 commits September 16, 2025 07:49
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…tes-observability-with-elastic-distributions-of-opentelemetry-edot.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…ot-collector-auth.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…tes-observability-with-elastic-distributions-of-opentelemetry-edot.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…-starting.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
theletterf and others added 14 commits September 16, 2025 07:51
…ch/hosts_vms.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…elf-managed/docker.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…elf-managed/hosts_vms.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…erverless/docker.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…erverless/hosts_vms.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…bernetes/deployment.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…bernetes/instrumenting-applications.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…ms/index.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…ms/index.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
…tes-observability-with-elastic-distributions-of-opentelemetry-edot.md

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
@theletterf theletterf requested a review from a team as a code owner September 16, 2025 07:44
Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

Approving to unblock, not reviewing

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM to not block it but I think a few frontmatters (more than I commented on) have to be fixed:

  • variables aren't supported in descriptions today
  • some applies_to and product IDs mention serverless instead of ECH (maybe others too I just reviewed the 2 first I found)

Copy link
Contributor

@yetanothertw yetanothertw left a comment

Choose a reason for hiding this comment

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

Approving to unblock, found a few small typos.

theletterf and others added 7 commits September 16, 2025 12:06
…ch/docker.md

Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
…ch/hosts_vms.md

Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
…ch/docker.md

Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
…ch/hosts_vms.md

Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
…elf-managed/index.md

Co-authored-by: Vlada Chirmicci <vlada.chirmicci@elastic.co>
…bernetes/upgrade.md

Co-authored-by: Vlada Chirmicci <vlada.chirmicci@elastic.co>
…elf-managed/index.md

Co-authored-by: Vlada Chirmicci <vlada.chirmicci@elastic.co>
@theletterf theletterf merged commit 1f593a9 into main Sep 16, 2025
8 checks passed
@theletterf theletterf deleted the add-edot-narrative-docs branch September 16, 2025 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants