Skip to content

[7.x] [Logs UI] Generalize ML module management (#50662)#52777

Merged
weltenwort merged 1 commit intoelastic:7.xfrom
weltenwort:backport/7.x/pr-50662
Dec 11, 2019
Merged

[7.x] [Logs UI] Generalize ML module management (#50662)#52777
weltenwort merged 1 commit intoelastic:7.xfrom
weltenwort:backport/7.x/pr-50662

Conversation

@weltenwort
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

This abstracts the specific job details out of the ML module management hooks to enable re-use with the upcoming categorization module.

closes elastic#50322
@weltenwort weltenwort requested a review from a team as a code owner December 11, 2019 16:49
@weltenwort weltenwort added the backport This PR is a backport of another PR label Dec 11, 2019
@weltenwort weltenwort self-assigned this Dec 11, 2019
@weltenwort
Copy link
Copy Markdown
Member Author

why does it suddenly request code-owner reviews on backport PRs?

@simianhacker
Copy link
Copy Markdown
Member

@weltenwort That's weird... do you need me to just approve?

@weltenwort
Copy link
Copy Markdown
Member Author

I think I can just override the merge restriction - but thanks for the offer. Looks like the CODEOWNERS file was accidentally merged to 7.x a few hours ago.

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@weltenwort weltenwort merged commit 737ce28 into elastic:7.x Dec 11, 2019
@weltenwort weltenwort deleted the backport/7.x/pr-50662 branch December 11, 2019 18:26
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.

3 participants