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

[Logs UI] Replace legacy elasticsearch client usage #73978

Closed
weltenwort opened this issue Jul 31, 2020 · 7 comments · Fixed by #100716
Closed

[Logs UI] Replace legacy elasticsearch client usage #73978

weltenwort opened this issue Jul 31, 2020 · 7 comments · Fixed by #100716
Assignees
Labels
chore Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Milestone

Comments

@weltenwort
Copy link
Member

Summary

With the removal of the legacy Elasticsearch client in the Kibana Platform, we need to replace all usage with calls to the data plugin's search or with the new client as described in the migration guide.

@weltenwort weltenwort added chore Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Jul 31, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@jasonrhodes
Copy link
Member

@weltenwort correct me if I'm wrong, but if we're moving to the data plugin in the client and using search strategies, we will no longer need to worry about this migration, right? See: #76677

@weltenwort
Copy link
Member Author

That should cover most of it, yes. To be sure I'd still recommend to search through our plugin codebase, though.

@joshdover
Copy link
Contributor

I'm doing an audit today on where we are on #83910 and it appears that there are no longer any usages of the legacy client in the infra plugin. Could someone confirm and if so, close this issue and update the table in #83910?

Thanks all!

@joshdover
Copy link
Contributor

@joshdover
Copy link
Contributor

@jasonrhodes could we get these last bits prioritized? Should be a pretty small lift to get this closed.

@jasonrhodes jasonrhodes added this to the Logs UI 7.14 milestone May 4, 2021
@jasonrhodes
Copy link
Member

@sgrodzicki I think we should just knock this out ASAP when @weltenwort or @Kerry350 have some bandwidth in between other tasks, since this should be pretty small.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants