Skip to content

fix(DOC-1959): remove orphaned raft_election_timeout_ms section from cluster properties#1712

Open
Feediver1 wants to merge 2 commits into
mainfrom
fix/doc-1959-remove-raft-election-timeout-section
Open

fix(DOC-1959): remove orphaned raft_election_timeout_ms section from cluster properties#1712
Feediver1 wants to merge 2 commits into
mainfrom
fix/doc-1959-remove-raft-election-timeout-section

Conversation

@Feediver1
Copy link
Copy Markdown
Contributor

Summary

Follow-up to #1611. Closes out DOC-1959.

PR #1611 added an exclude_from_docs override for raft_election_timeout_ms in docs-data/property-overrides.json. That override succeeded at the source-JSON layer (modules/reference/attachments/redpanda-properties-v26.1.6.json on main no longer contains the property), but the rendered modules/reference/partials/properties/cluster-properties.adoc partial was never regenerated. As a result, the published Cluster Properties page still showed a raft_election_timeout_ms entry, leaving DOC-1959 unsatisfied.

This PR deletes the orphaned 47-line section directly from the partial. Future auto-docs regeneration runs will preserve the deletion because the source v26.1.6 JSON no longer contains the property.

Verify

  • On the Netlify deploy preview, navigate to Reference > Properties > Cluster Properties and confirm Ctrl+F raft_election_timeout_ms returns no hits.
  • Confirm election_timeout_ms (the correct user-facing property) is unchanged.

Test plan

  • Netlify build passes
  • grep -rn raft_election_timeout_ms modules/ returns zero hits

🤖 Generated with Claude Code

…roperties partial

Follow-up to #1611. The PR added an `exclude_from_docs` override for
`raft_election_timeout_ms` in `docs-data/property-overrides.json`, which
correctly removed the property from the source v26.1.6 attachment JSON.
However, the rendered `cluster-properties.adoc` partial was never
regenerated, so the property still appeared on the published cluster
properties page.

Delete the orphaned section directly. Future auto-docs regeneration runs
will preserve the deletion because the source JSON no longer contains
the property.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@Feediver1 Feediver1 requested a review from a team as a code owner May 26, 2026 15:51
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 26, 2026

Review Change Stack

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.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 02fa391b-d1d9-4dc3-992a-953d8624194a

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 removes the documentation section for the raft_election_timeout_ms cluster property from the cluster properties reference file. The deletion eliminates a 47-line documentation block, after which the raft_enable_longest_log_detection property documentation now appears immediately following the quota_manager_gc_sec property entry. The change is a straightforward documentation cleanup with no alterations to code entities or functional definitions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • redpanda-data/docs#1359: Makes adjacent documentation formatting changes in cluster-properties.adoc without modifying property content or semantics.
  • redpanda-data/docs#1620: Modifies cluster property documentation related to cluster-properties.adoc, specifically including adjustments to the raft_election_timeout_ms section.

Suggested reviewers

  • micheleRP
  • joe-redpanda
  • paulohtb6
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: removing an orphaned raft_election_timeout_ms section from cluster properties documentation.
Description check ✅ Passed The description provides comprehensive context, rationale, and verification steps, though it omits the Jira ticket link and check box selection from the template.
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 fix/doc-1959-remove-raft-election-timeout-section

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

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 26, 2026

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit b5ab982
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/6a15c2496d8cfb0008f32e0d
😎 Deploy Preview https://deploy-preview-1712--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.

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.

2 participants