Make sure cryptohome diagnostic logs are available in chrome://system |
|||||||||
Issue descriptionwe're logging errno etc when something goes wrong, let's make sure it's reported and actionable if something happens.
,
Apr 14 2017
syslog survives a reboot.
,
Apr 14 2017
Ok, good to know. We should be fine in either case then.
,
Apr 14 2017
So the remaining items are to see errno is stored correctly and the logs would be useful and actionable.
,
Apr 14 2017
Yes, although I'm not sure how to test this since I don't have a way of causing the migration to fail anymore (outside of just putting in a random return false, but that doesn't help test the logging).
,
Apr 14 2017
From other threads we should be able to craft a broken SQLite database, broken symlink we can't delete, and a long xattr that gets corrupted. But also you can inject failures and see how logs look like.
,
Apr 14 2017
Ok, created an invalid file and verified that it shows up in syslog with errno string.
,
Apr 18 2017
There was a CL to add more diagnostic information submitted. https://chromium-review.googlesource.com/c/477690/ done?
,
Apr 19 2017
,
Apr 23 2017
,
Apr 24 2017
,
Apr 24 2017
Please tag with applicable OSs. Thanks.
,
Apr 24 2017
,
Apr 25 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 28 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 30 2017
,
Jan 22 2018
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by dspaid@chromium.org
, Apr 14 2017