Skip to content

auto-docs: Update property docs for tag v25.3.8#1587

Merged
paulohtb6 merged 3 commits intomainfrom
update-property-docs-v25.3.8
Feb 25, 2026
Merged

auto-docs: Update property docs for tag v25.3.8#1587
paulohtb6 merged 3 commits intomainfrom
update-property-docs-v25.3.8

Conversation

@vbotbuildovich
Copy link
Contributor

This PR auto-generates updated Redpanda property documentation for v25.3.8.

@vbotbuildovich vbotbuildovich requested a review from a team as a code owner February 25, 2026 19:46
@netlify
Copy link

netlify bot commented Feb 25, 2026

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit 4c78c48
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/699f5b7a4f6328000858ff74
😎 Deploy Preview https://deploy-preview-1587--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 Feb 25, 2026

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.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

This pull request updates documentation and configuration metadata for Redpanda version 25.3.8. It increments version references from 25.3.7 to 25.3.8 in the antora configuration file, adds a new log compaction configuration property log_compaction_max_priority_wait_ms with its metadata across multiple documentation files, and refines descriptions for existing properties including Iceberg namespace configuration and consumer group metrics documentation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • paulohtb6
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is minimal and lacks required template sections including Jira ticket link, review deadline, page previews, and categorization checkboxes. Add Jira ticket link, review deadline, page preview links, and select appropriate category checkbox from the template.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically summarizes the main change: updating property documentation for v25.3.8.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-property-docs-v25.3.8

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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: 1

🧹 Nitpick comments (1)
modules/reference/partials/properties/cluster-properties.adoc (1)

6525-6525: Keep description behavior-focused; remove type metadata from prose.

The phrase “as an array of strings” duplicates metadata already captured in the property table and makes the description less behavior-focused.

✏️ Suggested edit
-The default namespace (database name) for Iceberg tables. All tables created by Redpanda will be placed in this namespace within the Iceberg catalog. Supports nested namespaces as an array of strings.
+The default namespace (database name) for Iceberg tables. All tables created by Redpanda are placed in this namespace within the Iceberg catalog. Nested namespaces are supported.

Based on learnings: In property descriptions, keep descriptions focused on behavior and avoid metadata information like type information.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@modules/reference/partials/properties/cluster-properties.adoc` at line 6525,
Update the property description for the default Iceberg namespace to remove type
metadata and focus on behavior: delete the phrase "as an array of strings" from
the sentence and reword the end to something like "Supports nested namespaces"
so the description reads behavior-first (e.g., "The default namespace (database
name) for Iceberg tables. All tables created by Redpanda will be placed in this
namespace within the Iceberg catalog. Supports nested namespaces."). Target the
description text for the default namespace property in cluster-properties.adoc.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@modules/reference/partials/properties/cluster-properties.adoc`:
- Line 8985: In the broker-side fetch description sentence starting "Broker-side
target for the duration of a single fetch request..." replace the incorrect
phrase "less bytes" with "fewer bytes" so the sentence reads "...even if it
means returning fewer bytes in the fetch than are available."; update the
sentence in the cluster-properties text (the broker fetch-duration description)
to use the grammatically correct word "fewer".

---

Nitpick comments:
In `@modules/reference/partials/properties/cluster-properties.adoc`:
- Line 6525: Update the property description for the default Iceberg namespace
to remove type metadata and focus on behavior: delete the phrase "as an array of
strings" from the sentence and reword the end to something like "Supports nested
namespaces" so the description reads behavior-first (e.g., "The default
namespace (database name) for Iceberg tables. All tables created by Redpanda
will be placed in this namespace within the Iceberg catalog. Supports nested
namespaces."). Target the description text for the default namespace property in
cluster-properties.adoc.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between b3af7e0 and d0e9c90.

📒 Files selected for processing (5)
  • antora.yml
  • docs-data/property-overrides.json
  • docs-data/redpanda-property-changes-v25.3.7-to-v25.3.8.json
  • modules/reference/attachments/redpanda-properties-v25.3.8.json
  • modules/reference/partials/properties/cluster-properties.adoc

@paulohtb6 paulohtb6 merged commit 7c7507c into main Feb 25, 2026
6 of 7 checks passed
@paulohtb6 paulohtb6 deleted the update-property-docs-v25.3.8 branch February 25, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants