Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[elasticsearch] Don't break when some indices are closed #36677

Closed
miltonhultgren opened this issue Sep 26, 2023 · 4 comments
Closed

[elasticsearch] Don't break when some indices are closed #36677

miltonhultgren opened this issue Sep 26, 2023 · 4 comments
Labels
bug Feature:Stack Monitoring Module:elasticsearch Elasticsearch Beats modules module Team:Monitoring Stack Monitoring team

Comments

@miltonhultgren
Copy link
Contributor

elastic/elasticsearch#88752 describes an issue with the/_stats API when there are any indices that are closed, that can be worked around by sending different query parameters.
We should update the Elasticsearch module to account for this possibility so that collection does not stop simply because there are closed indices.
@miltonhultgren miltonhultgren added module Feature:Stack Monitoring Team:Infra Monitoring UI - DEPRECATED Infrastructure Monitoring UI team - DEPRECATED - Use Team:Monitoring Module:elasticsearch Elasticsearch Beats modules labels Sep 26, 2023
@bczifra bczifra mentioned this issue Oct 11, 2023
Closed
@smith smith removed the Team:Infra Monitoring UI - DEPRECATED Infrastructure Monitoring UI team - DEPRECATED - Use Team:Monitoring label Jan 26, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 26, 2024
@miltonhultgren miltonhultgren added the Team:Monitoring Stack Monitoring team label Jan 29, 2024
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 29, 2024
@miltonhultgren
Copy link
Contributor Author

In verifying this, the issue is rather that it fails when stats for a closed index are collected because the closed index is behind an alias.
It's most likely that we need to handle this error more gracefully and inform the user to either open the index or remove it from the alias to allow collection to proceed.

@BenB196
Copy link

BenB196 commented Mar 19, 2024

I haven't tested it, but would supporting setting ?forbid_closed_indices=false as described, elastic/elasticsearch#88752 (comment), fix the issue without requiring user intervention and not breaking stats collection?

@miltonhultgren
Copy link
Contributor Author

Yes, I have a PR here doing this #38408

@smith smith added the bug label Apr 3, 2024
@miltonhultgren
Copy link
Contributor Author

The above PR was merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Feature:Stack Monitoring Module:elasticsearch Elasticsearch Beats modules module Team:Monitoring Stack Monitoring team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@smith @miltonhultgren @BenB196