-
Notifications
You must be signed in to change notification settings - Fork 510
[system] Add pipeline to rename process.ppid in process data_stream #2610
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
[system] Add pipeline to rename process.ppid in process data_stream #2610
Conversation
|
Pinging @elastic/integrations (Team:Integrations) |
|
/test |
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
Migrate data from Agent 7.x to ECS 8.0 format. Agent 7.x produces process.ppid, but in order to comply with ECS 8.0 we want all data to use process.parent.pid. The process data stream lacked any tests so I added a system test to verify the pipeline works and validate the fields.yml. I found some undocumented fields. Relates #2363 #2512
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.
Looks fine to me, is this something we also want to change on the agent side?
packages/system/data_stream/process/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com>
It was changed in 8.0 in Beats (https://github.com/elastic/beats/blob/v8.0.0/metricbeat/module/system/process/process.go#L117-L118). So this ensure that data coming from 7.x Agent's will match. |
|
/test |
What does this PR do?
Migrate data from Agent 7.x to ECS 8.0 format.
Agent 7.x produces process.ppid, but in order to comply with ECS 8.0
we want all data to use process.parent.pid.
The process data stream lacked any tests so I added a system test to
verify the pipeline works and validate the fields.yml. I found some
undocumented fields.
Relates #2363 #2512
Checklist
changelog.ymlfile.Related issues
References