tree 9f6980fa11033debbab9b0c24832c0ff623de020
parent b25425747bbd43e3255763b3e4479f1b0ac5b85f
author Robert Kolchmeyer <rkolchmeyer@google.com> 1594265107 +0000
committer Robert Kolchmeyer <rkolchmeyer@google.com> 1594276224 +0000

Revert "lakitu: Remove metrics daemon"

This reverts commit ca0b56c9f2c7ca424d720b191c6d0a1aad91e7ef.

Reason for revert: This is breaking tests, and I don't have time to fix them before this week's release.

Original change's description:
> lakitu: Remove metrics daemon
> 
> Metrics daemon has not been useful for COS. Recently it introduced a
> regression where the daemon is reading /proc entries every 10mins
> causing noticeable performance degradation for our users.
> So disable the agent. This is done by setting '-passive_metrics' USE
> flag because chromeos-base/metrics package is an implicit dependency of
> crash-reporter and update-engine and we can't simply remove the package.
> 
> BUG=b:156603981
> TEST=Verified metrics_daemon binary is not installed anymore.
> all validation tests / cos_tryjob.
> RELEASE_NOTE=Removed metrics_daemon from the image
> 
> Change-Id: I1fa771d8260aca1ec6186d36ad7d72ff9e2f0f33
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/2277581
> Commit-Queue: Robert Kolchmeyer <rkolchmeyer@google.com>
> Tested-by: Robert Kolchmeyer <rkolchmeyer@google.com>
> Reviewed-by: Roy Yang <royyang@google.com>
> Reviewed-by: Vaibhav Rustagi <vaibhavrustagi@google.com>

Bug: b:156603981
Change-Id: I5082df8f852dd9f61e269bc2a53b051a89ca04db
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/overlays/board-overlays/+/2289537
Reviewed-by: Robert Kolchmeyer <rkolchmeyer@google.com>
Reviewed-by: Vaibhav Rustagi <vaibhavrustagi@google.com>
Tested-by: Robert Kolchmeyer <rkolchmeyer@google.com>