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

[Infrastructure UI] Logs tabs error handling #160100

Closed
crespocarlos opened this issue Jun 21, 2023 · 7 comments
Closed

[Infrastructure UI] Logs tabs error handling #160100

crespocarlos opened this issue Jun 21, 2023 · 7 comments
Labels
Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability GA good first issue low hanging fruit Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team technical debt Improvement of the software architecture and operational architecture

Comments

@crespocarlos
Copy link
Contributor

Summary

With the introduction of the customization of the logs columns passed by the Hosts view to the LogStream, a call to saved objects is made to retrieve the default log view indices.

The logs tabs now depend on this call for the LogStream to start loading the logs. Should this request fail, the logs tab will remain in loading state forever.

Acceptance Criteria

  • Logs tabs (hosts view and asset details) should consider the error state and show an error message, allowing users to try reloading the content again or fallback to DEFAULT_LOG_VIEW
@crespocarlos crespocarlos added Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Feature:ObsHosts Hosts feature within Observability labels Jun 21, 2023
@elasticmachine
Copy link
Contributor

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

@roshan-elastic
Copy link

Hey @crespocarlos - do you know if this issue still needs resolving?

@roshan-elastic
Copy link

@smith - maybe something to fix during usual bug fixing allocation? I've moved to the team backlog but not prioritised (will wait to hear from Carlos if it is still an issue)

@crespocarlos crespocarlos added the technical debt Improvement of the software architecture and operational architecture label Nov 10, 2023
@crespocarlos
Copy link
Contributor Author

@roshan-elastic we're still not handling errors in the logs tab. Added the tech debt label.

@roshan-elastic
Copy link

@smith - you OK prioritising this as you normally would do for tech debt? Let me know if you need some input.

@smith smith added Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Nov 14, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@smith smith added the good first issue low hanging fruit label Jul 10, 2024
@roshan-elastic
Copy link

Closing as the logs embeddable will be superseded

@roshan-elastic roshan-elastic closed this as not planned Won't fix, can't repro, duplicate, stale Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability GA good first issue low hanging fruit Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@smith @crespocarlos @elasticmachine @roshan-elastic