Skip to content

Conversation

@martijnvg
Copy link
Member

@martijnvg martijnvg commented Feb 2, 2022

This moves the data stream actions to the server module and moves the data streams module from ./x-pack/plugin/ to ./modules/

The data stream info and usage actions were moved to the xpack core project,
because the info and usage infrastructure only exists in xpack.

Also the qa module is replaced by yamlRestTest and javaRestTest directories.
The multi node qa test didn't really test multi node capabilities, so that was removed.
(The only test in this module tested data stream info and usage features)

This moves the data stream actions to the server module

The data stream info and usage actions were moved to the xpack core project,
because the info and usage infrastructure only exists in xpack.
@martijnvg martijnvg changed the title Move data streams from xpack plugin to a normal module. Change data streams from being a xpack module to be a normal module. Feb 2, 2022
* Move xpack related yaml tests back to xpack.
* Move info and usage related rest tests to xpack core module
* Move yaml tests to data stream module
* Move all qa tests to regular java rest tests inside data stream module
@martijnvg martijnvg added the :Data Management/Data streams Data streams and their lifecycles label Feb 2, 2022
@martijnvg martijnvg marked this pull request as ready for review February 2, 2022 12:03
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Feb 2, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Copy link
Contributor

@danhermann danhermann left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@martijnvg martijnvg added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 2, 2022
@elasticsearchmachine elasticsearchmachine merged commit 7f8b70a into elastic:master Feb 2, 2022
@martijnvg martijnvg deleted the move_data_streams_to_modules branch February 2, 2022 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Data Management/Data streams Data streams and their lifecycles >refactoring Team:Data Management Meta label for data/management team v8.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants