Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(717)

Unified Diff: tools/perf/docs/perf_bot_sheriffing.md

Issue 2565793002: Add logdog and logcat tips to perf sheriffing doc. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/docs/perf_bot_sheriffing.md
diff --git a/tools/perf/docs/perf_bot_sheriffing.md b/tools/perf/docs/perf_bot_sheriffing.md
index 0e64a485f902b61e27336492e82190276dff8eb7..419dea53fd9b01aeabdec18d5143fee3efb1c1a4 100644
--- a/tools/perf/docs/perf_bot_sheriffing.md
+++ b/tools/perf/docs/perf_bot_sheriffing.md
@@ -190,6 +190,14 @@ be investigated. When a test fails:
ensure that the bug title reflects something like "Fix and re-enable
testname".
4. Investigate the failure. Some tips for investigating:
+ * When viewing buildbot step logs, **use the **<font color="blue">[stdout]</font>** link to view logs!**.
+ This will link to logdog logs which do not expire. Do not use or link
+ to the logs found through the <font color="blue">stdio</font> link
+ whenever possible as these logs will expire.
+ * When investigating Android, look for the logcat which is uploaded to
+ Google Storage at the end of the run. logcat will contain much more
+ detailed Android device and crash info than will be found in
+ Telemetry logs.
* If it's a non flaky failure, indentify the first failed
build so you can narrow down the range of CLs that causes the failure.
You can use the
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698