Skip to content

Conversation

@probakowski
Copy link
Contributor

This change fixes problem with GeoIpProcessor when there's GeoIpTaskState present in the cluster state but there's no database matching the one used by the processor. It can happen when there are some but not all databases already updated.

Marking it as non-issue as it's in code that hasn't been released yet.

@probakowski probakowski added >non-issue :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP v8.0.0 auto-backport Automatically create backport pull requests when merged v7.14.1 v7.15.0 labels Jul 5, 2021
@probakowski probakowski requested a review from martijnvg July 5, 2021 20:15
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Jul 5, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM

@probakowski probakowski merged commit 04a9282 into elastic:master Jul 6, 2021
@probakowski probakowski deleted the fix_geoip_with_no_custom_db branch July 6, 2021 05:54
elasticsearchmachine pushed a commit to elasticsearchmachine/elasticsearch that referenced this pull request Jul 6, 2021
This change fixes problem with GeoIpProcessor when there's GeoIpTaskState present in the cluster state but there's no database matching the one used by the processor. It can happen when there are some but not all databases already updated.
elasticsearchmachine pushed a commit to elasticsearchmachine/elasticsearch that referenced this pull request Jul 6, 2021
This change fixes problem with GeoIpProcessor when there's GeoIpTaskState present in the cluster state but there's no database matching the one used by the processor. It can happen when there are some but not all databases already updated.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
7.14
7.x

elasticsearchmachine added a commit that referenced this pull request Jul 6, 2021
* Fix GeoIpProcessor when there's no updated db (#74944)

This change fixes problem with GeoIpProcessor when there's GeoIpTaskState present in the cluster state but there's no database matching the one used by the processor. It can happen when there are some but not all databases already updated.

* fix compilation

Co-authored-by: Przemko Robakowski <[email protected]>
elasticsearchmachine added a commit that referenced this pull request Jul 6, 2021
* Fix GeoIpProcessor when there's no updated db (#74944)

This change fixes problem with GeoIpProcessor when there's GeoIpTaskState present in the cluster state but there's no database matching the one used by the processor. It can happen when there are some but not all databases already updated.

* fix compilation

Co-authored-by: Przemko Robakowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >non-issue Team:Data Management Meta label for data/management team v7.14.0 v7.15.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants