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

Issue 711545 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 688900



Sign in to add a comment

Make sure cryptohome diagnostic logs are available in chrome://system

Project Member Reported by uekawa@google.com, Apr 14 2017

Issue description

we're logging errno etc when something goes wrong, let's make sure it's reported and actionable if something happens.
 

Comment 1 by dspaid@chromium.org, Apr 14 2017

I can confirm that logs seem to be present in chrome://system under syslog.  Note that we don't have a way of collecting these logs for successful migrations (the migration flow ends in a reboot, which will wipe the logs).  Assuming we don't reboot after reporting an error and requiring the user recreate their profile the logs should still be there but I can't verify this until we have end-to-end testing capability.

Comment 2 by uekawa@chromium.org, Apr 14 2017

syslog survives a reboot.

Comment 3 by dspaid@chromium.org, Apr 14 2017

Ok, good to know.  We should be fine in either case then.

Comment 4 by uekawa@google.com, Apr 14 2017

So the remaining items are to see errno is stored correctly and the logs would be useful and actionable.


Comment 5 by dspaid@chromium.org, 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).

Comment 6 by uekawa@google.com, 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.

Comment 7 by dspaid@chromium.org, Apr 14 2017

Ok, created an invalid file and verified that it shows up in syslog with errno string.

Comment 8 by uekawa@google.com, Apr 18 2017

Owner: dspaid@chromium.org
Status: Started (was: Available)
There was a CL to add more diagnostic information submitted.

https://chromium-review.googlesource.com/c/477690/


done?

Comment 9 by dspaid@chromium.org, Apr 19 2017

Status: Fixed (was: Started)
Labels: Merge-Request-59

Comment 11 by uekawa@google.com, Apr 24 2017

Labels: ArcExt4Migration
Please tag with applicable OSs.  Thanks.
Labels: OS-Chrome
Project Member

Comment 14 by sheriffbot@chromium.org, Apr 25 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
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
Project Member

Comment 15 by sheriffbot@chromium.org, 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
Labels: -Hotlist-Merge-Approved -Merge-Approved-59

Comment 17 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment