Skip to content

DOCS-1875 - Split IC vs OTel benchmarks by collection method - #7145

Merged
amee-sumo merged 7 commits into
mainfrom
DOCS-1875
Sep 17, 2026
Merged

amee-sumo merged 7 commits into
mainfrom
DOCS-1875

Conversation

@amee-sumo

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

This pull request splits the single "Installed Collector vs OpenTelemetry Collector Performance Benchmarks" doc into a landing page plus two collection-method-specific pages, since Remote Windows Event Log (RWEL) collection has a fundamentally different performance profile than local file sources (WinRM-bound throughput ceiling vs. a linear ramp).

  • New landing page: ic-vs-ot-collector-performance-benchmarks/index.md, with card links to both sub-pages
  • local-file-source-benchmarks.md: existing local file source benchmark content (renamed from the original single-file doc)
  • remote-windows-event-log-benchmarks.md: new RWEL-specific benchmark data, with its own recommendation, test environment, and results tables
  • Updated sidebars.ts to a category linking to the new index and its two children
  • Fixed two Remote Windows Event Log source docs that linked to the old combined page path

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/DOCS-1875

Splits the single benchmarks doc into a landing page plus separate Local File Source and Remote Windows Event Log (RWEL) pages, since RWEL's WinRM-bound throughput ceiling produces a different performance profile than local file sources. Fixes two Remote Windows Event Log source docs that linked to the old combined page path.
@amee-sumo amee-sumo self-assigned this Sep 16, 2026
@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Sep 16, 2026

@kimsauce kimsauce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Structure, sidebar wiring, and internal links all check out, and CI is green (spellcheck, build checks). Left three inline notes on remote-windows-event-log-benchmarks.md: an inconsistent heading level, a sentence running into the previous bullet from a missing blank line, and a missing id field in the frontmatter. None of that blocks merging.


Generated by Claude Code

Comment thread docs/send-data/ic-vs-ot-collector-performance-benchmarks/index.md
Comment thread docs/send-data/ic-vs-ot-collector-performance-benchmarks/index.md
amee-sumo and others added 6 commits September 17, 2026 10:10
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
…-file-source-benchmarks.md

Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
…e-windows-event-log-benchmarks.md

Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
…-file-source-benchmarks.md

Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Fixes heading level and missing bullet in remote-windows-event-log-benchmarks.md, adds an IC vs OTel Performance Benchmarks card to the Send Data cards list, and removes a duplicate id key in local-file-source-benchmarks.md left over from applying a suggested edit.
@amee-sumo
amee-sumo requested a review from kimsauce September 17, 2026 04:50

@kimsauce kimsauce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Re-reviewed after the latest push. All prior findings are fixed: the heading level on remote-windows-event-log-benchmarks.md is now consistent (### Test conditions), the missing bullet before "Neither collector exceeded the 190% safety cutoff..." is in, id fields are set on both new/renamed pages, the landing page now uses slug per the suggested fix, and a card for the new landing page was added to docs/send-data/index.md. CI is green (spellcheck, build checks, merge window). Nothing further from me.


Generated by Claude Code

@kimsauce kimsauce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks good

@amee-sumo
amee-sumo added this pull request to the merge queue Sep 17, 2026
Merged via the queue into main with commit 7f9ff20 Sep 17, 2026
4 checks passed
@amee-sumo
amee-sumo deleted the DOCS-1875 branch September 17, 2026 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants