619780 - talos needs to support environment variables on remote testing

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: patch to bring device manager up to speed with mozilla central (1.0)

currently we need to disable crashreporter and this requires us adding environment variables into the command that we send to android. 

instead of:
'exec org.mozilla.fennec -profile /tmp/profile http://my/test.html'

it needs to be:
'exec "MOZ_CRASHREPORTER_NO_REPORT=1" org.mozilla.fennec -profile /tmp/profile http://my/test.html'

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: support android style environment variables from the .config file (1.0)

secondary patch which fixes the talos code to support the environment variables on the command line which android needs.

These will be running on talos staging

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: patch to bring device manager up to speed with mozilla central (1.5)

updated patch to fix a few devicemanager specific things.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: support android style environment variables from the .config file (1.5)

moved stuff out of here into the devicemanager patch to keep it unique.  Also added in a new ENV variable that mobile supports as well as a cleanup on the run_tests.py application.ini discovery.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

updated both patches.  The original patches were green on try!  I am running these new patches on try and we will see the results.  I will have to update the results in this bug this weekend as I will be AFK the majority of today.

Comment 5

[alice nodelman [:alice] [:anode]]

Please update when you get a green run through so that I can complete my review.

Comment 6

[alice nodelman [:alice] [:anode]]

Just a general comment that we may need to clean up in future the general case setting of env vars from the specific remote setting - instead of just having one thing called 'env' might be smarter to split into local_env and remote_env and then handle those things separately.  Could just become confusing when pretty much the same config file has different results depending on if you are running remote or not.

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

as a note, these passed talos staging all green.

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

landed:
http://hg.mozilla.org/build/talos/pushloghtml?changeset=3efbac1f685a
http://hg.mozilla.org/build/talos/pushloghtml?changeset=aa376673fab5

I filed bug 620104 to track the remote vs local environment variable issue.