Skip to content

[ML] Fix registering home screen panel#79080

Merged
peteharverson merged 7 commits intoelastic:masterfrom
walterra:ml-fix-register
Oct 7, 2020
Merged

[ML] Fix registering home screen panel#79080
peteharverson merged 7 commits intoelastic:masterfrom
walterra:ml-fix-register

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented Oct 1, 2020

Summary

Fixes #78965.

Fixes a race condition on plugin load which could lead to registered features not showing up in other parts of Kibana.

Originally we had the bundle size down to 40Kb. With some other later PRs related to shared code and this fix we're at 64Kb. In relation to the 40Kb that's quite the increase, but the important bit here is we fixed the race condition in time for 7.10 FF and are still way down from the previous ~800Kb we had.

Checklist

Delete any items that are not applicable to this PR.

@pheyos
Copy link
Copy Markdown
Member

pheyos commented Oct 1, 2020

Checking stability in a flaky test runner job ... no failure in 50 runs ✔️

@pheyos
Copy link
Copy Markdown
Member

pheyos commented Oct 2, 2020

@elasticmachine merge upstream

@pheyos
Copy link
Copy Markdown
Member

pheyos commented Oct 2, 2020

Checked stability in another flaky test runner job: no failure in 50 runs ✔️

@walterra walterra self-assigned this Oct 4, 2020
@walterra walterra added :ml v7.10.0 v8.0.0 bug Fixes for quality problems that affect the customer experience regression release_note:skip Skip the PR/issue when compiling release notes labels Oct 4, 2020
@walterra
Copy link
Copy Markdown
Contributor Author

walterra commented Oct 5, 2020

@elasticmachine merge upstream

@walterra walterra marked this pull request as ready for review October 5, 2020 14:56
@walterra walterra requested a review from a team as a code owner October 5, 2020 14:56
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@walterra
Copy link
Copy Markdown
Contributor Author

walterra commented Oct 5, 2020

@elasticmachine merge upstream

Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

@peteharverson
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@qn895
Copy link
Copy Markdown
Member

qn895 commented Oct 6, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

async chunks size

id before after diff
ml 11.3MB 11.3MB -9.7KB

page load bundle size

id before after diff
ml 59.4KB 65.6KB +6.2KB

History

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

@peteharverson peteharverson merged commit f1905ec into elastic:master Oct 7, 2020
peteharverson pushed a commit to peteharverson/kibana that referenced this pull request Oct 7, 2020
* [ML] Import tweak.

* [ML] Attempt at fixing flaky registering.

* [ML] Re-enable tests.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
peteharverson added a commit that referenced this pull request Oct 7, 2020
* [ML] Import tweak.

* [ML] Attempt at fixing flaky registering.

* [ML] Re-enable tests.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Walter Rafelsberger <walter@elastic.co>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@walterra walterra deleted the ml-fix-register branch October 17, 2020 10:31
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* [ML] Import tweak.

* [ML] Attempt at fixing flaky registering.

* [ML] Re-enable tests.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience :ml regression release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ML] File upload card sometimes missing on Kibana home page

7 participants