Skip to content

Conversation

@uri-weisman
Copy link
Contributor

@uri-weisman uri-weisman commented Dec 4, 2025

1. What does this PR do?

Adding the target field to be nested under host.

2. Which ECS fields are affected/introduced?

host.target.*

3. Why is this change necessary?

To support the security graph and align with user and entity schemas.

4. Have you added/updated documentation?

YES / NO / N/A

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

@uri-weisman uri-weisman requested a review from a team as a code owner December 4, 2025 12:09
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🤖 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 bot commented Dec 4, 2025

Documentation changes preview: https://docs-v3-preview.elastic.dev/elastic/ecs/pull/2571/reference/

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🔍 Preview links for changed docs

@uri-weisman uri-weisman changed the title Adding host.target to schema Adding target to host schema Dec 4, 2025
@uri-weisman
Copy link
Contributor Author

uri-weisman commented Dec 4, 2025

FYI @kfirpeled @alexreal1314

Copy link
Contributor

@trisch-me trisch-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mjwolf could you check this PR as well?

@uri-weisman
Copy link
Contributor Author

@trisch-me @mjwolf This change is important to support https://github.com/elastic/security-team/issues/14151 for 9.3

@mjwolf mjwolf merged commit c916aa9 into elastic:main Dec 10, 2025
7 checks passed
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.

3 participants