Skip to content

[v/25.2] manage/k8s: document decommission timing (--decommission-wait-interval)#1764

Closed
david-yu wants to merge 3 commits into
v/25.2from
dyu/backport-1761-decommission-v25.2
Closed

[v/25.2] manage/k8s: document decommission timing (--decommission-wait-interval)#1764
david-yu wants to merge 3 commits into
v/25.2from
dyu/backport-1761-decommission-v25.2

Conversation

@david-yu

Copy link
Copy Markdown
Contributor

Backport of #1761 to v/25.2.

Adds the "Tune automatic decommission timing" section to the Kubernetes decommission page:

  • --decommission-wait-interval (Operator, via additionalCmdFlags) — the Decommission controller's requeue interval (RequeueAfter).
  • decommissionRequeueTimeout / decommissionAfter (Helm sidecar).
  • A TIP cross-reference from the Operator install step, the shell-correct --set "additionalCmdFlags={...}" quoting, and guidance for adjusting the intervals.

Backport note

The Operator-example rewrite from #1761 (the "do not add sideCars.brokerDecommissioner here" paragraph + <1> callout) is main-only and intentionally omitted here: v/25.2's Operator example uses the version-appropriate brokerDecommissioner sidecar, so backporting that paragraph would contradict the example on this branch.

Verified that all cross-references resolve on v/25.2: <<decommission-timing>>, <<Automated>>, and reference:{cluster,tunable}-properties.adoc#... (the latter via the existing properties/cluster-properties.adoc page-aliases, already used elsewhere on this branch).

🤖 Generated with Claude Code

…t-interval, RequeueAfter)

Backport of #1761 to v/25.2. Adds the "Tune automatic decommission timing"
section (--decommission-wait-interval for the Operator; decommissionRequeueTimeout
and decommissionAfter for the Helm sidecar), a TIP cross-reference, and the
shell-correct --set "additionalCmdFlags={...}" quoting.

The Operator-example rewrite from #1761 is main-only and intentionally omitted:
v/25.2's Operator example uses the version-appropriate brokerDecommissioner
sidecar, so backporting that paragraph/callout would contradict the example.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Jun 24, 2026

Copy link
Copy Markdown

Deploy Preview for redpanda-docs-preview ready!

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

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: 9ab8cfaa-6e1c-4e9a-be01-2be7607ac6b0

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dyu/backport-1761-decommission-v25.2

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.

david-yu and others added 2 commits June 24, 2026 11:00
The new Netlify build image (noble-new-builds) defaults to node 22 / npm
10.9.3 when no version is pinned, which fails `npm install` on this branch
with `ERR_INVALID_ARG_TYPE: The "from" argument must be of type string.
Received undefined`. This branch's deps install cleanly on node 20 (the
version pinned in .github/workflows/test-docs.yml, where the Antora build
passes) but not on node 22. This branch had no netlify.toml, so add a
minimal one pinning node 20 and matching main's NODE_OPTIONS. Restores the
broken Netlify preview build; no content changes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Empty commit to re-run the Netlify deploy preview after the build cache
was cleared. The node-20 pin is already in place; this forces a fresh
dependency install (no poisoned node_modules cache).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@david-yu

Copy link
Copy Markdown
Contributor Author

Superseded by #1765. Recreated on a fresh branch so the Netlify preview builds from a clean dependency cache — this branch's preview cache was poisoned by the recent Netlify build-image upgrade (noble-new-builds) and kept failing npm install with ERR_INVALID_ARG_TYPE even after the node-20 pin. The new branch has no prior (poisoned) cache, so it installs clean like GHA and like the v/25.3 backport (#1763).

@david-yu david-yu closed this Jun 24, 2026
@david-yu david-yu deleted the dyu/backport-1761-decommission-v25.2 branch June 24, 2026 21:32
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.

1 participant