[auditbeat][fim] Update Auditbeat file_integrity module to default to auto backend with intelligent fallback#47498
Conversation
…o backend with intelligent fallback
|
Pinging @elastic/sec-linux-platform (Team:Security-Linux Platform) |
🤖 GitHub commentsJust comment with:
|
🔍 Preview links for changed docs |
…file integrity module
… test configuration for backend fallback
…ng to maintain test behavior
…n for file integrity tests
colleenmcginnis
left a comment
There was a problem hiding this comment.
I left some suggestions below on how to structure cumulative documentation.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
cc @elastic/ingest-docs please if anyone can take a look |
alexandra5000
left a comment
There was a problem hiding this comment.
LGTM with some minor changes 🙂
vishaangelova
left a comment
There was a problem hiding this comment.
Some suggestions to update the applies_to badges with the syntax for version ranges.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@auditbeat/module/file_integrity/_meta/docs.md`:
- Around line 103-104: The Linux backend applicability is inconsistent: the
block showing "{applies_to}`stack: preview 9.4.0+` `auto`" conflicts with the
other occurrence that marks Linux `auto` as GA; update this snippet so the Linux
backend uses the same GA applicability range as the other reference (replace the
preview applicability for `auto` with the GA range used elsewhere), ensuring the
two lines that mention `{applies_to}`stack: preview 9.4.0+` `auto`` and
`{applies_to}`stack: ga 9.0.0-9.3` `fsnotify`` are aligned and consistent.
🪄 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: a26faf8a-e57a-415f-9945-a0cdb2b82156
📒 Files selected for processing (2)
auditbeat/module/file_integrity/_meta/docs.mddocs/reference/auditbeat/auditbeat-module-file_integrity.md
Proposed commit message
The auto backend option now tries the backends in order until the best available option is found
Checklist
stresstest.shscript to run them under stress conditions and race detector to verify their stability../changelog/fragmentsusing the changelog tool.Disruptive User Impact
Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs