Skip to content

Conversation

@mjwolf
Copy link
Contributor

@mjwolf mjwolf commented Sep 24, 2025

1. What does this PR do?

This advanced RFC 0049 to stage 2, and adds entity fields to the ECS schema as beta fields.

2. Which ECS fields are affected/introduced?

This adds new entity fields as beta, and modifies host, user, cloud and service fieldset by adding entity fields to them through re-use.

3. Why is this change necessary?

This advanced RFC 0049 through the RFC process, and makes the entity fields part of the ECS schema, which will unlock use cases using these fields.

4. Have you added/updated documentation?

YES

5. Have you built ECS and committed any newly generated files?

YES

6. Have you run the ECS validation tests locally?

YES

7. Anything else for the reviewers?

Commit Message

Advance RFC 0049 (entity store) to stage 2

This advanced RFC 0049 to stage 2, and adds entity fields to the ECS
schema as beta fields.
@mjwolf mjwolf requested a review from a team as a code owner September 24, 2025 22:51
@github-actions
Copy link

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@github-actions
Copy link

@github-actions
Copy link

github-actions bot commented Sep 24, 2025

@kgeller
Copy link
Contributor

kgeller commented Sep 25, 2025

7. Anything else for the reviewers?

These changes do not completely implement the RFC requirements in the schema yet. There are tooling changes needed to allow user.target.entity.* reuse to work. user.target is itself a reused field, and adding a reuse field to another reuse field is not working properly.

@mjwolf did you try altering the subsets file to allow this? I'm pretty sure that's how we allow the complex nested re-use ie client.user.group. I think you just need to add entity to the user.target section https://github.com/elastic/ecs/blob/main/schemas/subsets/main.yml#L600

Using the subsets file, add entity under target, for the fieldsets which
use this. For `host`, entity is not nested under target, as it doesn't
have this field.

This matches the requirments of the RFC.
@mjwolf mjwolf merged commit 0d0fa54 into elastic:main Sep 26, 2025
7 checks passed
mjwolf added a commit to mjwolf/ecs that referenced this pull request Sep 26, 2025
Advance RFC 0049 (entity store) to stage 2
mjwolf added a commit that referenced this pull request Sep 26, 2025
Advance RFC 0049 (entity store) to stage 2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants