New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 842675 link

Starred by 1 user
Status: Fixed
Owner:
marinaciocea@chromium.org
Closed: May 2018
Cc:
jonasolsson@chromium.org
█ olka@chromium.org
maxmorin@chromium.org
solenberg@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature



Sign in to add a comment

Add Audio Service lifetime metrics

Project Member Reported by marinaciocea@chromium.org, May 14 2018 
Add UMA stats:
- Duration of “service has incoming connections” intervals: Time between first incoming connection and last disconnected connection.
- Duration of “service has no incoming connections” intervals (including Service teardown timer cancelled because a new client request is received): Time between last disconnect and when quit_closure_ is called OR quit_closure_ is canceled.
- Service uptime (time between startup and shutdown)
Project Member

Comment 1 by bugdroid1@chromium.org, May 17 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/18633ae2b68f34d37ca713c4bd0a01f1432eaa71

commit 18633ae2b68f34d37ca713c4bd0a01f1432eaa71
Author: Marina Ciocea <marinaciocea@chromium.org>
Date: Thu May 17 08:57:33 2018

Add audio service lifetime UMA stats.

Add UMA stats:
- Duration of “service has incoming connections” intervals: Time between first incoming connection and last disconnected connection.
- Duration of “service has no incoming connections” intervals: Time between last disconnect and when quit_closure_ is called OR quit_closure_ is canceled.
- Service uptime (time between startup and shutdown)

Design doc: https://www.google.com/url?q=https://docs.google.com/document/d/1s_Fd1WRDdpb5n6C2MSJjeC3fis6hULZwfKMeDd4K5tI/edit%23heading%3Dh.2ap5hw4bex5a&sa=D&source=hangouts&ust=1526373977505000&usg=AFQjCNHfaxd8BHSQOi_HmB1cVe-bjilw5w

Bug:  842675 
Change-Id: I429e34560ddcd52c814473bff6d6c1cdcc13e418
Reviewed-on: https://chromium-review.googlesource.com/1057628
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#559461}
[modify] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/BUILD.gn
[modify] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/debug_recording.cc
[modify] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/debug_recording.h
[modify] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/service.cc
[modify] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/service.h
[add] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/service_metrics.cc
[add] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/service_metrics.h
[add] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/services/audio/service_metrics_unittest.cc
[modify] https://crrev.com/18633ae2b68f34d37ca713c4bd0a01f1432eaa71/tools/metrics/histograms/histograms.xml

Comment 2 by marinaciocea@chromium.org, May 17 2018

Status: Fixed (was: Started)

Comment 3 by marinaciocea@chromium.org, May 17 2018

Labels: -Type-Bug Type-Feature

Comment 4 by anatolid@chromium.org, Jun 7 2018 

[bulk-edit: disregard if N/A] Can the owner please set milestone to this bug if applicable?
►

Sign in to add a comment