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

Kuery bar suggestions values are broken across observability/security #95269

Closed
shahzad31 opened this issue Mar 24, 2021 · 5 comments
Closed

Kuery bar suggestions values are broken across observability/security #95269

shahzad31 opened this issue Mar 24, 2021 · 5 comments
Assignees
@lizozom
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 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability

Comments

@shahzad31
Copy link
Contributor

shahzad31 commented Mar 24, 2021 •
edited
Loading

Kibana version:
Master

I suspect PR #92861 broke kuery bar suggestion values in kuery bar in Uptime, logs/metrics and security.

This PR specifically requires kibana saved object index pattern for the api to work.

These solutions were using sort of dynamic index pattern where we would get fields for wildcard and pass those to autocomplete service.

image

API throws http://localhost:5601/api/kibana/suggestions/values/heartbeat-8*

404 not found.
@shahzad31 shahzad31 added bug Fixes for quality problems that affect the customer experience Team:AppServices labels Mar 24, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServices)

@shahzad31 shahzad31 added Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Mar 24, 2021
@andrewvc
Copy link
Contributor

Pinging @lizozom , thoughts on this? Can we back out the PR till we have a non-breaking solution here

@lizozom
Copy link
Contributor

lizozom commented Mar 24, 2021

@mattkime I'm going to revert #92861
Lets talk about this

@lizozom lizozom self-assigned this Mar 24, 2021
@lizozom
Copy link
Contributor

lizozom commented Mar 24, 2021

Reverting for now #95335

@shahzad31
Copy link
Contributor Author

Closing this, since this was fixed by above revert

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lizozom lizozom

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 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@andrewvc @lizozom @shahzad31 @elasticmachine