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

Increase timeout for relocation tests #46554

Merged
merged 3 commits into from Sep 11, 2019

Conversation

@dnhatn
Copy link
Contributor

commented Sep 10, 2019

There's nothing wrong in the logs from these failures. I think 30 seconds might not be enough to relocate shards with many documents as CI is quite slow. This change increases the timeout to 60 seconds for these relocation tests. It also dumps the hot threads in case of timed out.

Closes #46526
Closes #46439

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Sep 10, 2019

@dnhatn dnhatn requested a review from original-brownbear Sep 11, 2019

@tlrx
tlrx approved these changes Sep 11, 2019
Copy link
Member

left a comment

LGTM, thanks for taking care of this @dnhatn

@original-brownbear
Copy link
Member

left a comment

LGTM2 :)thanks @dnhatn !

@dnhatn

This comment has been minimized.

Copy link
Contributor Author

commented Sep 11, 2019

@dnhatn dnhatn merged commit 16ac1b1 into elastic:master Sep 11, 2019

8 checks passed

CLA All commits in pull request signed
Details
elasticsearch-ci/1 Build finished.
Details
elasticsearch-ci/2 Build finished.
Details
elasticsearch-ci/bwc Build finished.
Details
elasticsearch-ci/default-distro Build finished.
Details
elasticsearch-ci/docs Build finished.
Details
elasticsearch-ci/oss-distro-docs Build finished.
Details
elasticsearch-ci/packaging-sample Build finished.
Details

@dnhatn dnhatn deleted the dnhatn:increase-timeout-relocation-test branch Sep 11, 2019

dnhatn added a commit that referenced this pull request Sep 12, 2019
Increase timeout for relocation tests (#46554)
There's nothing wrong in the logs from these failures. I think 30
seconds might not be enough to relocate shards with many documents as CI
is quite slow. This change increases the timeout to 60 seconds for these
relocation tests. It also dumps the hot threads in case of timed out.

Closes #46526
Closes #46439
dnhatn added a commit that referenced this pull request Sep 12, 2019
Increase timeout for relocation tests (#46554)
There's nothing wrong in the logs from these failures. I think 30
seconds might not be enough to relocate shards with many documents as CI
is quite slow. This change increases the timeout to 60 seconds for these
relocation tests. It also dumps the hot threads in case of timed out.

Closes #46526
Closes #46439
turackangal added a commit to turackangal/elasticsearch that referenced this pull request Sep 14, 2019
Increase timeout for relocation tests (elastic#46554)
There's nothing wrong in the logs from these failures. I think 30
seconds might not be enough to relocate shards with many documents as CI
is quite slow. This change increases the timeout to 60 seconds for these
relocation tests. It also dumps the hot threads in case of timed out.

Closes elastic#46526
Closes elastic#46439

@colings86 colings86 added v7.4.0 and removed v7.4.1 labels Sep 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.