Skip to content

[7.x] [Reporting] Convert plugin setup and start to synchronous (#68326)#68932

Merged
tsullivan merged 1 commit intoelastic:7.xfrom
tsullivan:backport/7.x/pr-68326
Jun 11, 2020
Merged

[7.x] [Reporting] Convert plugin setup and start to synchronous (#68326)#68932
tsullivan merged 1 commit intoelastic:7.xfrom
tsullivan:backport/7.x/pr-68326

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

)

* [Reporting] Convert plugin setup and start to synchronous

* revert class conversion

* diff prettify

* Fix test mocks + add some plugin async helpers where needed

* no setupReady startReady needed

* Add plugin test for sync/error cases

* PR feedback on tests/setup logs

* rename symbol

Co-authored-by: Joel Griffith <joel.griffith@elastic.co>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@tsullivan tsullivan added the backport This PR is a backport of another PR label Jun 11, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tsullivan tsullivan merged commit cc9c54e into elastic:7.x Jun 11, 2020
@tsullivan tsullivan deleted the backport/7.x/pr-68326 branch June 11, 2020 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants