Skip to content

Conversation

@nielsbauman
Copy link
Contributor

In some cases, deleting unassigned indices in multi-project mode would result in an error log due to a MultiProjectPendingException which was caused by the use of Metadata#getProject(). We fix that by passing the correct (potentially null) project metadata instead.

In some cases, deleting unassigned indices in multi-project mode would
result in an error log due to a `MultiProjectPendingException` which was
caused by the use of `Metadata#getProject()`. We fix that by passing the
correct (potentially null) project metadata instead.
@nielsbauman nielsbauman requested review from a team and ywangd May 6, 2025 11:49
@nielsbauman nielsbauman added >non-issue :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. Team:Distributed Indexing Meta label for Distributed Indexing team v9.1.0 labels May 6, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@nielsbauman nielsbauman enabled auto-merge (squash) May 7, 2025 08:12
@nielsbauman nielsbauman merged commit eaa7329 into elastic:main May 7, 2025
17 checks passed
@nielsbauman nielsbauman deleted the fix-index-deletion branch May 7, 2025 09:24
ywangd pushed a commit to ywangd/elasticsearch that referenced this pull request May 9, 2025
In some cases, deleting unassigned indices in multi-project mode would
result in an error log due to a `MultiProjectPendingException` which was
caused by the use of `Metadata#getProject()`. We fix that by passing the
correct (potentially null) project metadata instead.
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request May 9, 2025
In some cases, deleting unassigned indices in multi-project mode would
result in an error log due to a `MultiProjectPendingException` which was
caused by the use of `Metadata#getProject()`. We fix that by passing the
correct (potentially null) project metadata instead.
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request May 12, 2025
In some cases, deleting unassigned indices in multi-project mode would
result in an error log due to a `MultiProjectPendingException` which was
caused by the use of `Metadata#getProject()`. We fix that by passing the
correct (potentially null) project metadata instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >non-issue Team:Distributed Indexing Meta label for Distributed Indexing team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants