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

esArchiver: retry kibana config update #43987

Merged
merged 5 commits into from
Aug 27, 2019

Conversation

dmlemeshko
Copy link
Member

@dmlemeshko dmlemeshko commented Aug 26, 2019

Summary

This closes #43221, closes #43776, closes #42550

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko dmlemeshko requested a review from spalger August 26, 2019 16:28
@dmlemeshko dmlemeshko marked this pull request as ready for review August 26, 2019 16:28
@dmlemeshko dmlemeshko added release_note:skip Skip the PR/issue when compiling release notes v7.4.0 labels Aug 26, 2019
@dmlemeshko dmlemeshko requested a review from LeeDr August 26, 2019 16:42
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@LeeDr
Copy link

LeeDr commented Aug 27, 2019

@dmlemeshko have we seen this PR resolve the issue? Have you found any logs where the update failed and was retried successfully? Maybe it doesn't fail often enough and we'll just have to try this and see?

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I'm not 100% sure this will fix things, but I can't imagine it making things worse.

@dmlemeshko
Copy link
Member Author

@dmlemeshko have we seen this PR resolve the issue? Have you found any logs where the update failed and was retried successfully? Maybe it doesn't fail often enough and we'll just have to try this and see?

I ran ciGroup7 60x times on CI and checked around 20 reports - retry was never called. Since recently we got the same error in other tests (#43776, #42550) I plan to monitor build-stats with retry in master/7.x.

@dmlemeshko dmlemeshko merged commit 48c6109 into elastic:master Aug 27, 2019
dmlemeshko added a commit to dmlemeshko/kibana that referenced this pull request Aug 27, 2019
* [services/es_archiver] retry uiSettings update

* run x-pack-ciGroup7 30x times

* Revert "run x-pack-ciGroup7 30x times"

This reverts commit 80e199c.

* [saved_object_api_integration/common/services] add retry service to the set

* add retry service for x-pack api tests
spalger pushed a commit that referenced this pull request Aug 28, 2019
* [services/es_archiver] retry uiSettings update

* run x-pack-ciGroup7 30x times

* Revert "run x-pack-ciGroup7 30x times"

This reverts commit 80e199c.

* [saved_object_api_integration/common/services] add retry service to the set

* add retry service for x-pack api tests
@spalger
Copy link
Contributor

spalger commented Aug 28, 2019

7.x/7.4: 145945e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment