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

Revert "x-pack/plugin/apm-data: download geoip DB on pipeline creation" #104505

Merged
merged 2 commits into from Jan 18, 2024

Conversation

axw
Copy link
Member

@axw axw commented Jan 18, 2024

Reverts #104501

Per discussion Slack, we should avoid eagerly downloading this for all users. The plugin is disabled at the moment, but we intend to enable it by default at some point. This does mean that enrichment will be skipped for initial documents, but we consider this to be acceptable. If users want different behaviour, they can enable eager geoip DB downloads.

@axw axw added >bug :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP v8.13.0 labels Jan 18, 2024
@axw axw requested a review from a team as a code owner January 18, 2024 09:27
@elasticsearchmachine
Copy link
Collaborator

@axw please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine elasticsearchmachine added external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team labels Jan 18, 2024
@elasticsearchmachine
Copy link
Collaborator

Hi @axw, I've created a changelog YAML for you.

@elasticsearchmachine
Copy link
Collaborator

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

@axw axw merged commit 4e223f6 into main Jan 18, 2024
16 checks passed
@axw axw deleted the revert-104501-apmdata-geoip-downloadoncreation branch January 18, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants