BP-2395: Microsoft Sentinel#246
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThis pull request adds comprehensive documentation for a new Microsoft Sentinel integration with BloodHound Enterprise. It includes navigation updates, a configuration guide covering setup and deployment, a usage guide explaining available dashboards and incident workflows, and an overview card in the integrations listing. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (1)
docs/integrations/microsoft/sentinel/configure.mdx (1)
84-103: Consider using a stable/released ARM template URL.The ARM template link currently points to a feature branch (
bloodhound) in an external repository. While the link is accessible, feature branches can be temporary, deleted, or renamed. Consider hosting the template at a stable/released location or a canonical stable branch to ensure the deployment link remains reliable long-term.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/integrations/microsoft/sentinel/configure.mdx` around lines 84 - 103, The ARM template link in the "Deploy the workbook and analytics rules template" step points to a feature branch URL (the raw.githubusercontent.com link containing "bloodhound" and "Package/mainTemplate.json"); update this to a stable/released location—for example point to a release tag, the repository's main branch, or a hosted canonical URL (or include the template in project release assets) so the Deploy to Azure link remains reliable long-term and does not depend on a transient feature branch.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/docs.json`:
- Around line 590-596: Normalize the indentation inside the JSON object where
"group": "Microsoft Sentinel" and its "pages" array are defined by removing the
extra leading whitespace before the second array entry so both entries have
consistent indentation; update the "pages" array formatting to align entries
(e.g., same number of spaces as the first entry) to improve readability.
In `@docs/integrations/microsoft/sentinel/configure.mdx`:
- Around line 66-68: The img tag inside the <Frame> element has the wrong alt
text ("Create Log Analytics Workspace"); update its alt attribute to accurately
describe the screenshot (e.g., "Entra ID application registration" or similar)
so the image reflects Step 2: Register a Microsoft Entra ID application; locate
the <img src="/images/integrations/microsoft/sentinel/image7.jpeg" alt="..."/>
and replace the alt string accordingly.
- Line 51: Replace the UI button text "Review and Create" in the docs with the
actual Azure Portal label "Review + create" to match the portal's UI; locate the
exact string "Review and Create" in the content
(docs/integrations/microsoft/sentinel/configure.mdx) and update it to "Review +
create".
---
Nitpick comments:
In `@docs/integrations/microsoft/sentinel/configure.mdx`:
- Around line 84-103: The ARM template link in the "Deploy the workbook and
analytics rules template" step points to a feature branch URL (the
raw.githubusercontent.com link containing "bloodhound" and
"Package/mainTemplate.json"); update this to a stable/released location—for
example point to a release tag, the repository's main branch, or a hosted
canonical URL (or include the template in project release assets) so the Deploy
to Azure link remains reliable long-term and does not depend on a transient
feature branch.
🪄 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: fbf9bd67-5623-4d85-98ab-8bf25cc7ebdd
⛔ Files ignored due to path filters (30)
docs/images/integrations/microsoft/sentinel/image14.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image15.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image16.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image17.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image18.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image19.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image20.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image21.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image22.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image23.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image24.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image26.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image27.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image28.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image29.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image30.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image31.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image32.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image33.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image34.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image35.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image36.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image37.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image38.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image39.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image4.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image40.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image41.pngis excluded by!**/*.pngdocs/images/integrations/microsoft/sentinel/image42.jpegis excluded by!**/*.jpegdocs/images/integrations/microsoft/sentinel/image7.jpegis excluded by!**/*.jpeg
📒 Files selected for processing (4)
docs/docs.jsondocs/integrations/microsoft/sentinel/configure.mdxdocs/integrations/microsoft/sentinel/use.mdxdocs/integrations/overview.mdx
zaton-netizen
left a comment
There was a problem hiding this comment.
Holding off approval as we are close to having the app published, recommend we review once it's live and adjust the documentation where needed, which at this point is difficult to assess without it being available in the marketplace. We expect this to be live by 5/29.
Purpose
This pull request (PR) adds docs for the Microsoft Sentinel integration for BloodHound Enterprise.
It's in draft because the instructions need to be updated after the integration has been published to the Azure Marketplace. For example, the steps for configuring and deploying the ARM templates may no longer be necessary.
Staging
https://specterops-bp-2395-ms-sentinel.mintlify.app/integrations/microsoft/sentinel/configure
Summary by CodeRabbit
Documentation