-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
filebeat/module/cef: import genericised cef dashboards from integrations #32766
Conversation
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
This pull request is now in conflicts. Could you fix it? 🙏
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the dashboards require some changes to work with Beats.
- Beats uses
event.dataset
instead ofdata_stream.dataset
. - The index pattern is
filebeat-*
rather thanlogs-*
. - Are the object IDs the same as what's in the Fleet integration? We should make them unique to prevent collisions Kibana with Fleets saved objects.
I have deconflicted the object IDs with clonedash (second commit) and the index patterns have been adjusted (last commit). I'll look into the What may still be an issue is that the beats dashboards have |
1. upload to 8.3.3 kibana as an integration assets bundle 2. filebeat export dashboard
...k/filebeat/module/cef/_meta/kibana/8/dashboard/cef-1ef1282f-382b-4bdd-9cb1-810b8f083e74.json
Show resolved
Hide resolved
…ons (#32766) Used clonedash to construct new identifiers, then adjusted title formats, index patterns and query targets. Added the dashboard assets from the integrations config page on kibana and then exported with filebeat export dashboard.
What does this PR do?
This adds the dashboards that exist in the CEF integration to the CEF module in filebeat.
Why is it important?
The existing dashboards do not work in the general case as they were originally written against the ArcSight fields that the original module worked with.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
See screenshots in elastic/integrations#3526.
Logs