openstack/deb-horizon
Code Issues Proposed changes

Fix reporting of profiler traces with multiple toplevels

Browse Source 
The Profiler panel was only showing the first of potentally
many toplevel events in a request. This patch reinstates
those events by displaying the root event rather than
its first child.

Change-Id: Ifeb0c3ee16168335e7b41c1063897ccdf5511800
Closes-Bug: 1660449
(cherry picked from commit d256d0025f)
This commit is contained in:
Richard Jones 2017-01-31 11:50:32 +11:00 committed by Rob Cresswell
parent 100f77d895
commit f5aefa03a4
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openstack_dashboard/contrib/developer/profiler/api.py
View File

@ -105,9 +105,7 @@ def get_trace(request, trace_id):
engine = _get_engine(request)
trace = engine.get_report(trace_id)
# NOTE(tsufiev): throw away toplevel node which is dummy and doesn't
# contain any info, use its first and only child as the toplevel node
data, max_finished = rec(trace['children'][0])
data, max_finished = rec(trace)
data['info']['max_finished'] = max_finished
return data