Skip to content

Use fallback GeoDB in Elasticsearch#335

Merged
mtojek merged 2 commits intoelastic:masterfrom
mtojek:use-fallback-db
Apr 23, 2021
Merged

Use fallback GeoDB in Elasticsearch#335
mtojek merged 2 commits intoelastic:masterfrom
mtojek:use-fallback-db

Conversation

@mtojek
Copy link
Copy Markdown
Contributor

@mtojek mtojek commented Apr 23, 2021

This PR disables the GeoDB downloader in Elasticsearch.

It's a similar PR to the one introduced in Beats: elastic/beats#25248

@mtojek mtojek self-assigned this Apr 23, 2021
- "script.context.ingest.cache_max_size=2000"
- "script.context.processor_conditional.cache_max_size=2000"
- "script.context.template.cache_max_size=2000"
- "ingest.geoip.downloader.enabled=false"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we test with 7.12.0 anywhere? I assume this property doesn't exist in that version and this would cause a failure.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The 7.12.0 is unsupported since we introduced changes for the Fleet Server. Unfortunately it was an acknowledged risk.

@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented Apr 23, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #335 updated

  • Start Time: 2021-04-23T12:20:55.361+0000

  • Duration: 23 min 45 sec

  • Commit: f5aebd5

Test stats 🧪

Test Results
Failed 0
Passed 316
Skipped 1
Total 317

Trends 🧪

Image of Build Times

Image of Tests

@mtojek mtojek requested a review from P1llus April 23, 2021 12:48
Copy link
Copy Markdown
Member

@P1llus P1llus left a comment

Choose a reason for hiding this comment

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

LGTM, I saw andrew's comment was already accepted.

@mtojek mtojek merged commit d9d1d2d into elastic:master Apr 23, 2021
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.

4 participants