Skip to content

Conversation

@paulohtb6
Copy link
Collaborator

@paulohtb6 paulohtb6 commented Dec 18, 2025

Description

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

Page previews

Checks

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

@paulohtb6 paulohtb6 requested a review from a team as a code owner December 18, 2025 18:38
@netlify
Copy link

netlify bot commented Dec 18, 2025

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit 6e6bc41
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/69444a1f1703c00007556130
😎 Deploy Preview https://deploy-preview-1525--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 18, 2025

📝 Walkthrough

Walkthrough

This change updates the beta installation documentation file with environment-specific setup guidance across multiple deployment methods. It adds macOS installation notes redirecting to Docker, expands Docker-based beta setup with image pulling and Docker Compose YAML configuration, introduces Linux package manager installation steps for Fedora/RedHat/Amazon Linux and Debian/Ubuntu repositories, and extends Kubernetes/Helm deployment instructions with repository setup and manifest generation commands. The changes consolidate beta installation information into organized sections for each platform.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

  • Verify accuracy of all installation commands across different Linux distributions and package managers
  • Confirm Docker Compose YAML configuration syntax and completeness for Redpanda broker and console service setup
  • Validate Kubernetes/Helm commands and repository URLs
  • Check formatting consistency and clarity of instructions across all platform-specific sections

Suggested reviewers

  • JakeSCahill

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating beta installation instructions for macOS, which aligns with the primary focus of the PR's file modifications.
Description check ✅ Passed The description follows the required template structure with JIRA ticket reference and includes a completed checkbox indicating 'Content gap'; however, the page previews section is incomplete with no actual preview links provided.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-beta-install

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira 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 be83200 and 6e6bc41.

📒 Files selected for processing (1)
  • modules/get-started/pages/install-beta.adoc (1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: Feediver1
Repo: redpanda-data/docs PR: 1153
File: antora.yml:3-5
Timestamp: 2025-07-14T19:28:43.296Z
Learning: In Redpanda docs, during beta releases, the version metadata may intentionally show inconsistencies where the header displays the beta version (e.g., 25.2 Beta) while internal attributes like full-version, latest-redpanda-tag, operator-beta-tag still reference the stable version (e.g., 25.1). This is resolved during the GA merge process when all version references are synchronized.
📚 Learning: 2025-07-14T19:28:43.296Z
Learnt from: Feediver1
Repo: redpanda-data/docs PR: 1153
File: antora.yml:3-5
Timestamp: 2025-07-14T19:28:43.296Z
Learning: In Redpanda docs, during beta releases, the version metadata may intentionally show inconsistencies where the header displays the beta version (e.g., 25.2 Beta) while internal attributes like full-version, latest-redpanda-tag, operator-beta-tag still reference the stable version (e.g., 25.1). This is resolved during the GA merge process when all version references are synchronized.

Applied to files:

  • modules/get-started/pages/install-beta.adoc
📚 Learning: 2025-12-12T16:18:28.126Z
Learnt from: andrewstucki
Repo: redpanda-data/docs PR: 1514
File: modules/troubleshoot/partials/errors-and-solutions.adoc:618-632
Timestamp: 2025-12-12T16:18:28.126Z
Learning: In Redpanda ShadowLink CRD usage, do not copy CA certificates into pods via kubectl cp because they won't persist across pod restarts. Store CA certs in Kubernetes Secrets and reference them in the ShadowLink's staticConfiguration block (or inline the cert data in that block) to ensure persistence and proper secret management.

Applied to files:

  • modules/get-started/pages/install-beta.adoc
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Redirect rules - redpanda-docs-preview
  • GitHub Check: Header rules - redpanda-docs-preview
  • GitHub Check: Pages changed - redpanda-docs-preview
🔇 Additional comments (3)
modules/get-started/pages/install-beta.adoc (3)

150-158: Clear macOS guidance for beta users.

The macOS section appropriately informs users that beta versions are not available through Homebrew and redirects them to Docker. This is accurate and helpful.


163-174: Kubernetes Redpanda installation looks good.

The Helm setup properly installs cert-manager and configures the Redpanda chart to use the unstable image. The console is correctly disabled here to be deployed separately.


214-216: Verify the Console service name for port-forwarding.

Confirm that the service name consolev3-console-unstable matches the actual Kubernetes service name generated by the helm template consolev3 command. If the Helm chart uses different naming conventions, this port-forward command may fail.

You can verify the generated service name by:

  1. Running the helm template command and examining the generated YAML for the Service resource name, or
  2. After deploying, running kubectl get svc -n <namespace> and noting the actual service name.

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

@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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (4)
modules/get-started/pages/install-beta.adoc (4)

27-27: Fix the lines attribute to display the complete Docker Compose YAML.

The lines=35 attribute restricts the output to only line 35, displaying just the services: key. Remove this attribute to show the entire Docker Compose configuration.

🔎 Apply this diff:
-.Reveal the YAML content
+.Reveal the YAML content
 [%collapsible]
 ====
-[source,yaml,subs="attributes+",lines=35]
+[source,yaml,subs="attributes+"]
 ----

115-120: Complete the curl commands by piping to sh.

Both the Fedora/RedHat and Debian/Ubuntu repository setup commands are incomplete. The curl commands download the repository setup script but do not execute it. This will cause the subsequent package installation to fail.

🔎 Apply this diff:
 Fedora/RedHat/Amazon Linux::
 +
 . Add the `redpanda-unstable` repository to your package manager:
 +
 ```bash
 curl -1sLf \
-  'https://dl.redpanda.com/E4xN1tVe3Xy60GTx/redpanda-unstable/setup.rpm.sh'
+  'https://dl.redpanda.com/E4xN1tVe3Xy60GTx/redpanda-unstable/setup.rpm.sh' | sh

```diff
 Debian/Ubuntu::
 +
 . Add the `redpanda-unstable` repository to your package manager:
 +
 ```bash
 curl -1sLf \
-  'https://dl.redpanda.com/E4xN1tVe3Xy60GTx/redpanda-unstable/setup.deb.sh'
+  'https://dl.redpanda.com/E4xN1tVe3Xy60GTx/redpanda-unstable/setup.deb.sh' | sh
</details>



Also applies to: 130-135

---

`189-191`: **Fix the broker array index in the third broker configuration.**

Line 191 incorrectly reuses `brokers[0]` instead of `brokers[2]`, which overwrites the first broker configuration. This results in only two unique broker addresses being registered instead of three.


<details>
<summary>🔎 Apply this diff:</summary>

```diff
   --set "config.kafka.brokers[0]=redpanda-0.redpanda.redpanda.svc.cluster.local.:9093" \
   --set "config.kafka.brokers[1]=redpanda-1.redpanda.redpanda.svc.cluster.local.:9093" \
-  --set "config.kafka.brokers[0]=redpanda-2.redpanda.redpanda.svc.cluster.local.:9093" \
+  --set "config.kafka.brokers[2]=redpanda-2.redpanda.redpanda.svc.cluster.local.:9093" \

180-183: Fix hardcoded Console chart version and broker index duplication.

Two issues found in the helm template command:

  1. Hardcoded chart version (line 183): The version v0.0-k8s0-968fcab9-helm-chart is not a valid public chart version. This should use a dynamic placeholder like {console-beta-chart-version} to match the pattern used for image.tag={console-beta-tag}.

  2. Broker index error (line 191): The third broker configuration incorrectly uses brokers[0] instead of brokers[2], which overwrites the first broker and leaves only two unique brokers configured:

--set "config.kafka.brokers[0]=redpanda-0.redpanda.redpanda.svc.cluster.local.:9093" \
--set "config.kafka.brokers[1]=redpanda-1.redpanda.redpanda.svc.cluster.local.:9093" \
--set "config.kafka.brokers[2]=redpanda-2.redpanda.redpanda.svc.cluster.local.:9093" \
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira 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 be83200 and 6e6bc41.

📒 Files selected for processing (1)
  • modules/get-started/pages/install-beta.adoc (1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: Feediver1
Repo: redpanda-data/docs PR: 1153
File: antora.yml:3-5
Timestamp: 2025-07-14T19:28:43.296Z
Learning: In Redpanda docs, during beta releases, the version metadata may intentionally show inconsistencies where the header displays the beta version (e.g., 25.2 Beta) while internal attributes like full-version, latest-redpanda-tag, operator-beta-tag still reference the stable version (e.g., 25.1). This is resolved during the GA merge process when all version references are synchronized.
📚 Learning: 2025-07-14T19:28:43.296Z
Learnt from: Feediver1
Repo: redpanda-data/docs PR: 1153
File: antora.yml:3-5
Timestamp: 2025-07-14T19:28:43.296Z
Learning: In Redpanda docs, during beta releases, the version metadata may intentionally show inconsistencies where the header displays the beta version (e.g., 25.2 Beta) while internal attributes like full-version, latest-redpanda-tag, operator-beta-tag still reference the stable version (e.g., 25.1). This is resolved during the GA merge process when all version references are synchronized.

Applied to files:

  • modules/get-started/pages/install-beta.adoc
📚 Learning: 2025-12-12T16:18:28.126Z
Learnt from: andrewstucki
Repo: redpanda-data/docs PR: 1514
File: modules/troubleshoot/partials/errors-and-solutions.adoc:618-632
Timestamp: 2025-12-12T16:18:28.126Z
Learning: In Redpanda ShadowLink CRD usage, do not copy CA certificates into pods via kubectl cp because they won't persist across pod restarts. Store CA certs in Kubernetes Secrets and reference them in the ShadowLink's staticConfiguration block (or inline the cert data in that block) to ensure persistence and proper secret management.

Applied to files:

  • modules/get-started/pages/install-beta.adoc
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Redirect rules - redpanda-docs-preview
  • GitHub Check: Header rules - redpanda-docs-preview
  • GitHub Check: Pages changed - redpanda-docs-preview
🔇 Additional comments (3)
modules/get-started/pages/install-beta.adoc (3)

150-158: Clear macOS guidance for beta users.

The macOS section appropriately informs users that beta versions are not available through Homebrew and redirects them to Docker. This is accurate and helpful.


163-174: Kubernetes Redpanda installation looks good.

The Helm setup properly installs cert-manager and configures the Redpanda chart to use the unstable image. The console is correctly disabled here to be deployed separately.


214-216: Verify the Console service name for port-forwarding.

Confirm that the service name consolev3-console-unstable matches the actual Kubernetes service name generated by the helm template consolev3 command. If the Helm chart uses different naming conventions, this port-forward command may fail.

You can verify the generated service name by:

  1. Running the helm template command and examining the generated YAML for the Service resource name, or
  2. After deploying, running kubectl get svc -n <namespace> and noting the actual service name.

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