Skip to content
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] Remove process.ppid - Stage 0 #1337

Merged
merged 5 commits into from
May 3, 2021

Conversation

ebeahan
Copy link
Member

@ebeahan ebeahan commented Mar 31, 2021

Summary

Stage 0 proposal for deprecating and later removing the process.ppid field now that ECS also includes process.parent.pid.

Stage 0 (Strawperson) Criteria:

  • Discuss with domain or subject matter experts the utility of these changes
  • Discuss with the ECS team whether these changes seem appropriate for ECS

Preview of markdown proposal

@ebeahan ebeahan added the RFC label Mar 31, 2021
@ebeahan ebeahan self-assigned this Mar 31, 2021

### Ingestion mechanisms

APM, Beats, Elastic Agent, and any processors that populate `process.ppid` today will need to be identified and a migration plan to `process.parent.pid` established.
Copy link
Contributor

Choose a reason for hiding this comment

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

Will that a part of this RFC's process, or is that a separate effort?

Copy link
Member Author

Choose a reason for hiding this comment

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

Separate effort; updating those data sources will happen outside the RFC process.

Here we're noting the Elastic data sources that may be impacted when the field is removed from ECS. As this RFC advances, we'll continue communicating that this change is coming, especially to the teams/areas most impacted. The teams can then plan and implement any necessary changes.

Copy link
Contributor

@djptek djptek left a comment

Choose a reason for hiding this comment

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

LGTM

@ebeahan ebeahan merged commit 8dcc4f0 into elastic:master May 3, 2021
@ebeahan ebeahan deleted the rfc/remove-process.ppid branch May 3, 2021 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants