Skip to content

Minor fixes to OTLP collector page - #39293

Open
asopkin wants to merge 5 commits into
masterfrom
asopkin/pure-otlp-pipeline-fixes
Open

Minor fixes to OTLP collector page#39293
asopkin wants to merge 5 commits into
masterfrom
asopkin/pure-otlp-pipeline-fixes

Conversation

@asopkin

@asopkin asopkin commented Aug 18, 2026

Copy link
Copy Markdown

What does this PR do? What is the motivation?

  1. Renames to the Pure OTel Collector because "Community Collector" had a negative connotation
  2. Adjusts nesting after feedback that it was confusing that the other OTel options were buried under "Other setup options"
  3. Some minor changes to reframe the pure OTel Collector page to focus more on OTel concepts rather than Datadog-specific terms.

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

@asopkin
asopkin requested a review from a team as a code owner August 18, 2026 19:38
@asopkin

asopkin commented Aug 18, 2026

Copy link
Copy Markdown
Author

/review

@github-actions github-actions Bot added the Architecture Everything related to the Doc backend label Aug 18, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

Terminology inconsistency: the file title changed from "Community" to "Pure OpenTelemetry Collector", but references to "Community OTel Collector"/"Community OpenTelemetry Collector" remain in several body locations. See inline comment on the title for the specific lines.

One minor style suggestion on the Unified Service Tagging paragraph.

Reviewed 584c76b1b0947a79d39ab88d567be4921a51769eworkflow run

@@ -1,9 +1,10 @@
---
title: Set Up the Community OpenTelemetry Collector
title: Set Up the Pure OpenTelemetry Collector

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Issue: The title (and description at L7) now use "Pure OpenTelemetry Collector", but the body still says "Community" in several places that were left unchanged:

  • L22 (Overview intro): "...using the Community OpenTelemetry Collector, which is based on..."
  • L36 (Prerequisites): "...see the [feature compatibility table][7] under OTel SDK + Community OTel Collector."
  • L934 (info alert under dd-otel-metric-config): "The recommended Community OTel Collector configuration uses..."

If the rename is intentional, please update these occurrences (and coordinate the corresponding heading rename in hugo/content/en/opentelemetry/compatibility.md, which still uses "OTel SDK + Community OTel Collector" — otherwise the L36 link anchor breaks) so the naming is consistent.

Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@rtrieu
rtrieu self-requested a review August 18, 2026 20:37

@rtrieu rtrieu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

some minor feedback for your review - please request a re-review when ready!

Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
Comment thread hugo/content/en/opentelemetry/setup/collector_exporter/pure_otlp_collector.md Outdated
asopkin and others added 4 commits August 18, 2026 17:19
…lp_collector.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…lp_collector.md

Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
…lp_collector.md

Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
@github-actions github-actions Bot added the Guide Content impacting a guide label Aug 18, 2026
@asopkin
asopkin requested a review from rtrieu August 18, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Everything related to the Doc backend Guide Content impacting a guide

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants