Skip to content

[7.x] Improve stability of GeoIpDownloaderIT test suite. (#74558)#75175

Open
elasticsearchmachine wants to merge 1 commit intoelastic:7.17from
elasticsearchmachine:backport/7.x/pr-74558
Open

[7.x] Improve stability of GeoIpDownloaderIT test suite. (#74558)#75175
elasticsearchmachine wants to merge 1 commit intoelastic:7.17from
elasticsearchmachine:backport/7.x/pr-74558

Conversation

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Backports the following commits to 7.x:

Adjust GeoIpDownloaderIT test suit to wait for managed databases files
to be removed after each test.

After each test geoip downloader is disabled,  which should eventually
remove the managed geoip database files. This happens in the background.
However a new test starts that assumes that the builtin databases are used
then that test can fail, because expected assertions will fail. The changes
in this commit should address this.

Closes elastic#74358
@probakowski probakowski added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jul 9, 2021
@danhermann danhermann added v8.1.0 and removed v7.16.0 labels Oct 27, 2021
@mark-vieira mark-vieira changed the base branch from 7.16 to 7.17 December 16, 2021 14:40
@mark-vieira mark-vieira added v8.2.0 and removed v8.1.0 labels Feb 2, 2022
@arteam arteam added v7.17.3 and removed v7.17.2 labels Mar 28, 2022
@mark-vieira mark-vieira added v8.5.0 and removed v8.4.0 labels Jul 27, 2022
@csoulios csoulios added v8.6.0 and removed v8.5.0 labels Sep 21, 2022
@kingherc kingherc added v8.7.0 and removed v8.6.0 labels Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v7.17.3 v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.