Skip to content

fix(api): place per-output sent events in Events Out column#24951

Merged
pront merged 4 commits intomasterfrom
fix/top-output-subrow-column
Mar 18, 2026
Merged

fix(api): place per-output sent events in Events Out column#24951
pront merged 4 commits intomasterfrom
fix/top-output-subrow-column

Conversation

@pront
Copy link
Copy Markdown
Member

@pront pront commented Mar 17, 2026

Summary

  • Components with multiple output ports (e.g. route transforms with _unmatched, success outputs) render sub-rows in the vector top dashboard
  • The per-output sent events metric was being placed in column index 5 (Bytes In) instead of column index 6 (Events Out)
  • This is a one-line fix: data[5]data[6]

Test plan

  • Run vector top against a Vector instance with a route transform that has named output ports
  • Verify per-output sent events appear in the Events Out column, not Bytes In
Screenshot 2026-03-18 at 9 40 20 AM

🤖 Generated with Claude Code

Output sub-rows for components with multiple output ports (e.g. route
transforms with `_unmatched`, `success` outputs) were placing the
per-output sent events metric in column index 5 (Bytes In) instead of
column index 6 (Events Out).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Mar 17, 2026
@github-actions github-actions Bot added the domain: ci Anything related to Vector's CI environment label Mar 18, 2026
@pront pront marked this pull request as ready for review March 18, 2026 13:42
@pront pront requested a review from a team as a code owner March 18, 2026 13:42
@pront pront added this pull request to the merge queue Mar 18, 2026
@pront pront removed the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Mar 18, 2026
@pront pront removed this pull request from the merge queue due to a manual request Mar 18, 2026
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Comment thread changelog.d/24951_top_events_out_column.fix.md Fixed
@pront pront enabled auto-merge March 18, 2026 14:07
@pront pront changed the title fix(top): place per-output sent events in Events Out column fix(api): place per-output sent events in Events Out column Mar 18, 2026
@pront pront added this pull request to the merge queue Mar 18, 2026
Merged via the queue into master with commit 95f4394 Mar 18, 2026
58 of 60 checks passed
@pront pront deleted the fix/top-output-subrow-column branch March 18, 2026 15:11
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

domain: ci Anything related to Vector's CI environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants