Skip to content

Conversation

@JakeSCahill
Copy link
Contributor

Description

Resolves https://redpandadata.atlassian.net/browse/
Review deadline:

Page previews

Checks

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

@JakeSCahill JakeSCahill requested a review from a team as a code owner December 19, 2025 08:16
@netlify
Copy link

netlify bot commented Dec 19, 2025

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit 0b7c667
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/695d48899340cd0008cfdbed
😎 Deploy Preview https://deploy-preview-1526--redpanda-docs-preview.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 Dec 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 PR reorganizes documentation structure by moving the Kubernetes shadowing monitoring content from the monitoring section to the shadowing section. The file modules/manage/pages/kubernetes/monitoring/k-monitor-shadowing.adoc is relocated to modules/manage/pages/kubernetes/shadowing/k-monitor-shadowing.adoc, with all cross-references updated accordingly across navigation, release notes, and related documentation pages. The relocated content includes minor text formatting adjustments for capitalization consistency in status indicators and descriptive labels.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10–15 minutes

  • This is a straightforward documentation reorganization with consistent cross-reference updates applied uniformly across multiple files
  • Review focus: verify all cross-references have been correctly updated to point to the new shadowing path, and confirm no references to the old monitoring path remain broken

Possibly related PRs

  • Add Shadowing docs for Kubernetes deployments #1514 — Directly related; both PRs reorganize Kubernetes shadowing monitoring documentation and update cross-references to move content into the manage:kubernetes/shadowing path
  • add shadowing metrics #1496 — Modifies the same shadowing monitoring documentation files with overlapping changes to disaster-recovery shadowing content
  • dr: adds shadowing docs #1381 — Introduces shadowing documentation structure including shadow link monitoring content that forms the basis for this reorganization

Suggested reviewers

  • david-yu
  • andrewstucki
  • paulohtb6

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description uses the required template structure but contains incomplete/placeholder information: no actual Jira ticket URL, no review deadline specified, no page previews added, and no checks selected. Replace placeholder Jira URL with actual ticket, add review deadline, include specific page preview links from Netlify bot, and check the appropriate box (likely 'Small fix' for documentation reorganization).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: deleting a duplicate monitoring topic for Kubernetes Shadowing as shown in the file changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Contributor

@micheleRP micheleRP left a comment

Choose a reason for hiding this comment

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

tiny style suggestions

@JakeSCahill JakeSCahill merged commit 2f20d4c into main Jan 6, 2026
6 of 7 checks passed
@JakeSCahill JakeSCahill deleted the delete-duplicate branch January 6, 2026 17:38
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