New issue
Advanced search Search tips

Issue 647242 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Archiving of AppRTC APK in client.webrtc waterfall

Project Member Reported by kjellander@chromium.org, Sep 15 2016

Issue description

To easy manual testing, we'd like to archive the AppRTCDemo artifacts from the Android builders to Google storage. It should be added as a separate step, similar to
https://cs.chromium.org/chromium/build/scripts/slave/recipes/webrtc/ios_api_framework.py?rcl=0&l=54
or https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/webrtc/api.py?rcl=0&l=271

The file to be archived is located in out/{Debug,Release}/apks/AppRTCDemo.apk


 
Preferably for these four builders:
Android32 Builder	
Android32 Builder (dbg)	
Android64 Builder	
Android64 Builder (dbg)

Project Member

Comment 2 by bugdroid1@chromium.org, Sep 19 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/a2b056bd69e365ff4c2a9bfa961ad8e13565470f

commit a2b056bd69e365ff4c2a9bfa961ad8e13565470f
Author: ehmaldonado <ehmaldonado@chromium.org>
Date: Mon Sep 19 12:45:21 2016

Archiving of AppRTC APK in client.webrtc waterfall

To ease manual testing, the AppRTCDemo artifacts from the Android builders are being archived to Google storage for the following builders:
  Android32 Builder
  Android32 Builder (dbg)
  Android64 Builder
  Android64 Builder (dbg)

BUG= 647242 

Review-Url: https://codereview.chromium.org/2344173002

[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipe_modules/webrtc/__init__.py
[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipe_modules/webrtc/api.py
[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipe_modules/webrtc/builders.py
[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipes/webrtc/standalone.expected/client_webrtc_android32_builder.json
[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipes/webrtc/standalone.expected/client_webrtc_android32_builder__dbg_.json
[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipes/webrtc/standalone.expected/client_webrtc_android32_builder_forced.json
[modify] https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f/scripts/slave/recipes/webrtc/standalone.expected/client_webrtc_android64_builder.json

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 19 2016

Project Member

Comment 4 by bugdroid1@chromium.org, Sep 19 2016

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

commit 2c3e964420124cb86cf760067073649180098a88
Author: recipe-roller <recipe-roller@chromium.org>
Date: Mon Sep 19 13:00:47 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f Archiving of AppRTC APK in client.webrtc waterfall (ehmaldonado@chromium.org)
  https://crrev.com/2dd50f3f904647f20ed62fdb3561543d5f1f0a73 WebRTC: Add android swarming bots. (ehmaldonado@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 583318 , 647242 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2354533002
Cr-Commit-Position: refs/heads/master@{#419441}

[modify] https://crrev.com/2c3e964420124cb86cf760067073649180098a88/infra/config/recipes.cfg

Project Member

Comment 5 by bugdroid1@chromium.org, Sep 19 2016

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

commit ba62b5c6aceb2614f75592b6ec622964f31bbd2d
Author: recipe-roller <recipe-roller@chromium.org>
Date: Mon Sep 19 13:30:37 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/a2b056bd69e365ff4c2a9bfa961ad8e13565470f Archiving of AppRTC APK in client.webrtc waterfall (ehmaldonado@chromium.org)
  https://crrev.com/2dd50f3f904647f20ed62fdb3561543d5f1f0a73 WebRTC: Add android swarming bots. (ehmaldonado@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 583318 , 647242 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2350043002

[modify] https://crrev.com/ba62b5c6aceb2614f75592b6ec622964f31bbd2d/infra/config/recipes.cfg

Status: Fixed (was: Assigned)
Thanks!
Now testers just need to go into a build like https://build.chromium.org/p/client.webrtc/builders/Android32%20Builder/builds/8706 and click the gsutil.upload link at step 15. I'm wondering if we can make the download link more visible, e.g. by changing the text being displayed or something.

Sign in to add a comment