-
Notifications
You must be signed in to change notification settings - Fork 539
mount static geoip db into es container #8065
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
mount static geoip db into es container #8065
Conversation
fixes flaky test that depended on the geoip db being downloaded. closes elastic#7260
|
This pull request does not have a backport label. Could you fix it @stuartnelson3? 🙏
NOTE: |
axw
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice and simple, thanks :)
Can you please revert b5d52d7 as part of this PR?
🌐 Coverage report
|
This reverts commit ec38207.
|
shouldn't we back port this to 8.2, 8.1 and 7.17, to avoid having these failures happening regularly? |
|
Don't need 8.1 since 8.2.0 is released, but yes for the others. |
* mount static geoip db into es container fixes flaky test that depended on the geoip db being downloaded. closes #7260 * Revert "systemtest: wait for geoIP database availability (#7272)" This reverts commit ec38207. (cherry picked from commit dc846d9) Co-authored-by: stuart nelson <stuartnelson3@gmail.com>
fixes flaky test that depended on the geoip db
being downloaded.
closes #7260
I suppose the way to test this would be to run it several times?