0
Commit Graph

14 Commits

Author SHA1 Message Date
Joel Hockey
f5f032d434 Update instructions for symbolizing crash with local build
Change-Id: I68083d83c4d7dd94a9277214462763cb815711a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5298134
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Auto-Submit: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1261410}
2024-02-16 00:03:39 +00:00
Roman Sorokin
faf5ec7c94 Revise "Symbolizing a crash dump" commands in debugging tips
Seems like these got outdated

Bug: none
Change-Id: Iea2af064a12b75a136b48d5d1ede2b160f8a15bc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3834455
Reviewed-by: Fabian Sommer <fabiansommer@chromium.org>
Commit-Queue: Fabian Sommer <fabiansommer@chromium.org>
Auto-Submit: Roman Sorokin <rsorokin@google.com>
Cr-Commit-Position: refs/heads/main@{#1036522}
2022-08-18 09:23:23 +00:00
Timothy Loh
eb761b06b3 Update documentation re. disabling tast tests
This documentation is pointed at directly from failing test runs.
Disabling tast tests on Chrome's builder alone is often
counterproductive so explicitly point sheriffs at the sheriffing docs
which have more guidance on what action should be taken.

Change-Id: I31bc9375114a062d32182823fa5ecdd8cde07e6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3671926
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1010901}
2022-06-06 01:46:12 +00:00
Ben Pastene
f8d6812077 docs: Update chromeos_debugging_tips.md for the new Milo UI.
Now that the RDB has rolled out more broadly, we can replace references
to the isolate UI viewer with links to the new UI.

Bug: None
Change-Id: I9f48007dbe10a05ae4de92602f442a08bb6b6d44
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3171763
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/main@{#923113}
2021-09-20 22:49:10 +00:00
Ben Pastene
15cae75ba4 docs: Update usage of generate_breakpad_symbols.py to use vpython.
Plain old python fails with:
ImportError: No module named six.moves.queue

Using vpython should fix that.

Also replace "0" in the output dir with the name of a task that it
would really look like.

Bug: None
Change-Id: I51138442a368edc0e4c88f2f8ab24262a6d233e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3152067
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/main@{#919996}
2021-09-09 22:58:22 +00:00
Roman Sorokin
2339741df4 Fix the references to the ui.ChromeLogin
ui.ChromeLogin and ui.ChromeLoginGAIA were renamed in
http://crrev/c/2981353

Bug: b:187788829, 1227801, 1167243, 1115622
Change-Id: I31359f59ac35347aa75809d461a61ea383eb8512
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3018208
Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Auto-Submit: Roman Sorokin [CET] <rsorokin@chromium.org>
Reviewed-by: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#900399}
2021-07-12 10:29:09 +00:00
Takuto Ikuta
2d01a49ebf docs: replace python swarming client usage with go client
Removed run_isolated.py usage as there are no alternatives.

Bug: 984869, 1163817
Change-Id: Ieb72ebb2356d982ac5e90fde0799713daf8aa2c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2936424
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#889080}
2021-06-04 00:28:58 +00:00
Ben Pastene
4327118d09 docs: Clarify steps needed to disable CrOS Tast tests.
Change-Id: I4f9f259e53468b13f2ece5104752bf321c10f530
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2863259
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#878050}
2021-04-30 19:57:07 +00:00
Nicholas Verne
f4b76305d5 Split out tast_control so we can watch changes to tast_disabled_tests.
Bug: 1157356
Change-Id: Ic6762bfa9c1e3f1978ff0a8197f0f80f4800eec2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643678
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Reviewed-by: Shuhei Takahashi <nya@chromium.org>
Commit-Queue: Nicholas Verne <nverne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#847490}
2021-01-27 04:08:05 +00:00
Ben Pastene
115c1a20e3 docs: Fix typo in chromeos_debugging_tips.md
Change-Id: I5aaa8cc34eb8b660a894d08f8a47e4a39569249b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591452
Auto-Submit: Ben Pastene <bpastene@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Cr-Commit-Position: refs/heads/master@{#837168}
2020-12-15 18:39:00 +00:00
Ben Pastene
44862a2d83 docs: Add instructions on how to symbolize Chrome crashes from Tast.
These instructions should let folks manually symbolize any Chrome
crash dumps generated during Tast tests on bots, which will make
test debugging a bit easier.

Bug: 1155738
Change-Id: I5cd9f11d53721b82fd23003d1f89c6e921a0a122
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587583
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#836342}
2020-12-12 00:53:07 +00:00
Ben Pastene
0fed3a7542 docs: Update chromeos_debugging_tips.
The doc includes a number of isolate links, but items in isolate expire
after 90 days. So this updates these to a set of new isolate links.
Hopefully by the time these new links expire, we'll be on RDB which will
have a slightly different debugging flow.

This also adds some tips on running the tests locally.

Bug: None
Change-Id: Ib6244f787d2a914b7c3a79aaba2a33b688d346c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2522409
Reviewed-by: Dirk Pranke <dpranke@google.com>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#824965}
2020-11-06 19:25:15 +00:00
Ben Pastene
48021b47b3 docs: Revise Tast test disabling options.
This should hopefully make it clear to sheriffs that either option is
valid.

Change-Id: Ida41d139dfdb09be3067228d6a05cb3b46806562
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2159605
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#761076}
2020-04-21 19:18:46 +00:00
Ben Pastene
b30c66e969 chromeos: Add a doc for debugging test failures on Chrome OS.
Add a link to that doc in Tast test failure log snippets.

Will make failure snippets look like:
https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8904924613768986736/+/steps/chrome_all_tast_tests__with_patch__on_ChromeOS/0/logs/Deterministic_failure:_session.LogoutCleanup__status_FAILURE_/0

Bug: 923426
Change-Id: Id7d19fa94126a013763e6671cfc0cd8f6263c71b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759234
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#692275}
2019-08-30 23:25:10 +00:00