Skip to content

[7.x] [ML] have DELETE analytics ignore stats failures and clean up unused stats (#60776)#60784

Merged
benwtrent merged 2 commits intoelastic:7.xfrom
benwtrent:backport/7.x/pr-60776
Aug 6, 2020
Merged

[7.x] [ML] have DELETE analytics ignore stats failures and clean up unused stats (#60776)#60784
benwtrent merged 2 commits intoelastic:7.xfrom
benwtrent:backport/7.x/pr-60776

Conversation

@benwtrent
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

…stats (elastic#60776)

When deleting an analytics configuration, the request MIGHT fail if
the .ml-stats index does not exist or is in strange state (shards unallocated).

Instead of making the request fail, we should log that we were unable to delete the stats docs and then
have them cleaned up in the 'delete_expire_data' janitorial process.
@benwtrent benwtrent added :ml Machine learning backport labels Aug 5, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/ml-core (:ml)

@benwtrent benwtrent merged commit bc17afc into elastic:7.x Aug 6, 2020
@benwtrent benwtrent deleted the backport/7.x/pr-60776 branch August 6, 2020 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :ml Machine learning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants