Skip to content

Conversation

@micheleRP
Copy link
Contributor

@micheleRP micheleRP commented Aug 19, 2025

Description

This pull request updates the comparison table to the Redpanda Cloud overview, making it easier to understand the differences between Serverless, BYOC, and Dedicated cluster types. Is also adds manual deserialization of Schema Registry to the SM/Cloud feature compatibility list.

Resolves https://redpandadata.atlassian.net/browse/DOC-1261
Resolves https://redpandadata.atlassian.net/browse/DOC-1574
Review deadline:

Page previews

Cloud Overview/Cluster types
Redpanda Cloud vs Self-Managed feature compatibility

Checks

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

@netlify
Copy link

netlify bot commented Aug 19, 2025

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit e48faf2
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/68a54e638b623b00080f583e
😎 Deploy Preview https://deploy-preview-393--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 Aug 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

The PR updates modules/get-started/pages/cloud-overview.adoc to introduce Bring Your Own Cloud (BYOC) as a distinct deployment option, adds a new three-column comparison table (Serverless, Dedicated, BYOC) with detailed attributes, and reorganizes sections to include BYOC/ BYOVPC/BYOVNet content and signup guidance. It updates feature references (e.g., Kafka Connect availability) and adjusts feature status notes (removing Iceberg from Beta; retaining Remote Read Replicas as Beta). Content is restructured to foreground BYOC, revise networking/governance notes, and update cross-references.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Assessment against linked issues

Objective Addressed Explanation
Provide a comparison table for features supported on BYOC, Dedicated, and Serverless (DOC-1261)

Assessment against linked issues: Out-of-scope changes

Code Change Explanation
Remove “Integration with Apache Iceberg” from Beta section (modules/get-started/pages/cloud-overview.adoc) The linked issue only requests a comparison table; changing feature status for Iceberg is unrelated.
Reposition/retain “Remote Read Replicas for AWS and GCP” as Beta (modules/get-started/pages/cloud-overview.adoc) Adjusting Beta placement/status is outside the comparison table objective.
Add BYOC signup guidance and expanded governance/networking notes (modules/get-started/pages/cloud-overview.adoc) Procedural/signup guidance and governance elaboration go beyond the requested comparison table.

Possibly related PRs

Suggested reviewers

  • tomasz-sadura
  • c4milo
  • paulohtb6
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch DOC-1261-comparison-table-for-cloud-deployments

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@micheleRP micheleRP marked this pull request as ready for review August 19, 2025 16:43
@micheleRP micheleRP requested a review from a team as a code owner August 19, 2025 16:43
@micheleRP micheleRP requested a review from hcoyote August 19, 2025 16:44
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (6)
modules/get-started/pages/cloud-overview.adoc (6)

6-6: Resolve support/SLA wording inconsistencies

Line 6 claims “24x7 support,” while the comparison table and note later narrow this to “Enterprise support … Monday through Friday, plus immediate escalation for production outages 24/7,” and “Enterprise support with annual contracts” for Serverless. This is confusing.

Recommend clarifying the global statement to reflect monitoring vs. support hours, and align the table/notes with official Support terms.

Proposed neutral edits (adjust as per official Support policy):

-Redpanda Cloud is a complete data streaming platform delivered as a fully-managed service. It provides automated upgrades and patching, data balancing, and 24x7 support while continuously monitoring your clusters and underlying infrastructure to meet strict performance, availability, reliability, and security requirements.
+Redpanda Cloud is a complete data streaming platform delivered as a fully-managed service. It provides automated upgrades and patching, data balancing, and 24x7 monitoring while meeting strict performance, availability, reliability, and security requirements. Enterprise Support options vary by cluster type and contract.

Optionally tweak the note to make hours explicit and consistent:

-* Enterprise support is staffed by streaming experts around the clock Monday through Friday, plus immediate escalation for production outages 24/7.
+* Enterprise Support: staffed 24x5 (Mon–Fri) with 24x7 P1 escalation. Contact Redpanda for plan details.

Also applies to: 97-101, 107-108


33-36: Clarify current Serverless provider scope

“Redpanda’s cloud (AWS only)” likely reflects the current LA scope. Consider making this time-bound to avoid future confusion.

-| Redpanda's cloud (AWS only)
+| Redpanda's cloud (currently AWS only)

47-56: Qualify throughput and partition limits as tier-dependent

These maxima typically vary by tier/region/provider and may change. To avoid hard commitments in an overview table, qualify with “Up to …” and add a note pointing to tiers docs.

-| *Max throughput (write, read)*
-| 100 MB/s, 300 MB/s
-| 400 MB/s, 800 MB/s
-| 2 GB/s, 4 GB/s
+| *Max throughput (write, read)*
+| Up to 100 MB/s, 300 MB/s
+| Up to 400 MB/s, 800 MB/s
+| Up to 2 GB/s, 4 GB/s
@@
-| *Partitions*
-| 5,000
-| 45,600
-| 112,500
+| *Partitions*
+| Up to 5,000
+| Up to 45,600
+| Up to 112,500

And add a clarifying bullet to the note:

 * The partition limit is the number of logical partitions before replication occurs. Redpanda Cloud uses a replication factor of three.
+* Throughput and partition limits vary by tier, region, and cloud provider. See the BYOC and Dedicated tiers pages for details.

62-66: Confirm OIDC support for Serverless SSO

Serverless lists only GitHub/Google. If org-level OIDC SSO is (or will be) supported for Serverless, update this row to avoid under-selling. If not supported by design, consider adding a footnote clarifying “SSO refers to Cloud UI login; Kafka API OIDC is not supported.”


148-150: Add GCP example for object storage for completeness

You cite S3 and Azure Storage but not Google Cloud Storage. Add GCS to keep provider parity.

-... including subnets (subnetworks in GCP), IAM roles, and object storage resources (for example, S3 buckets or Azure Storage accounts).
+... including subnets (subnetworks in GCP), IAM roles, and object storage resources (for example, Amazon S3 buckets, Google Cloud Storage buckets, or Azure Storage accounts).

159-160: Verify BYOC marketplace billing options (include Azure?)

This mentions direct billing, Google Cloud Marketplace, and AWS Marketplace. If BYOC on Azure supports Azure Marketplace billing, add it here for completeness.

-To start using BYOC, contact https://redpanda.com/try-redpanda?section=enterprise-trial[Redpanda sales^] to request a private offer with possible discounts. You are billed directly or through Google Cloud Marketplace or AWS Marketplace. 
+To start using BYOC, contact https://redpanda.com/try-redpanda?section=enterprise-trial[Redpanda Sales^] to request a private offer with possible discounts. You are billed directly or through AWS Marketplace, Azure Marketplace, or Google Cloud Marketplace (depending on your cloud and contract).
📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between a1ea7ab and b82819c.

📒 Files selected for processing (1)
  • modules/get-started/pages/cloud-overview.adoc (4 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#361
File: modules/networking/pages/cloud-security-network.adoc:176-179
Timestamp: 2025-07-21T21:52:17.061Z
Learning: In Redpanda Cloud documentation, BYOVPC and BYOVNet are different product names for different cloud providers: BYOVPC is used for AWS and GCP, while BYOVNet is used for Azure. When documentation sections cover multiple cloud providers, the combined notation "BYOVPC/BYOVNet" is correct and should not be changed to use only one term.
📚 Learning: 2025-07-21T21:52:17.061Z
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#361
File: modules/networking/pages/cloud-security-network.adoc:176-179
Timestamp: 2025-07-21T21:52:17.061Z
Learning: In Redpanda Cloud documentation, BYOVPC and BYOVNet are different product names for different cloud providers: BYOVPC is used for AWS and GCP, while BYOVNet is used for Azure. When documentation sections cover multiple cloud providers, the combined notation "BYOVPC/BYOVNet" is correct and should not be changed to use only one term.

Applied to files:

  • modules/get-started/pages/cloud-overview.adoc
📚 Learning: 2025-08-07T16:36:45.111Z
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#377
File: modules/networking/pages/dedicated/gcp/configure-psc-in-ui.adoc:35-71
Timestamp: 2025-08-07T16:36:45.111Z
Learning: In Redpanda Cloud documentation, NAT subnet and firewall rule creation commands for GCP Private Service Connect are producer-side infrastructure steps that only BYOC (Bring Your Own Cloud) customers need to perform, since they manage their own VPC. Dedicated cluster customers don't need these steps because Redpanda manages the infrastructure for them. The ifdef::env-byoc[] guard correctly hides these commands from Dedicated users.

Applied to files:

  • modules/get-started/pages/cloud-overview.adoc
🔇 Additional comments (7)
modules/get-started/pages/cloud-overview.adoc (7)

151-154: Correct use of BYOVPC/BYOVNet naming across providers

The combined “BYOVPC/BYOVNet” phrasing is correct when covering AWS/GCP (BYOVPC) and Azure (BYOVNet) together. Thanks for keeping this consistent.


161-177: Nice, concise positioning of Serverless vs BYOC/Dedicated

This section cleanly sets expectations and links to tier docs. Looks good.


186-221: BYOC shared-responsibility table reads clear and balanced

Responsibilities split make sense (for example, VPC peering/customer, service endpoints/Redpanda). No issues spotted.


222-254: BYOVPC/BYOVNet responsibility split matches product intent

Good call-out that IAM and Tiered Storage shift to customer in BYOVPC/BYOVNet, with observability nuance noted. This aligns with past guidance.


256-286: Dedicated responsibility table is consistent with managed model

Assignments look correct. No changes requested.


295-296: Kafka Connect note aligns with comparison table

“Disabled by default” and “BYOC/Dedicated via Support” matches the table’s “✓ (disabled by default)” entries. Good cross-consistency.


346-350: Beta features list change: confirm cross-doc consistency

Remote Read Replicas are listed here as beta (and removed from any “not supported” mentions). Ensure any other pages that previously flagged RRR as unsupported are updated, and that ICEBERG references were intentionally removed from beta here.

Comment on lines 42 to 46
| *Service-level agreement (SLA)*
| 99.9%
| 99.99%
| 99.99%

Copy link

Choose a reason for hiding this comment

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

dedicated/byoc are "up to" 99.99% availability, it depends on if they pick multi-az or not. single-az is 99.5%.

I'm not sure how to best convey this (maybe linking to our SLA doc for authoritative SLAs?)

| Up to 400 MB/s, 800 MB/s
| Up to 2 GB/s, 4 GB/s

| *Partitions*
Copy link

Choose a reason for hiding this comment

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

I know we've debated elsewhere about whether this should be called out as logical partitions, or partitions before replication. Shoudl we do similar here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct, and it's in the list of Notes under this table. But since it was unclear to you when you eyeballed this, I'll update the row in the table to say Partitions, pre-replication

| 99.99%
| 99.99%

| *Max throughput (write, read)*
Copy link

Choose a reason for hiding this comment

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

do we need to mention flex tiers somehow? This sort of changes the mechanics of the high end on tiers (or we can wait and revisit once flex tiers are officially released)


| *Kafka Connect*
| ✗
| ✓ (disabled by default)
Copy link
Contributor

@paulohtb6 paulohtb6 Aug 19, 2025

Choose a reason for hiding this comment

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

This is ok as is, but we could also add an asterisk, add a reference after the table, and explain to users how to enable it (a link is good enough)

Co-authored-by: Paulo Borges <paulohtb6@gmail.com>
@micheleRP micheleRP merged commit 82ef6b1 into main Aug 20, 2025
5 checks passed
@micheleRP micheleRP deleted the DOC-1261-comparison-table-for-cloud-deployments branch August 20, 2025 17:09
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.

4 participants