Skip to content
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

bug/Set a longer timeout for ES integration tests using allegro #2242

Closed
ghost opened this issue Jan 20, 2020 · 2 comments
Closed

bug/Set a longer timeout for ES integration tests using allegro #2242

ghost opened this issue Jan 20, 2020 · 2 comments
Labels
Milestone

Comments

@ghost
Copy link

ghost commented Jan 20, 2020

Among the many issues spotted when running Elasticsearch integration tests on the CI environment, there is one that has not been addressed yet, and is failing some of the tests with errors.

See empty pull requests #2237 and #2238 (used to test whether concurrency of ES tests works now that the installation directory is specified).

The issue at hand is a timeout to actually spin up the Elasticsearch runtime at Allegro's level.

This experimental task is about trying to specify a much larger timeout to accommodate with the machine's limitations.

@kolotu kolotu assigned ghost Jan 20, 2020
@kolotu kolotu added the bug label Jan 20, 2020
@ghost
Copy link
Author

ghost commented Jan 20, 2020

Looks like the jenkins builds on the two concurrent "empty" PRs I used for testing this is green. Waiting for finalization of the travis build in case something else comes up - otherwise we might have a winner here finally (although only continued usage will tell if those measures were good enough).

@ghost
Copy link
Author

ghost commented Jan 20, 2020

Merged into concurrent empty PRs and tested building concurrently again - all green for now, so closing with hope.
We can still iterate later if 1' not enough.
See also #2229.

@ghost ghost closed this as completed Jan 20, 2020
@kolotu kolotu added this to the 0.13 milestone Jan 20, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant