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

[Log Explorer] Show log.level by default in table (if available) #166680

Closed
ruflin opened this issue Sep 19, 2023 · 7 comments
Closed

[Log Explorer] Show log.level by default in table (if available) #166680

ruflin opened this issue Sep 19, 2023 · 7 comments
Assignees
Labels
Feature:LogsExplorer Logs Explorer feature Team:obs-ux-logs Observability Logs User Experience Team

Comments

@ruflin
Copy link
Contributor

ruflin commented Sep 19, 2023

As discussed in #166595, log.level is an important field. Ideally we would show this field also by default in the Log Explorer table. But I would prefer that it doesn't take up much space as a separate column and instead we could use icons or similar to indicate what log level a message is on or colors. The goal is that user can quickly spot what are for example warn or critical messages. More fields like event.severity could play into the state of this.

@ruflin ruflin added 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 Sep 19, 2023
@elasticmachine
Copy link
Contributor

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

@weltenwort
Copy link
Member

  • built-in for known message types (e.g. log level for syslog messages)
  • apply to a marker at a well-defined position in the line (e.g. a signs column at the beginning as found in many editors and IDEs)

@weltenwort
Copy link
Member

This could be implemented using the approach proposed in #169874. I'll mark it as blocked until a decision has been made on that.

@gbamparop gbamparop added Team:obs-ux-logs Observability Logs 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 9, 2023
@botelastic botelastic bot added the needs-team Issues missing a team label label Nov 9, 2023
@elasticmachine
Copy link
Contributor

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

@botelastic botelastic bot removed the needs-team Issues missing a team label label Nov 9, 2023
@gbamparop gbamparop added Feature:LogsExplorer Logs Explorer feature and removed Feature:Logs UI Logs UI feature labels Nov 10, 2023
@isaclfreire
Copy link

My proposal is to have log.level as part of the 'content' smart field / virtual columns. Figma here.
image
cc @felixbarny

@weltenwort
Copy link
Member

If we reserve a fixed width for it, why not put it into its own column? Then we'd just have to implement the nice formatter for it.

@ruflin
Copy link
Contributor Author

ruflin commented Dec 7, 2023

I think of this box as a "special content" box. It can be used for log.level but also for a severity number of no log.level exists or other things we can come up with in the future. So this is no a log.level column much more something we want to highlight from the content.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:LogsExplorer Logs Explorer feature Team:obs-ux-logs Observability Logs User Experience Team
Projects
None yet
Development

No branches or pull requests

5 participants