Skip to content

Backport #11230 to 7.0: Skip flaky ES integration tests for now (#11230)#11249

Merged
webmat merged 1 commit intoelastic:7.0from
webmat:backport_11230_7.0
Mar 14, 2019
Merged

Backport #11230 to 7.0: Skip flaky ES integration tests for now (#11230)#11249
webmat merged 1 commit intoelastic:7.0from
webmat:backport_11230_7.0

Conversation

@webmat
Copy link
Copy Markdown
Contributor

@webmat webmat commented Mar 14, 2019

No description provided.

@webmat webmat requested a review from a team as a code owner March 14, 2019 14:09
@webmat webmat self-assigned this Mar 14, 2019
@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Mar 14, 2019

@ruflin @ycombinator FYI got started on this, while waiting on another build. Not 100% sure if it's appropriate for 7.0 as the previous commit builds are green.

But as far as I understand, this is related to timeouts, so perhaps a transient problem, depending on worker load levels? I get the feeling this is what was breaking for me in this 7.0 backport #11241

Copy link
Copy Markdown
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM, but let's also get the green light from @ycombinator

Copy link
Copy Markdown
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

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

LGTM. WFG.

@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Mar 14, 2019

Thanks guys. I'll merge at the end of the test run

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants