Skip to content

Add RFC for extended entity fields#2598

Merged
trisch-me merged 16 commits into
elastic:mainfrom
uri-weisman:entity_fields_9_4_rfc_strawperson
May 6, 2026
Merged

Add RFC for extended entity fields#2598
trisch-me merged 16 commits into
elastic:mainfrom
uri-weisman:entity_fields_9_4_rfc_strawperson

Conversation

@uri-weisman
Copy link
Copy Markdown
Contributor

@uri-weisman uri-weisman commented Mar 31, 2026

Summary

Introduce a 9.4 strawperson RFC for new entity attributes, lifecycle, and relationship leaves, and propose enabling entity.risk reuse for normalized entities.

Made-with: Cursor
@uri-weisman uri-weisman requested a review from a team March 31, 2026 18:29
@github-actions
Copy link
Copy Markdown
Contributor

🤖 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!)

Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
- Add stage-0 header comments and link to RFC 0049
- Merge entity-type guidance into Fields table; keyword (list) for arrays
- Document flat relationship object shape and Field Re-use nesting
- Fix entity.last_seen_timestamp reference; restore administered_by row
- Match People, Source data, References, and Concerns style to other RFCs
- Update usage example entity.type to array form

Made-with: Cursor
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Copy link
Copy Markdown

@chemamartinez chemamartinez left a comment

Choose a reason for hiding this comment

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

LGTM! My review is mostly focused on conversation's I've been involved and from an integration perspective so please wait for the rest of reviewer's approval.

Copy link
Copy Markdown

@erikh-elastic erikh-elastic left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md
Copy link
Copy Markdown
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.

lgtm in general with some minor feedback

uri-weisman and others added 3 commits April 26, 2026 12:48
Co-authored-by: Alexandra Konrad <alexandra.konrad@elastic.co>
Co-authored-by: Alexandra Konrad <alexandra.konrad@elastic.co>
Co-authored-by: Alexandra Konrad <alexandra.konrad@elastic.co>
Comment thread rfcs/text/0054-extend-entity-fields.md Outdated
Comment thread rfcs/text/0054-extend-entity-fields.md
Comment thread rfcs/text/0054-extend-entity-fields.md
@uri-weisman
Copy link
Copy Markdown
Contributor Author

@andrewkroh can I please ask you to have a second look?

@uri-weisman
Copy link
Copy Markdown
Contributor Author

We decided to wait with follow up PRs that utilize the fields defined in this RFC until it's merged

Can I get another reviewer with write access to be able to merge this one? @trisch-me asking for your assistance.

@trisch-me
Copy link
Copy Markdown
Contributor

@taylor-swanson @andrewkroh could you check this out?

Copy link
Copy Markdown
Contributor

@bhapas bhapas left a comment

Choose a reason for hiding this comment

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

LGTM

@uri-weisman
Copy link
Copy Markdown
Contributor Author

Thanks everyone, @trisch-me I need your assistance with pushing this one?

@trisch-me trisch-me merged commit 7745837 into elastic:main May 6, 2026
7 checks passed
@trisch-me
Copy link
Copy Markdown
Contributor

@uri-weisman can you create a follow up PR with fields put into yaml files? Or do you need help with it?

@uri-weisman
Copy link
Copy Markdown
Contributor Author

@uri-weisman can you create a follow up PR with fields put into yaml files? Or do you need help with it?

Will create one, thanks!

@uri-weisman uri-weisman deleted the entity_fields_9_4_rfc_strawperson branch May 6, 2026 16:22
@narph
Copy link
Copy Markdown

narph commented May 11, 2026

@uri-weisman can you create a follow up PR with fields put into yaml files? Or do you need help with it?

Will create one, thanks!

@uri-weisman Is the PR already in? We want to make sure the ECS fields are added in before we make changes to our integrations, so we can reduce future friction or potential breaking changes for users.

@uri-weisman
Copy link
Copy Markdown
Contributor Author

Hey @narph, IIUC @trisch-me is working on introducing the follow up PR - ticket.

@trisch-me
Copy link
Copy Markdown
Contributor

hey @narph - the PR is there, but actual fields are not yet in ECS, I was going to introduce them, but there is a technical question I need to solve first, I hope it will be ready soon

@taylor-swanson
Copy link
Copy Markdown
Contributor

We want to make sure the ECS fields are added in before we make changes to our integrations, so we can reduce future friction or potential breaking changes for users.

@narph, just as a heads up, the dynamic ECS templates in elasticsearch won't be updated until the next stack release, so any new fields that don't already match the dynamic template won't map properly if relying on that method. Looking at the list of fields, entity.lifecycle.last_activity (date) is one of those fields. The field can be mapped statically through external: ecs as long as the ECS reference in the integration is updated to the next ECS release that includes these fields.

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.

8 participants