https://chromium-review.googlesource.com/c/558098/4/scripts%252Fslave%252Frecipe_modules%252Fchromium_tests%252Fsteps.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/a302648c0c1570b579d1408336ea4afe9e2b5cb0 commit a302648c0c1570b579d1408336ea4afe9e2b5cb0 Author: Katie Dillon <kdillon@chromium.org> Date: Mon Jul 10 13:02:12 2017 Added code for reporting browser test resuls to flake-predictor bucket via gsutil module. Bug: 734591, 738924 Change-Id: Ice976f8b590bafee4699e6a408e0f1e78a654a51 Reviewed-on: https://chromium-review.googlesource.com/558098 Commit-Queue: Katie Dillon <kdillon@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> [modify] https://crrev.com/a302648c0c1570b579d1408336ea4afe9e2b5cb0/scripts/slave/recipe_modules/chromium_tests/tests/steps/generate_gtest.py [modify] https://crrev.com/a302648c0c1570b579d1408336ea4afe9e2b5cb0/scripts/slave/recipes/findit/chromium/test.py [modify] https://crrev.com/a302648c0c1570b579d1408336ea4afe9e2b5cb0/scripts/slave/README.recipes.md [add] https://crrev.com/a302648c0c1570b579d1408336ea4afe9e2b5cb0/scripts/slave/recipe_modules/chromium_tests/tests/steps/generate_gtest.expected/gsutil_upload_local.json [modify] https://crrev.com/a302648c0c1570b579d1408336ea4afe9e2b5cb0/scripts/slave/recipe_modules/chromium_tests/steps.py [add] https://crrev.com/a302648c0c1570b579d1408336ea4afe9e2b5cb0/scripts/slave/recipe_modules/chromium_tests/tests/steps/generate_gtest.expected/gsutil_upload_swarming.json
Since we rewrote the CL not to add the hard-coded step name, this bug doesn't need fixing.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/08ec962b5c5690eb18132daad53188bd0b9ca053 commit 08ec962b5c5690eb18132daad53188bd0b9ca053 Author: Katie Dillon <kdillon@chromium.org> Date: Tue Jul 11 12:39:03 2017 Added flake-predictor upload flag to browser_tests Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=738924 Change-Id: I383b1d1b95e1869061d08e537b90232e876b3e5c Reviewed-on: https://chromium-review.googlesource.com/563211 Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Katie Dillon <kdillon@chromium.org> Cr-Commit-Position: refs/heads/master@{#485609} [modify] https://crrev.com/08ec962b5c5690eb18132daad53188bd0b9ca053/testing/buildbot/chromium.linux.json
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/59e330cafa593ebb6af33475b273769c7c7755f4 commit 59e330cafa593ebb6af33475b273769c7c7755f4 Author: Katie Dillon <kdillon@chromium.org> Date: Thu Aug 24 15:22:48 2017 Revert "Added flake-predictor upload flag to browser_tests" This reverts commit 08ec962b5c5690eb18132daad53188bd0b9ca053. Reason for revert: Project is not complete. We will resurrect when another intern can complete the flake-predictor project. Original change's description: > Added flake-predictor upload flag to browser_tests > > Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=738924 > Change-Id: I383b1d1b95e1869061d08e537b90232e876b3e5c > Reviewed-on: https://chromium-review.googlesource.com/563211 > Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Commit-Queue: Katie Dillon <kdillon@chromium.org> > Cr-Commit-Position: refs/heads/master@{#485609} TBR=dpranke@chromium.org,sergiyb@chromium.org,kdillon@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=738924 Change-Id: Ia8dadd3446991da061b24de4bcc605ed4bb86d4d Reviewed-on: https://chromium-review.googlesource.com/632416 Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> Commit-Queue: Katie Dillon <kdillon@chromium.org> Cr-Commit-Position: refs/heads/master@{#497058} [modify] https://crrev.com/59e330cafa593ebb6af33475b273769c7c7755f4/testing/buildbot/chromium.linux.json
Comment 1 Deleted