Skip to content

Conversation

@masseyke
Copy link
Member

EnterpriseGeoIpDownloaderIT fails, but rarely (see #115163). We're currently unable to run the test repeatedly to attempt to reproduce failure using -Dtests.iters. This test cleans up the database configurations after the test completes so that it is in a state that can run again. I also suspect that this change might fix the failures (I don't have any direct evidence of that, but I was able to reproduce the failure in that ticket when I artificially ran this test without properly cleaning up).

@masseyke masseyke added >test Issues or PRs that are addressing/adding tests :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP v9.0.0 v8.18.1 v8.19.0 v9.1.0 labels Mar 31, 2025
@masseyke masseyke requested a review from joegallo March 31, 2025 14:58
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Mar 31, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@masseyke masseyke added the auto-backport Automatically create backport pull requests when merged label Mar 31, 2025
@masseyke masseyke merged commit 7a9edb5 into elastic:main Mar 31, 2025
16 of 17 checks passed
@masseyke masseyke deleted the EnterpriseGeoIpDownloaderIT-cleanup-method branch March 31, 2025 19:28
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.0
8.18 Commit could not be cherrypicked due to conflicts
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 125958

masseyke added a commit to masseyke/elasticsearch that referenced this pull request Mar 31, 2025
masseyke added a commit to masseyke/elasticsearch that referenced this pull request Mar 31, 2025
(cherry picked from commit 7a9edb5)

# Conflicts:
#	modules/ingest-geoip/src/internalClusterTest/java/org/elasticsearch/ingest/geoip/EnterpriseGeoIpDownloaderIT.java
@masseyke
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.x
8.18

Questions ?

Please refer to the Backport tool documentation

masseyke added a commit to masseyke/elasticsearch that referenced this pull request Mar 31, 2025
(cherry picked from commit 7a9edb5)

# Conflicts:
#	modules/ingest-geoip/src/internalClusterTest/java/org/elasticsearch/ingest/geoip/EnterpriseGeoIpDownloaderIT.java
elasticsearchmachine pushed a commit that referenced this pull request Mar 31, 2025
…5994)

(cherry picked from commit 7a9edb5)

# Conflicts:
#	modules/ingest-geoip/src/internalClusterTest/java/org/elasticsearch/ingest/geoip/EnterpriseGeoIpDownloaderIT.java
elasticsearchmachine pushed a commit that referenced this pull request Mar 31, 2025
…5990)

(cherry picked from commit 7a9edb5)

# Conflicts:
#	modules/ingest-geoip/src/internalClusterTest/java/org/elasticsearch/ingest/geoip/EnterpriseGeoIpDownloaderIT.java
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request Apr 1, 2025
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 backport pending :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.18.1 v8.19.0 v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants