diff options
| author | 2024-05-28 14:18:27 +0100 | |
|---|---|---|
| committer | 2024-05-30 14:21:59 +0000 | |
| commit | 9b760c5aa6663d362e7df9de69a38be40ba3f539 (patch) | |
| tree | a0181a9767f8fbd2c2afb59a8cc35852d3011e7d /runtime/exec_utils.cc | |
| parent | 70e2248b514949098f170346b6368db8930ac86b (diff) | |
Fixed missing end time on previous runs.
Before this change, there was a bug with metrics reporting. When a job
was rerun, the end time of previous runs were unexpectedly cleared.
The root cause is that, when a job was cancelled and rerun,
`recordJobEnded` was called by a reporter inside chroot, and then
`recordJobStarted` was called by a reporter outside chroot. The latter
didn't have the job end time.
After this change, both `recordJobEnded` and `recordJobStarted` are
called by a reporter outside chroot. This change also brings a bonus:
the job duration now includes the time spend on teardown.
Bug: 336239721
Bug: 311377497
Test: Run Pre-reboot Dexopt twice.
Change-Id: Iaf8f574087e2ebbc99aef1004d862ae2777e80c7
Diffstat (limited to 'runtime/exec_utils.cc')
0 files changed, 0 insertions, 0 deletions