Cryptohome should output performance related info to log |
|||||||||
Issue descriptionIt'd be useful if cryptohome outputs info like the total amount of data to the log to include them in feedback reports.
,
May 2 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/41da3335fe9bbd17cc9c201b6c802d08f479a3ef commit 41da3335fe9bbd17cc9c201b6c802d08f479a3ef Author: Ryo Hashimoto <hashimoto@google.com> Date: Tue May 02 12:20:28 2017 cryptohome: Leave migration performance log The log will look like: INFO cryptohomed[1541]: Migrating to dircrypto. INFO cryptohomed[1541]: Number of files: 14083 INFO cryptohomed[1541]: Number of directories: 3845 INFO cryptohomed[1541]: Number of symlinks: 15 INFO cryptohomed[1541]: Preparation took 1524 ms. INFO cryptohomed[1541]: Migrated 1454637243 bytes in 226361 ms at 6426 KB/s. INFO cryptohomed[1541]: Migration done. BUG= chromium:716324 TEST=manually Change-Id: I0eeadc2af4677cae8974b73e3c73e42a4a5eaec2 Reviewed-on: https://chromium-review.googlesource.com/484163 Commit-Ready: Ryo Hashimoto <hashimoto@chromium.org> Tested-by: Ryo Hashimoto <hashimoto@chromium.org> Reviewed-by: Dan Spaid <dspaid@chromium.org> [modify] https://crrev.com/41da3335fe9bbd17cc9c201b6c802d08f479a3ef/cryptohome/dircrypto_data_migrator/migration_helper.cc
,
May 8 2017
,
May 12 2017
,
May 12 2017
,
May 13 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
,
May 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/ef7d8acf62dae076ce2ce65bf16268e631b26385 commit ef7d8acf62dae076ce2ce65bf16268e631b26385 Author: Ryo Hashimoto <hashimoto@google.com> Date: Mon May 15 03:27:54 2017 cryptohome: Leave migration performance log The log will look like: INFO cryptohomed[1541]: Migrating to dircrypto. INFO cryptohomed[1541]: Number of files: 14083 INFO cryptohomed[1541]: Number of directories: 3845 INFO cryptohomed[1541]: Number of symlinks: 15 INFO cryptohomed[1541]: Preparation took 1524 ms. INFO cryptohomed[1541]: Migrated 1454637243 bytes in 226361 ms at 6426 KB/s. INFO cryptohomed[1541]: Migration done. BUG= chromium:716324 TEST=manually Change-Id: I0eeadc2af4677cae8974b73e3c73e42a4a5eaec2 Reviewed-on: https://chromium-review.googlesource.com/484163 Commit-Ready: Ryo Hashimoto <hashimoto@chromium.org> Tested-by: Ryo Hashimoto <hashimoto@chromium.org> Reviewed-by: Dan Spaid <dspaid@chromium.org> (cherry picked from commit 41da3335fe9bbd17cc9c201b6c802d08f479a3ef) Reviewed-on: https://chromium-review.googlesource.com/505972 Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org> Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org> [modify] https://crrev.com/ef7d8acf62dae076ce2ce65bf16268e631b26385/cryptohome/dircrypto_data_migrator/migration_helper.cc
,
May 15 2017
,
Aug 1 2017
,
Jan 22 2018
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by hashimoto@chromium.org
, Apr 28 2017