Skip to content

[7.x] [ML] job saved objects initialization (#82639)#83115

Merged
jgowdyelastic merged 1 commit intoelastic:7.xfrom
jgowdyelastic:backport/7.x/pr-82639
Nov 11, 2020
Merged

[7.x] [ML] job saved objects initialization (#82639)#83115
jgowdyelastic merged 1 commit intoelastic:7.xfrom
jgowdyelastic:backport/7.x/pr-82639

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

* [ML] job saved objects initialization

* fixing job count logic

* adding missing files

* attempting to fix build crash

* fixing kibana.json

* changes based on review

* removing accidentally added export

* adding intialization promise

* use finally so errors dont stop initialization

* function rename

* removing duplicate header

* adding job initialization count to log message

* adding error to log message

* moving initialization file

* moving intialization file back again to fix git stash issues

* removing .kibana index search

* creating internal saved object client

* code clean up

* removing commented code

* adding check for spaces enabled

* adding ids to saved objects

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
ml 1242 1243 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 6.7MB 6.7MB +270.0B

Distributable file count

id before after diff
default 43120 43123 +3

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

@jgowdyelastic jgowdyelastic added the backport This PR is a backport of another PR label Nov 11, 2020
@jgowdyelastic jgowdyelastic merged commit 0fc82f1 into elastic:7.x Nov 11, 2020
@jgowdyelastic jgowdyelastic deleted the backport/7.x/pr-82639 branch November 11, 2020 09:31
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 :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants