-
Notifications
You must be signed in to change notification settings - Fork 8.4k
Added mapping for actor and target entity id, related.entity #224483
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
base: main
Are you sure you want to change the base?
Conversation
Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security) |
src/platform/packages/shared/kbn-alerts-as-data-utils/src/field_maps/alert_field_map.ts
Outdated
Show resolved
Hide resolved
...gration/spaces_only/tests/alerting/group4/alerts_as_data/alerts_as_data_dynamic_templates.ts
Outdated
Show resolved
Hide resolved
…s/alerting/group4/alerts_as_data/alerts_as_data_dynamic_templates.ts
* 2.0. | ||
*/ | ||
|
||
import { alertsFieldMap8190 } from '../8.19.0'; |
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.
Is that correct to import from 8.19? or should I've imported from earlier release?
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 this is fine, given that the new mappings are always based on those of the previous release.
src/platform/packages/shared/kbn-alerts-as-data-utils/src/field_maps/alert_field_map.ts
Outdated
Show resolved
Hide resolved
f0269b4
to
ffc9f2b
Compare
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.
response ops changes lgtm
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.
LGTM! I have ran the PR locally and tested that indeed I am able to filter by the added fields following the instructions provided.
💚 Build Succeeded
Metrics [docs]
History
|
🤖 Jobs for this PR can be triggered through checkboxes. 🚧
ℹ️ To trigger the CI, please tick the checkbox below 👇
|
Summary
resolve #221036
Adds new fields mapping:
actor.entity.id
,target.entity.id
,related.entity
of typekeyword
to the security's alert index.These fields are necessary for querying the alerts index as part of the graph feature (#221037).
related.entity
is currently waiting for ECS approval - elastic/ecs#2360actor.entity.id
,target.entity.id
are added only to security's alertsHow this PR effects:
actor.entity.id
,target.entity.id
,related.entity
Video 🎥
**Before:**Screen.Recording.2025-06-18.at.19.19.18.mov
After:
Screen.Recording.2025-06-18.at.19.21.51.mov
Besides the immediate effect of the ability to fetch data using these fields, this PR is a requirement to move forward with upcoming features like: #224783
Example video of upcoming use 🎥
Screen.Recording.2025-06-29.at.19.42.23.mov
How to test locally
Alerts
page in kibana. Update the date-picker to include data from a year ago. Then check one of the alerts details opening the right-side flyout and find the "Graph preview" section in it.Checklist
Identify risks