New issue
Advanced search Search tips

Issue 728404 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocking:
issue 725695



Sign in to add a comment

Add ability to remove system packages on swarming

Project Member Reported by martiniss@chromium.org, May 31 2017

Issue description

This is desired by both the VR and Speed teams.

Design doc forthcoming.
 
Cc: bsheedy@chromium.org
+bsheedy fyi
Blocking: 725695
Cc: perezju@chromium.org
Owner: jbudorick@chromium.org
Status: Started (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d7aa9b726f574cff3c44c068e49ec71ee6081449

commit d7aa9b726f574cff3c44c068e49ec71ee6081449
Author: catapult-deps-roller@chromium.org <catapult-deps-roller@chromium.org>
Date: Wed Jun 14 20:17:10 2017

Roll src/third_party/catapult/ 597f96ee4..c9212cbf5 (8 commits)

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/597f96ee44a5..c9212cbf5220

$ git log 597f96ee4..c9212cbf5 --date=short --no-merges --format='%ad %ae %s'
2017-06-14 ashleymarie Revert of Defaulting verify_names flag to True (patchset #1 id:1 of https://codereview.chromium.org/2924023003/ )
2017-06-13 lchoi Removed if clause from screenshot_supported. After running unit tests locally, was able to verify that screenshotting does indeed work with xvfb. Patching so that we can move forward with screenshotting for Cluster Telemetry purposes:  crbug.com/536285 
2017-06-13 benjhayden Plumb trace start timestamp DateRange diagnostic from telemetry.
2017-06-13 benjhayden Copy improvement directions from telemetry to legacy_unit_info.
2017-06-13 jbudorick [devil] Add system app replacement wrapper script.
2017-06-13 benjhayden Add a slider to histogram-set-controls to control alpha.
2017-06-13 ashleymarie Defaulting verify_names flag to True
2017-06-13 benjhayden Fix overview line charts in results2.

Created with:
  roll-dep src/third_party/catapult
BUG= 720002 , 728404 , 720002 


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I89c70230bd8dc1b4304961e891534abe56c9d79a
Reviewed-on: https://chromium-review.googlesource.com/535062
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#479484}
[modify] https://crrev.com/d7aa9b726f574cff3c44c068e49ec71ee6081449/DEPS

Project Member

Comment 6 by bugdroid1@chromium.org, Jun 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/88003bd291b005c5f1cf5cc29b1fae2d8bec68a4

commit 88003bd291b005c5f1cf5cc29b1fae2d8bec68a4
Author: catapult-deps-roller@chromium.org <catapult-deps-roller@chromium.org>
Date: Wed Jun 14 23:03:13 2017

Roll src/third_party/catapult/ c9212cbf5..59a182b2a (2 commits)

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/c9212cbf5220..59a182b2a694

$ git log c9212cbf5..59a182b2a --date=short --no-merges --format='%ad %ae %s'
2017-06-14 jbudorick [devil] Do nothing when removing a system app that isn't present.
2017-06-14 benjhayden Make tracing/bin/validate_all_diagnostics executable.

Created with:
  roll-dep src/third_party/catapult
BUG= 728404 


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I5f171c867017578981499c67579426fd7be22b8e
Reviewed-on: https://chromium-review.googlesource.com/535878
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#479536}
[modify] https://crrev.com/88003bd291b005c5f1cf5cc29b1fae2d8bec68a4/DEPS

Status: Fixed (was: Started)
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 26 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3fa4315be4d2fb7769629ac3af9ced0e5a39537e

commit 3fa4315be4d2fb7769629ac3af9ced0e5a39537e
Author: John Budorick <jbudorick@chromium.org>
Date: Mon Jun 26 18:32:36 2017

[android] Add an empty apk for devil testing.

To be used in testing system APK removal and replacement.

Bug:  728404 
Change-Id: I7e1dc884098f6639eef93a4534e3254fc5f7516a
Reviewed-on: https://chromium-review.googlesource.com/530102
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Michael Case <mikecase@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#482331}
[modify] https://crrev.com/3fa4315be4d2fb7769629ac3af9ced0e5a39537e/BUILD.gn
[add] https://crrev.com/3fa4315be4d2fb7769629ac3af9ced0e5a39537e/build/secondary/third_party/catapult/devil/BUILD.gn
[add] https://crrev.com/3fa4315be4d2fb7769629ac3af9ced0e5a39537e/testing/android/empty_apk/AndroidManifest.xml
[add] https://crrev.com/3fa4315be4d2fb7769629ac3af9ced0e5a39537e/testing/android/empty_apk/empty_apk.gni

Sign in to add a comment