Skip to content

Conversation

@JVerwolf
Copy link
Contributor

@JVerwolf JVerwolf commented Jan 30, 2025

Reverts #121120 with a fix for #120566, which was causing the system index migration service to fail when security was enabled.

This PR removes a potential cause of data loss when migrating system indices. It does this by changing the way we set a "write-block" on the system index to migrate - now using a dedicated transport request rather than a settings update. Furthermore, we no longer delete the write-block prior to deleting the index, as this was another source of potential data loss. Additionally, we now remove the block if the migration fails.

This is a backport of #121327
Jira: ES-9724

@JVerwolf JVerwolf added >bug :Core/Infra/Core Core issues without another label labels Jan 30, 2025
@JVerwolf JVerwolf requested a review from a team January 30, 2025 19:57
@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team v8.19.0 labels Jan 30, 2025
@JVerwolf JVerwolf changed the title Fix privileges in system migration WRITE block Fix privileges for system index migration WRITE block Jan 30, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Hi @JVerwolf, I've created a changelog YAML for you.

JVerwolf and others added 3 commits January 30, 2025 12:19
… github.com:JVerwolf/elasticsearch into bugfix/fix-priveledges-in-system-migration-block-8x
Copy link
Contributor

@alexey-ivanov-es alexey-ivanov-es left a comment

Choose a reason for hiding this comment

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

LGTM

@JVerwolf
Copy link
Contributor Author

@elasticmachine update branch

@JVerwolf JVerwolf enabled auto-merge (squash) February 10, 2025 22:06
@JVerwolf JVerwolf added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 10, 2025
@JVerwolf
Copy link
Contributor Author

@elasticmachine update branch

@JVerwolf JVerwolf merged commit 3b238be into elastic:8.x Feb 11, 2025
15 checks passed
@JVerwolf JVerwolf deleted the bugfix/fix-priveledges-in-system-migration-block-8x branch February 11, 2025 16:26
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!) backport >bug :Core/Infra/Core Core issues without another label Team:Core/Infra Meta label for core/infra team v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants