-
Notifications
You must be signed in to change notification settings - Fork 431
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
[RFC] Extend event.kind allowed values with asset #2178
Changes from 2 commits
cfc0a77
bb93fb5
00b3111
be7a29d
ae3c065
9f32dde
037129a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,158 @@ | ||
# 0038: Extend event.kind allowed values with asset | ||
<!-- Leave this ID at 0000. The ECS team will assign a unique, contiguous RFC number upon merging the initial stage of this RFC. --> | ||
|
||
- Stage: **0 (strawperson)** <!-- Update to reflect target stage. See https://elastic.github.io/ecs/stages.html --> | ||
- Date: **TBD** <!-- The ECS team sets this date at merge time. This is the date of the latest stage advancement. --> | ||
|
||
<!-- | ||
As you work on your RFC, use the "Stage N" comments to guide you in what you should focus on, for the stage you're targeting. | ||
Feel free to remove these comments as you go along. | ||
--> | ||
|
||
<!-- | ||
Stage 0: Provide a high level summary of the premise of these changes. Briefly describe the nature, purpose, and impact of the changes. ~2-5 sentences. | ||
--> | ||
|
||
<!-- | ||
Stage 1: If the changes include field additions or modifications, please create a folder titled as the RFC number under rfcs/text/. This will be where proposed schema changes as standalone YAML files or extended example mappings and larger source documents will go as the RFC is iterated upon. | ||
--> | ||
|
||
<!-- | ||
Stage X: Provide a brief explanation of why the proposal is being marked as abandoned. This is useful context for anyone revisiting this proposal or considering similar changes later on. | ||
--> | ||
|
||
## Fields | ||
|
||
This RFC proposes extending the list of allowed values for `event.kind` with an additional value, `asset`. | ||
|
||
<!-- | ||
Stage 1: Describe at a high level how this change affects fields. Include new or updated yml field definitions for all of the essential fields in this draft. While not exhaustive, the fields documented here should be comprehensive enough to deeply evaluate the technical considerations of this change. The goal here is to validate the technical details for all essential fields and to provide a basis for adding experimental field definitions to the schema. Use GitHub code blocks with yml syntax formatting, and add them to the corresponding RFC folder. | ||
--> | ||
|
||
<!-- | ||
Stage 2: Add or update all remaining field definitions. The list should now be exhaustive. The goal here is to validate the technical details of all remaining fields and to provide a basis for releasing these field definitions as beta in the schema. Use GitHub code blocks with yml syntax formatting, and add them to the corresponding RFC folder. | ||
--> | ||
|
||
## Usage | ||
|
||
For the Entity Analytics project, we are looking to identify user, host, and other types of entity assets. The `event.kind` field with `asset` value will be used to identity these types of documents. It has also been proposed that we combine this with other fields to have a more exact definition. For example, a user document may have these event fields specified: | ||
taylor-swanson marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
* `event.kind: asset` | ||
* `event.category: iam` | ||
* `event.type: user,info` | ||
taylor-swanson marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
<!-- | ||
Stage 1: Describe at a high-level how these field changes will be used in practice. Real world examples are encouraged. The goal here is to understand how people would leverage these fields to gain insights or solve problems. ~1-3 paragraphs. | ||
--> | ||
|
||
## Source data | ||
|
||
An example of a user entity document, with `event.kind: asset`: | ||
|
||
```json | ||
{ | ||
"input": { | ||
"type": "entity-analytics" | ||
}, | ||
"@timestamp": "2023-02-23T20:03:17.489Z", | ||
"event": { | ||
"agent_id_status": "verified", | ||
"ingested": "2023-02-23T20:03:18Z", | ||
"provider": "Azure AD", | ||
"kind": "asset", | ||
"action": "user-discovered", | ||
"category": "iam", | ||
taylor-swanson marked this conversation as resolved.
Show resolved
Hide resolved
|
||
"type": [ | ||
"user", | ||
"info" | ||
], | ||
"dataset": "entityanalytics_azure.users" | ||
}, | ||
"user": { | ||
"full_name": "Test User", | ||
"phone": [ | ||
"1235559999" | ||
], | ||
"name": [ | ||
"[email protected]" | ||
], | ||
"id": "b3e92fd4-0269-49ae-8a26-812e952bb7ad", | ||
"email": "[email protected]" | ||
}, | ||
"labels": { | ||
"identity_source": "entity-analytics-entityanalytics_azure.users-c1745610-d9a4-437f-a8f8-82ab45ace54a" | ||
} | ||
} | ||
``` | ||
|
||
<!-- | ||
Stage 1: Provide a high-level description of example sources of data. This does not yet need to be a concrete example of a source document, but instead can simply describe a potential source (e.g. nginx access log). This will ultimately be fleshed out to include literal source examples in a future stage. The goal here is to identify practical sources for these fields in the real world. ~1-3 sentences or unordered list. | ||
--> | ||
|
||
<!-- | ||
Stage 2: Included a real world example source document. Ideally this example comes from the source(s) identified in stage 1. If not, it should replace them. The goal here is to validate the utility of these field changes in the context of a real world example. Format with the source name as a ### header and the example document in a GitHub code block with json formatting, or if on the larger side, add them to the corresponding RFC folder. | ||
--> | ||
|
||
<!-- | ||
Stage 3: Add more real world example source documents so we have at least 2 total, but ideally 3. Format as described in stage 2. | ||
--> | ||
|
||
## Scope of impact | ||
|
||
<!-- | ||
Stage 2: Identifies scope of impact of changes. Are breaking changes required? Should deprecation strategies be adopted? Will significant refactoring be involved? Break the impact down into: | ||
* Ingestion mechanisms (e.g. beats/logstash) | ||
* Usage mechanisms (e.g. Kibana applications, detections) | ||
* ECS project (e.g. docs, tooling) | ||
The goal here is to research and understand the impact of these changes on users in the community and development teams across Elastic. 2-5 sentences each. | ||
--> | ||
|
||
## Concerns | ||
|
||
<!-- | ||
Stage 1: Identify potential concerns, implementation challenges, or complexity. Spend some time on this. Play devil's advocate. Try to identify the sort of non-obvious challenges that tend to surface later. The goal here is to surface risks early, allow everyone the time to work through them, and ultimately document resolution for posterity's sake. | ||
--> | ||
|
||
<!-- | ||
Stage 2: Document new concerns or resolutions to previously listed concerns. It's not critical that all concerns have resolutions at this point, but it would be helpful if resolutions were taking shape for the most significant concerns. | ||
--> | ||
|
||
<!-- | ||
Stage 3: Document resolutions for all existing concerns. Any new concerns should be documented along with their resolution. The goal here is to eliminate risk of churn and instability by ensuring all concerns have been addressed. | ||
--> | ||
|
||
## People | ||
|
||
The following are the people that consulted on the contents of this RFC. | ||
|
||
* @taylor-swanson | author | ||
* @SourinPaul | sponsor | ||
* @MikePaquette | sponsor | ||
|
||
<!-- | ||
Who will be or has been consulted on the contents of this RFC? Identify authorship and sponsorship, and optionally identify the nature of involvement of others. Link to GitHub aliases where possible. This list will likely change or grow stage after stage. | ||
|
||
e.g.: | ||
|
||
* @Yasmina | author | ||
* @Monique | sponsor | ||
* @EunJung | subject matter expert | ||
* @JaneDoe | grammar, spelling, prose | ||
* @Mariana | ||
--> | ||
|
||
|
||
## References | ||
|
||
<!-- Insert any links appropriate to this RFC in this section. --> | ||
|
||
### RFC Pull Requests | ||
|
||
<!-- An RFC should link to the PRs for each of it stage advancements. --> | ||
|
||
* Stage 0: https://github.com/elastic/ecs/pull/2178 | ||
|
||
<!-- | ||
* Stage 1: https://github.com/elastic/ecs/pull/NNN | ||
... | ||
--> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
# Licensed to Elasticsearch B.V. under one or more contributor | ||
# license agreements. See the NOTICE file distributed with | ||
# this work for additional information regarding copyright | ||
# ownership. Elasticsearch B.V. licenses this file to you under | ||
# the Apache License, Version 2.0 (the "License"); you may | ||
# not use this file except in compliance with the License. | ||
# You may obtain a copy of the License at | ||
# | ||
# http://www.apache.org/licenses/LICENSE-2.0 | ||
# | ||
# Unless required by applicable law or agreed to in writing, | ||
# software distributed under the License is distributed on an | ||
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY | ||
# KIND, either express or implied. See the License for the | ||
# specific language governing permissions and limitations | ||
# under the License. | ||
--- | ||
- name: event | ||
title: Event | ||
group: 2 | ||
short: Fields breaking down the event details. | ||
description: > | ||
The event fields are ... # Omitted for brevity | ||
type: group | ||
|
||
fields: | ||
- name: kind | ||
level: core | ||
type: keyword | ||
short: The kind of the event. The highest categorization field in the hierarchy. | ||
description: > | ||
This is one of four ECS ... # Omitted for brevity | ||
example: alert | ||
allowed_values: | ||
- name: asset | ||
description: > | ||
This value indicates events whose primary purpose is to store an inventory of | ||
assets/entities and their attributes. Assets/entities are objects that are | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it should be clearer here about what We can take it even further and clarify this value is reserved and to be used by features of the Elastic stack, like |
||
expected to be subjects of detailed analysis within the system. | ||
|
||
Examples include lists of user identities or accounts ingested from directory | ||
services such as Active Directory (AD), inventory of hosts pulled from | ||
configuration management databases (CMDB), and lists of cloud storage buckets | ||
pulled from cloud provider API's. | ||
|
||
`event.kind: asset` is not used for normal system events or logs that are coming | ||
from an asset/entity, nor is it used for system events or logs coming from a | ||
directory or CMDB system. | ||
|
||
By contrast, `event.kind: enrichment`` is used to indicate events whose primary | ||
taylor-swanson marked this conversation as resolved.
Show resolved
Hide resolved
|
||
purpose is to enrich other objects represented within the system. For example, | ||
threat indicators (IOC's) and Geo IP information are used primarily for | ||
enrichment, and therefore would be indicated by `event.kind: enrichment`. |
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.
Since the scope of this change is small and the proposal is already well detailed, I propose we target stage 2 initially instead of stage 0.
By moving to stage 2,
event.kind: asset
will be added into ECS but as abeta
allowed value. We'll still have some flexibility to make adjustments if necessary.@taylor-swanson if you agree with targeting to stage 2, can you capture any necessary detail in the
Scope of impact
andConcerns
?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.
@ebeahan, That seems to be reasonable to me.
@SourinPaul, do we have anything specific to call out here for Scope of Impact or Concerns? I can add them to the PR once we have them.
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.
cc @MikePaquette in case you have any additional input here (see my comment for Sourin immediately above).
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.
Scope of Impact:
We expect to consistently use the
asset
value within the security solution across asset management, external asset integration, and persisting related metadata. This field may be further leveraged in o11y topology work as they also are exploring ways to extend the asset.* fields within ECS.Concerns:
There are no breaking changes identified. I do not have any concerns to document at this time.
@taylor-swanson, please let me know if I can elaborate anything further. Thanks for driving this.
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.
Thanks, @SourinPaul, I think this looks good.
I think this could wait to Stage 3 as Eric mentioned, but to add to his comment below, should we include verbiage in description for
asset
to say something on the lines of: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.
For
event:kind: asset
events, data pipelines (like the Azure AD user entity integration) will be populating this field, right?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.
Oh good point. I think when I read that initially, I was thinking "anything not our pipeline". I'll remove this line in that case, just keeping that its intended use is with the security solution.