Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Fixed flaky test suite, that was breaking github action build #284

Merged
merged 1 commit into from
Nov 14, 2019

Conversation

galkk
Copy link
Contributor

@galkk galkk commented Nov 14, 2019

Description of changes:
Attempt to fix flaky test
The idea is that we clean and initialize test info in @before, instead of pair @before/@after

let's see if actions will work

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@galkk galkk changed the title Let's see action Fixed flaky test suite, that was breaking github action build Nov 14, 2019
Copy link
Member

@dai-chen dai-chen left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@galkk galkk requested a review from chloe-zh November 14, 2019 19:39
Copy link
Member

@chloe-zh chloe-zh left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@galkk galkk merged commit 33c5cad into opendistro-for-elasticsearch:master Nov 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants