Skip to content

feat: Support standalone nodes in BHE - BED-7577#213

Open
StephenHinck wants to merge 1 commit intomainfrom
BED-7577
Open

feat: Support standalone nodes in BHE - BED-7577#213
StephenHinck wants to merge 1 commit intomainfrom
BED-7577

Conversation

@StephenHinck
Copy link
Contributor

@StephenHinck StephenHinck commented Mar 5, 2026

Once SpecterOps/BloodHound#2470 and https://github.com/SpecterOps/bloodhound-enterprise/pull/1097 are merged, BloodHound Enterprise will no longer delete disconnected nodes automatically. They will instead be reconciled according to normal age-out reconciliation processes.

This PR can be retargeted to the v8.9.0 branch, once created.

Summary by CodeRabbit

  • Documentation
    • Removed OpenGraph FAQ section regarding node behavior after ingestion, including reconciliation details and workaround guidance.

@StephenHinck StephenHinck added the Waiting for BH PR Don't merge this doc update until the related BH feature is merged and released. label Mar 5, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f3a68a9e-7324-426d-b19f-20b1e513a25c

📥 Commits

Reviewing files that changed from the base of the PR and between 52d8006 and ba05616.

📒 Files selected for processing (1)
  • docs/opengraph/faq.mdx
💤 Files with no reviewable changes (1)
  • docs/opengraph/faq.mdx

Walkthrough

A 21-line FAQ accordion section describing OpenGraph node disappearance behavior after ingestion, including BHCE/BHE reconciliation details and workarounds, was removed from the documentation.

Changes

Cohort / File(s) Summary
Documentation cleanup
docs/opengraph/faq.mdx
Removed outdated FAQ accordion block regarding OpenGraph node disappearance behavior post-ingestion, including reconciliation details and recommended workarounds.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Old questions fade into the past,
FAQ entries cannot always last,
With whiskers twitching, content in flight,
We clean the docs and set things right! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions supporting standalone nodes in BHE, but the only change removes an FAQ section about OpenGraph nodes disappearing after ingestion—unrelated to BHE standalone nodes. Update the title to reflect the actual change, such as 'docs: Remove outdated OpenGraph FAQ about node reconciliation' or clarify which documentation changes support the BHE standalone node feature.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 BED-7577

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v8.9.0 Waiting for BH PR Don't merge this doc update until the related BH feature is merged and released.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants