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.

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

Elasticsearch integration: missing index_stats alias on index mappings #119984

Closed
klacabane opened this issue Nov 30, 2021 · 4 comments
Closed

Elasticsearch integration: missing index_stats alias on index mappings #119984

klacabane opened this issue Nov 30, 2021 · 4 comments
Labels
bug Fixes for quality problems that affect the customer experience Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@klacabane
Copy link
Contributor

klacabane commented Nov 30, 2021

The index_stats property is used to query the indices from the index documents to power the SM elasticsearch Indices tab. We're currently missing an alias definition on that property in the elasticsearch integration package which causes the code path to return empty results[1].

Note that running a similar query in the dev tools will return a 400 error which appears to be swallowed in our code. We should investigate if there's any reason behind that and consider returning an erroneous status code instead.

{
  "error" : {
    "root_cause" : [
      {
        "type" : "illegal_argument_exception",
        "reason" : "no mapping found for index_stats.index in order to collapse on"
      }
    ],
  ....
  "status" : 400
}

AC

  • indices tab loads corresponding documents from the index stream
  • (optional) API endpoint returns an erroneous http response when mandatory mapping is not defined

[1]
Screenshot 2021-11-30 at 18 41 31

@klacabane klacabane added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Nov 30, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@matschaffer
Copy link
Contributor

matschaffer commented Dec 1, 2021

Would @neptunian 's work to try to get us to metrics-* (#119112) also fix this issue maybe?

@klacabane
Copy link
Contributor Author

I think Integrations-related issues are out of this PR scope, so tracking this separately

@klacabane
Copy link
Contributor Author

Fixed in elastic/integrations#3928

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

3 participants