Skip to content

Add CNM TCP metrics for cloud monitor#36246

Merged
estherk15 merged 3 commits intomasterfrom
esther/docs-14145-cnm-metrics-update
Apr 28, 2026
Merged

Add CNM TCP metrics for cloud monitor#36246
estherk15 merged 3 commits intomasterfrom
esther/docs-14145-cnm-metrics-update

Conversation

@estherk15
Copy link
Copy Markdown
Contributor

@estherk15 estherk15 commented Apr 23, 2026

What does this PR do? What is the motivation?

Fixes DOCS-14145

Adds 7 new TCP congestion signal metrics to the Cloud Network Monitoring monitor page.

  • TCP Retransmit Timeouts
  • TCP Fast Recovery Events
  • TCP Reordering Events
  • TCP Out-of-Order Packets
  • TCP Congestion Events
  • TCP ECN Negotiated
  • TCP Zero Window Probes

Merge instructions

Merge readiness:

  • Ready for merge

@estherk15 estherk15 requested a review from a team as a code owner April 23, 2026 21:50
@estherk15 estherk15 added the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 23, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

- Spell out SACK as "selective acknowledgment (SACK)" on first mention
- Reorder rows so TCP ECN Negotiated precedes TCP Congestion Events,
  ensuring ECN is expanded before reuse
- Drop filler "successfully" and replace vague "meaningful" with
  "non-zero" in TCP ECN Negotiated description
@estherk15 estherk15 removed the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 28, 2026
@jeff-morgan-dd jeff-morgan-dd self-assigned this Apr 28, 2026
Copy link
Copy Markdown
Contributor

@jeff-morgan-dd jeff-morgan-dd left a comment

Choose a reason for hiding this comment

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

LGTM!

@estherk15 estherk15 merged commit c8241bf into master Apr 28, 2026
19 of 20 checks passed
@estherk15 estherk15 deleted the esther/docs-14145-cnm-metrics-update branch April 28, 2026 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants