Skip to content

Conversation

@kbatuigas
Copy link
Contributor

@kbatuigas kbatuigas commented Nov 19, 2025

Description

Related PR: redpanda-data/docs#1470 - this must be merged first.
Related PR (Data Plane API): redpanda-data/api-docs#30 - Data Plane API reference links will not work until this is merged

Resolves https://github.com/redpanda-data/documentation-private/issues/
Review deadline: 19 Nov

Page previews

What's New in Cloud
Manage Throughput > View connected client details
Monitor Redpanda Cloud > Throughput > Identify high-throughput clients
Configure client connections
rpk cluster connections list

Checks

  • New feature
  • Content gap
  • Support Follow-up
  • Small fix (typos, links, copyedits, etc)

@kbatuigas kbatuigas requested a review from a team as a code owner November 19, 2025 06:26
@netlify
Copy link

netlify bot commented Nov 19, 2025

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit 56bd392
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/691e27d28ada950008e934b3
😎 Deploy Preview https://deploy-preview-459--rp-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 19, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

This pull request adds documentation for a new feature: connected client monitoring in Redpanda Cloud. The changes include updating the Antora playbook to pull a specific documentation branch (DOC-1780-cloud-client-monitoring), reorganizing client connection configuration pages with aliases for backward compatibility, creating two new reference pages for rpk cluster connections commands, adding a "Connected client monitoring" subsection to the what's new guide, and updating several include directives to use versioned references (25.3@).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Multiple documentation page additions following a consistent pattern (rpk-cluster-connections.adoc, rpk-cluster-connections-list.adoc, configure-client-connections.adoc)
  • Simple include directive version updates throughout (25.3@ qualifier additions)
  • Single-line branch change in Antora playbook configuration
  • Straightforward page reorganization with alias for backward compatibility

Possibly related PRs

Suggested reviewers

  • paulohtb6
  • micheleRP
  • Feediver1

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description includes related PRs, page previews, and a review deadline, but the issue number placeholder is unfilled and no checklist items are marked. Fill in the GitHub issue number and mark at least one checkbox to indicate the type of change (New feature, Content gap, Support Follow-up, or Small fix).
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The PR title 'Add single-sourced connected client monitoring docs' directly aligns with the main change: introducing documentation for connected client monitoring features using single-sourced content.
Linked Issues check ✅ Passed The PR addresses DOC-1780 by adding comprehensive documentation for connected client monitoring features including new rpk command docs and whats-new content.
Out of Scope Changes check ✅ Passed All changes are in-scope: adding/reorganizing connected client monitoring docs, restructuring related configuration pages, and updating include directives for versioned content.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@Feediver1 Feediver1 left a comment

Choose a reason for hiding this comment

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

lgtm

@kbatuigas kbatuigas force-pushed the DOC-1780-cloud-add-connected-client-monitoring branch from f2ecb12 to 38c958c Compare November 19, 2025 20:18
@kbatuigas kbatuigas merged commit e546332 into main Nov 19, 2025
5 checks passed
@kbatuigas kbatuigas deleted the DOC-1780-cloud-add-connected-client-monitoring branch November 19, 2025 20:54
@coderabbitai coderabbitai bot mentioned this pull request Nov 25, 2025
4 tasks
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.

3 participants