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

Issue 735106 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Move authpolicy logs to a separate file

Project Member Reported by ljusten@chromium.org, Jun 20 2017

Issue description

- Move logs out of var/log/messages into separate file
- Make sure logs get uploaded with feedback
- See platform2/init/rsyslog.chromeos for an example how shill does it.

 
Labels: -M-62 M-61
Status: Started (was: Assigned)
Labels: -M-61 M-62
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 20 2017

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

commit c226566fc7b582ceb15b938409dae7e46919da93
Author: Lutz Justen <ljusten@chromium.org>
Date: Thu Jul 20 22:00:38 2017

Move authpolicy logs to a separate file

Logs normal logs to /var/log/authpolicy.log instead of
/var/log/messages since logs can be long, especially with debug
features enabled. ERR and higher levels are still logged to
/var/log/messages for visibility and context. Rotates logs and
uploads the last 50k of the current log file.

BUG= chromium:735106 
TEST=Tested on device that logs end up in correct files
     Triggered a crash report and verified that log tail is appended
     cros_run_unit_tests --board=enguarde --packages "crash-reporter chromeos-init"

Change-Id: I4e1fe6396a40a938c2ce2f220c3ecae629ecd4a4
Reviewed-on: https://chromium-review.googlesource.com/549318
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Will Drewry <wad@chromium.org>

[modify] https://crrev.com/c226566fc7b582ceb15b938409dae7e46919da93/init/chromeos-cleanup-logs
[modify] https://crrev.com/c226566fc7b582ceb15b938409dae7e46919da93/init/rsyslog.chromeos
[modify] https://crrev.com/c226566fc7b582ceb15b938409dae7e46919da93/crash-reporter/crash_reporter_logs.conf
[modify] https://crrev.com/c226566fc7b582ceb15b938409dae7e46919da93/init/upstart/syslog.conf
[modify] https://crrev.com/c226566fc7b582ceb15b938409dae7e46919da93/debugd/src/log_tool.cc

Status: Fixed (was: Started)
Labels: -M-62 M-61
Made it into M-61 (9764.0.0).
Cc: jingwee@chromium.org krishna...@chromium.org
Status: Verified (was: Fixed)
As verified in M61.0.3163.72:9765.49.0 beta peppy/reks, authpolicy logs are sent to /var/log/autopolicy.log while error logs remain in /var/log/messages.

Sign in to add a comment