ARC: add UMA metric for ARC shutdown time |
|||||||||||
Issue descriptionThis will be helpful in guiding how long to delay forcing shutdown when ARC is unresponsive.
,
Apr 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e794bb52914b5dd9b440776c56bbe09f4f3a9930 commit e794bb52914b5dd9b440776c56bbe09f4f3a9930 Author: elijahtaylor <elijahtaylor@chromium.org> Date: Sat Apr 23 00:09:34 2016 ARC: add histogram definition for ShutdownTime BUG= 605811 Review URL: https://codereview.chromium.org/1908313002 Cr-Commit-Position: refs/heads/master@{#389319} [modify] https://crrev.com/e794bb52914b5dd9b440776c56bbe09f4f3a9930/tools/metrics/histograms/histograms.xml
,
Jun 8 2016
,
Jul 1 2016
,
Aug 29 2016
,
Oct 7 2016
,
Oct 18 2016
The histogram was added but it doesn't look like it has any data: https://uma.googleplex.com/p/chrome/histograms/?endDate=latest&dayCount=1&histograms=Arc.ShutdownTime&fixupData=false&uniqueUsers=true&showMax=true&filters=isofficial%2Ceq%2CTrue&implicitFilters=isofficial
,
Oct 18 2016
We had data back when this was added in an init script, which was just advisory for picking a good timeout for killing the container. My guess is that this was not ported when this shutdown code was moved to session manager. Dylan, can you confirm? I think we do not strictly need this anymore, but am open to adding it back if we think it's useful
,
Nov 19 2016
,
Jan 21 2017
,
Mar 4 2017
,
Apr 5 2017
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by bugdroid1@chromium.org
, Apr 23 2016