Skip to content

Conversation

@alexey-ivanov-es
Copy link
Contributor

It seems the best way to fix #122949 is to use existing data stream reindex API. However, this API is located in the migrate x-pack plugin. This commit moves the system indices migration logic (REST handlers, transport actions, and task) to the migrate plugin.

Port of #123551

It seems the best way to fix elastic#122949 is to use existing data stream reindex API. However, this API is located in the migrate x-pack plugin. This commit moves the system indices migration logic (REST handlers, transport actions, and task) to the migrate plugin.

Port of elastic#123551
@alexey-ivanov-es alexey-ivanov-es added :Core/Infra/Core Core issues without another label >refactoring v9.0.0 labels Mar 13, 2025
@alexey-ivanov-es alexey-ivanov-es requested review from a team as code owners March 13, 2025 17:14
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Mar 13, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Contributor

@JVerwolf JVerwolf left a comment

Choose a reason for hiding this comment

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

LGTM, 🚢 it.

@alexey-ivanov-es alexey-ivanov-es merged commit 4ed9347 into elastic:9.0 Mar 14, 2025
16 checks passed
@alexey-ivanov-es alexey-ivanov-es deleted the move-system-indices-migration_9.0 branch March 14, 2025 14:39
@mark-vieira
Copy link
Contributor

Is there a reason this wasn't done in main? I ran into some weirdness recently with the fact that this lives in :server in main but the migrate module in 9.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >refactoring Team:Core/Infra Meta label for core/infra team v9.0.0 v9.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants