[g0rgamesh]

No description provided.

[coderabbitai]

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

• Source mode: Auto
• Repository: stackhpc/stackhpc-kayobe-config
• Branch: stackhpc/2026.1
• File: .coderabbit.yaml
• Commit: 96b68dc
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 0878dc0a-f77f-4808-b8c9-7d0cfbf1ea72

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

The change adds an operational procedure for draining a compute node, removing its services and control-plane records, deprovisioning its hardware, and checking for remaining references.

Changes: Compute node deprovisioning

Layer / File(s)	Summary
Inspect and drain workloads doc/source/operations/deprovision-compute.rst	The procedure checks compute services and instances, disables the compute service, drains ACTIVE instances, and describes manual cold migration for SHUTOFF, PAUSED, and ERROR instances.
Stop the node and clean control-plane records doc/source/operations/deprovision-compute.rst	The procedure stops Kolla services and containers, then removes the compute service, any resource provider, network agents, and OVN chassis.
Update inventory, deprovision hardware, and verify doc/source/operations/deprovision-compute.rst	The procedure updates Kayobe inventory and services, runs Bifrost hardware deprovisioning, rediscovers the overcloud inventory, and checks RabbitMQ connections, OVN encapsulations, compute services, and network agents.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: Moderate · previous review e8349

Resolve the paused/error server recovery instructions and verify the disk-erasure guarantee before merging. Otherwise, operators may proceed with incomplete draining or assume stronger data sanitization than the deployment provides.

Pre-merge checks: 3

Check name	Status	Explanation
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

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

• X
• Mastodon
• Reddit
• LinkedIn

CodeRabbit Verify

• Run CodeRabbit Verify to test the changes in this pull request.

Recent review info

Run configuration

• Source mode: Auto
• Repository: stackhpc/stackhpc-kayobe-config
• Branch: stackhpc/2026.1
• File: .coderabbit.yaml
• Commit: 96b68dc
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: f2fee539-76ca-47dd-9884-fbd64f3d8c55

Commits

Reviewing files that changed from the base of the PR and between 96b68dc and e8349db.

Files selected for processing: 1

• doc/source/operations/deprovision-compute.rst

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.