Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ML] Hide empty config lists in MlMetadata post migration #36421

Closed
davidkyle opened this issue Dec 10, 2018 · 2 comments
Closed

[ML] Hide empty config lists in MlMetadata post migration #36421

davidkyle opened this issue Dec 10, 2018 · 2 comments
Labels
:ml Machine learning

Comments

@davidkyle
Copy link
Member

davidkyle commented Dec 10, 2018 •
edited
Loading

Once all the job and datafeeds have been migrated don't write the empty lists in MlMetadata.toXContent otherwise you have this confusing and potentially alarming clusterstate where it looks like no jobs exist

GET /_cluster/state/metadata

ml : {
    jobs : [],
    datafeeds: []
}
@davidkyle davidkyle added the :ml Machine learning label Dec 10, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@davidkyle davidkyle mentioned this issue Dec 10, 2018
Closed
43 tasks
@davidkyle
Copy link
Member Author

Closing as this was fixed in #37493

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:ml Machine learning
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davidkyle @elasticmachine