Skip to content

Cherry-pick #10723 to 7.0: Follow up renaming of ML modules#10732

Merged
kvch merged 1 commit intoelastic:7.0from
kvch:backport_10723_7.0
Feb 14, 2019
Merged

Cherry-pick #10723 to 7.0: Follow up renaming of ML modules#10732
kvch merged 1 commit intoelastic:7.0from
kvch:backport_10723_7.0

Conversation

@kvch
Copy link
Copy Markdown
Contributor

@kvch kvch commented Feb 13, 2019

Cherry-pick of PR #10723 to 7.0 branch. Original message:

Two renamings has happened which lead to the problem:

  • the Filebeat module apache2 has been renamed to apache
  • the ID of ML modules has been suffixed with _ecs

As it is still a fragile solution, I have opened an issue for the ML team to add the possibility of listing all available modules with additional meta data: elastic/kibana#30934

Two renamings has happened which lead to the problem:

* the Filebeat module apache2 has been renamed to apache
* the ID of ML modules has been suffixed with _ecs

As it is still a fragile solution, I have opened an issue for the ML team to add the possibility of listing all available modules with additional meta data: elastic/kibana#30934
(cherry picked from commit 7542db1)
@kvch kvch requested a review from a team as a code owner February 13, 2019 17:18
@ph
Copy link
Copy Markdown
Contributor

ph commented Feb 13, 2019

I am tracking down the doc failures.

@kaiyan-sheng
Copy link
Copy Markdown
Contributor

jenkins, test this

Copy link
Copy Markdown
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM

This probably needs a changelog as it will not land in beta1. But let's merge it first to get the builds back to green.

@kvch
Copy link
Copy Markdown
Contributor Author

kvch commented Feb 14, 2019

Failing tests are unrelated.

@kvch kvch merged commit e26b2a6 into elastic:7.0 Feb 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants