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

[APM Service Logs] Want log line details similar to the flyout on the Observability Logs Stream screen or on Kibana Discover #129279

Closed
cyrille-leclerc opened this issue Apr 3, 2022 · 10 comments
Labels
apm:logs design Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@cyrille-leclerc
Copy link
Contributor

cyrille-leclerc commented Apr 3, 2022

Describe the feature:

On the APM Service Logs view, want log line details similar to the details available as flyouts in Kibana Discover and in the Observability Logs App.

When considering the log line details of the Observability Logs Stream screen, I also want the improvements discussed on:

image

Describe a specific use case for the feature:

When observing my service using Elastic Observability APM, I want to explore the logs of this service, slice & dice the data...
Improving the visualization of the details of log lines is one of the improvements to achieve this goal.

@botelastic botelastic bot added the needs-team Issues missing a team label label Apr 3, 2022
@cyrille-leclerc cyrille-leclerc added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. design Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services apm:logs and removed needs-team Issues missing a team label labels Apr 3, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@elasticmachine
Copy link
Contributor

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

@cyrille-leclerc cyrille-leclerc changed the title [APM Service Logs] Want log line details similar to the flyout on the Observability logs app or to Kibana Discover [APM Service Logs] Want log line details similar to the flyout on the Observability Logs Stream screen or on Kibana Discover Apr 3, 2022
@formgeist
Copy link
Contributor

@cyrille-leclerc Isn't this a duplicate of #111325?

@weltenwort
Copy link
Member

@cyrille-leclerc could you specify what "log line details" are? A specific subset of the source fields? a subset of mapped fields? all of them?

@cyrille-leclerc
Copy link
Contributor Author

@formgeist correct this request is a duplicate of the APM part of

@weltenwort could you specify what "log line details" are? A specific subset of the source fields? a subset of mapped fields? all of them?

The fields of the log line are the things that matter the most to me, I wouldn't filter them to be sure we don't forget any and to be consistent with other visualizations of details (e.g. document details in Discover, span details in APM...). So I would say "all of them". Do you recommend to explore filtering fields?

I described other characteristics I would like to have on

@weltenwort
Copy link
Member

I didn't realize this is duplicate of #111325. Which one will be the source of truth?

@cyrille-leclerc
Copy link
Contributor Author

I didn't realize this is duplicate of #111325. Which one will be the source of truth?

I would say that this ticket is not specifying the solution, it's more an enhancement request, so I don't think it could contradict #111325 .
When we prioritize adding details to log lines, I guess we will clarify what the flyout will actually look like. Reusing the flyout of Observability / Logs / Stream makes a lot of sense to me. I think we would just have to confirm that we prefer to extend the existing "Observability / Logs / Stream" flyout rather than reusing the new Flyout that Discover introduced.

Does it make sense?

@weltenwort
Copy link
Member

Ok - I'm still slightly confused which of these issues would serve as the implementation issue for the infra monitoring team.

I think we would just have to confirm that we prefer to extend the existing "Observability / Logs / Stream" flyout rather than reusing the new Flyout that Discover introduced.

Using the fly-out from the Logs UI would be pretty easy because it already uses a specialized search strategy for log entries. It would be my recommended approach.

@smith
Copy link
Contributor

smith commented Apr 5, 2022

Using the fly-out from the Logs UI would be pretty easy because it already uses a specialized search strategy for log entries. It would be my recommended approach.

+1 to that. I'm very skeptical that the Discover flyout could be used in another context, as it probably wasn't designed with that in mind.

@smith smith closed this as completed Apr 15, 2022
@smith smith reopened this Apr 15, 2022
@smith
Copy link
Contributor

smith commented Apr 15, 2022

Duplicate of #111325

@smith smith marked this as a duplicate of #111325 Apr 15, 2022
@smith smith closed this as completed Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:logs design Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

5 participants