Issue 854376 has been merged into this issue.

Hey Micah, I noticed you've been working on uma events. Will your CL
  https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1106046
fix this?

Yep, we are aware of this and the linked CL should be the fix :)

I updated the BUG line in the CL description to reference this bug as well.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/52fa073b862975503f015226a60c977cf7b6c183

commit 52fa073b862975503f015226a60c977cf7b6c183
Author: Micah Morton <mortonm@chromium.org>
Date: Thu Jun 21 15:14:27 2018

metrics: Fix syntax error in pre-start script

We took out an unneeded fchmod call in CL:956012, which required
changing the associated pre-start script. Fix a syntax error in the
script that was causing the script to fail/crash.

BUG= chromium:804933 ,  chromium:853701 
TEST=without syntax fix, verified that /var/lib/metrics/uma-events has
wrong ownership/perms and /run/metrics doesn't get created, but file
gets correct ownership/perms and dir gets created after this fix

Change-Id: I62a31f412162638eb648f6505414274a7dfae8bf
Reviewed-on: https://chromium-review.googlesource.com/1106046
Commit-Ready: Mattias Nissler <mnissler@chromium.org>
Tested-by: Micah Morton <mortonm@chromium.org>
Reviewed-by: Luigi Semenzato <semenzato@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/52fa073b862975503f015226a60c977cf7b6c183/metrics/init/metrics_library.conf

This was fixed by the CL uploaded above.

Verified fixed, no uma-events errors in authpolicy.log (attached).

Chrome OS: 10827.0.0
Chrome: 69.0.3475.0
Device: Robo

Deleted: authpolicy.log 4.4 KB

authpolicy.log 4.4 KB View Download