Skip to content

auto-docs: Update property docs for tag v26.1.9#1711

Open
vbotbuildovich wants to merge 1 commit into
mainfrom
update-property-docs-v26.1.9
Open

auto-docs: Update property docs for tag v26.1.9#1711
vbotbuildovich wants to merge 1 commit into
mainfrom
update-property-docs-v26.1.9

Conversation

@vbotbuildovich
Copy link
Copy Markdown
Contributor

This PR auto-generates updated Redpanda property documentation for v26.1.9.

@vbotbuildovich vbotbuildovich requested a review from a team as a code owner May 22, 2026 12:47
@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit 6f28e81
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/6a105048fe20fa00084e5f79
😎 Deploy Preview https://deploy-preview-1711--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
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request documents the Redpanda v26.1.9 release. The changes bump the Antora version metadata from 26.1.8 to 26.1.9, register a new cluster property features_auto_finalization (boolean, default true) that controls whether cluster version finalization happens automatically or via Admin API, and add comprehensive documentation across property definition files and AsciiDoc references. Version introduction metadata is also added to the iceberg_rest_catalog_credentials_host property from v26.1.8.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • redpanda-data/docs#1706: This PR extends the property set by updating the iceberg_rest_catalog_credentials_host metadata and introducing the new features_auto_finalization property, directly building on the prior version updates.
  • redpanda-data/docs#1589: Both PRs update the same Antora version metadata keys and refresh Redpanda property documentation files with new version annotations for release cycles.

Suggested reviewers

  • paulohtb6
  • Feediver1
  • JakeSCahill
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description lacks required template sections: missing Jira ticket link, review deadline, page previews, and check categories. Only a brief statement about auto-generation is provided. Add the missing template sections: Jira ticket reference, review deadline, preview links for updated documentation pages, and check one of the provided categories.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating property documentation for v26.1.9, which aligns with all file changes in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-property-docs-v26.1.9

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

Copy link
Copy Markdown
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

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@modules/reference/partials/properties/cluster-properties.adoc`:
- Line 6271: Remove license/entitlement metadata from the property description
that starts "Whether the cluster active logical version is advanced
automatically..." by deleting the sentence "Setting this to false is an
Enterprise feature and requires a valid license." and any similar licensing
phrasing; keep only behavior-focused guidance (including the note about
finalization behavior), and rely on the existing separate license note block
below to document licensing requirements.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1203292a-ab80-4a75-812b-466b0a1a6c2d

📥 Commits

Reviewing files that changed from the base of the PR and between 6078acc and 6f28e81.

📒 Files selected for processing (6)
  • antora.yml
  • docs-data/property-overrides.json
  • docs-data/redpanda-property-changes-v26.1.7-to-v26.1.8.json
  • docs-data/redpanda-property-changes-v26.1.8-to-v26.1.9.json
  • modules/reference/attachments/redpanda-properties-v26.1.9.json
  • modules/reference/partials/properties/cluster-properties.adoc
💤 Files with no reviewable changes (1)
  • docs-data/redpanda-property-changes-v26.1.7-to-v26.1.8.json
👮 Files not reviewed due to content moderation or server errors (1)
  • modules/reference/attachments/redpanda-properties-v26.1.9.json


=== features_auto_finalization

Whether the cluster active logical version is advanced automatically once all nodes have been upgraded (true), or only in response to an explicit request via the Admin API (false). When false, the cluster remains able to downgrade to the previous version until finalization is requested. Setting this to false is an Enterprise feature and requires a valid license. Note: if upgrade was performed with this set to false and the cluster is ready to finalize, flipping this to true does not reliably trigger finalization. Leave this set to false and use the Admin API to finalize; once the upgrade is complete this can be set back to true to restore automatic finalization for future upgrades.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Remove license metadata from the behavior description

Line 6271 mixes behavioral guidance with licensing metadata. Keep the description behavior-focused and rely on the existing license note block below.

Proposed fix
-Whether the cluster active logical version is advanced automatically once all nodes have been upgraded (true), or only in response to an explicit request via the Admin API (false). When false, the cluster remains able to downgrade to the previous version until finalization is requested. Setting this to false is an Enterprise feature and requires a valid license. Note: if upgrade was performed with this set to false and the cluster is ready to finalize, flipping this to true does not reliably trigger finalization. Leave this set to false and use the Admin API to finalize; once the upgrade is complete this can be set back to true to restore automatic finalization for future upgrades.
+Whether the cluster active logical version is advanced automatically once all nodes have been upgraded (true), or only in response to an explicit request via the Admin API (false). When false, the cluster remains able to downgrade to the previous version until finalization is requested. Note: if upgrade was performed with this set to false and the cluster is ready to finalize, flipping this to true does not reliably trigger finalization. Leave this set to false and use the Admin API to finalize; once the upgrade is complete this can be set back to true to restore automatic finalization for future upgrades.

As per coding guidelines, "In property descriptions, keep descriptions focused on behavior and avoid metadata information like version availability, cloud availability, license requirements, requires restart status, default values, and type information".

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Whether the cluster active logical version is advanced automatically once all nodes have been upgraded (true), or only in response to an explicit request via the Admin API (false). When false, the cluster remains able to downgrade to the previous version until finalization is requested. Setting this to false is an Enterprise feature and requires a valid license. Note: if upgrade was performed with this set to false and the cluster is ready to finalize, flipping this to true does not reliably trigger finalization. Leave this set to false and use the Admin API to finalize; once the upgrade is complete this can be set back to true to restore automatic finalization for future upgrades.
Whether the cluster active logical version is advanced automatically once all nodes have been upgraded (true), or only in response to an explicit request via the Admin API (false). When false, the cluster remains able to downgrade to the previous version until finalization is requested. Note: if upgrade was performed with this set to false and the cluster is ready to finalize, flipping this to true does not reliably trigger finalization. Leave this set to false and use the Admin API to finalize; once the upgrade is complete this can be set back to true to restore automatic finalization for future upgrades.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@modules/reference/partials/properties/cluster-properties.adoc` at line 6271,
Remove license/entitlement metadata from the property description that starts
"Whether the cluster active logical version is advanced automatically..." by
deleting the sentence "Setting this to false is an Enterprise feature and
requires a valid license." and any similar licensing phrasing; keep only
behavior-focused guidance (including the note about finalization behavior), and
rely on the existing separate license note block below to document licensing
requirements.

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.

1 participant