Skip to content

Conversation

@joegallo
Copy link
Contributor

@joegallo joegallo commented Apr 20, 2023

Follow up to #92951

The redact processor was originally introduced as a part of the ingest-common module, this PR promotes it to its own new x-pack-redact plugin.

@joegallo joegallo added WIP :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >refactoring Team:Data Management Meta label for data/management team v8.8.0 labels Apr 20, 2023
@joegallo joegallo requested review from dakrone and davidkyle April 20, 2023 17:09
This makes it easier to have IngestService instances in tests that can
just have a null matcherWatchdog.
Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM assuming CI is happy

Copy link
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

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

LTGM thanks @joegallo

@sophiec20
Copy link
Contributor

Thank you for this change...

The redact processor is now Elastic License 2.0 which is a pre-req for enabling Platinum license checks.

@joegallo joegallo removed the WIP label Apr 21, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >refactoring Team:Data Management Meta label for data/management team v8.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants