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] Support error.stack_trace #46211

Closed
felixbarny opened this issue Sep 20, 2019 · 5 comments · Fixed by #94906
Closed

[Logs UI] Support error.stack_trace #46211

felixbarny opened this issue Sep 20, 2019 · 5 comments · Fixed by #94906
Labels
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

Comments

@felixbarny
Copy link
Member

felixbarny commented Sep 20, 2019

The field error.stack_trace has recently been added to ECS: elastic/ecs#562

This issue is about supporting that field with the default log layout.

If error.stack_trace is available, it should be rendered after the message in a new line.

If it's possible, it would be great if the Logs app could also handle the case where error.stack_trace is an array, containing each line of the stack trace in a new array element. In that case, it should just join all lines like this event.get("error.stack_trace").join("\n"). For more background on that use case, see the ECS PR (elastic/ecs#562).

@felixbarny felixbarny 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 :Logs UI labels Sep 20, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-logs-ui

@felixbarny
Copy link
Member Author

We're planning to GA ECS loggers in 7.11.
Any chance we could prioritize this issue for the near-term?

@weltenwort
Copy link
Member

Thanks for the bump, I'll bring it up during the 7.12 planning.

@fzyzcjy
Copy link

fzyzcjy commented Mar 26, 2021

Hi, any updates?

@weltenwort
Copy link
Member

The PR @felixbarny has put together (#94906) looks like a big step into this direction. I hope to get around to reviewing and merging it soon ™️.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

5 participants