644309 - Improve logging of executed commands for better failure discovery

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Crowd Extension, defect)

Description

[Aaron Train [:aaronmt]]

On Linux (Ubuntu 10.10), I am unable to start a test run. The phase at which I see this error is after cloning mozmill-automation:

requesting all changes
adding changesets
adding manifests
adding file changes
added 1179 changesets with 4172 changes to 479 files (+3 heads)
updating to branch default
198 files updated, 0 files merged, 0 files removed, 0 files unresolved
No option 'SourceRepository' in section: 'App'
*** Removing repository '/tmp/tmpeqetL6.mozmill-tests'
 
In the extension's console, I see:

The log file "/home/aaronmt/Desktop/testrun.log" could not be read.

setup.sh has no issue running on its own

Comment 1

[Aaron Train [:aaronmt]]

For some reason it cant load the logfile because the file is not being created

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Which build of Firefox are you using?

Comment 3

[Aaron Train [:aaronmt]]

On my ubuntu VM I was using release 4.0

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Patch v1

This patch adds a logging message to the Error Console, which shows up the exact command which gets executed.

Comment 5

[Aaron Train [:aaronmt]]

Comment on attachment 521325 [details] [diff] [review]
Patch v1

Works for me, example output of the cmd

MozMill Crowd: Executing: '/home/aaronmt/Desktop/mozmill-env/run.sh python /home/aaronmt/Desktop/mozmill-automation/testrun_endurance.py --repository=https://hg.mozilla.org/qa/mozmill-tests/ --logfile=/home/aaronmt/Desktop/testrun.log --screenshot-path=/home/aaronmt/Desktop/screenshots --iterations=1 --delay=0.1 /usr/lib/firefox-4.0/firefox-bin'

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1]]

The problem here is that Ubuntu modifies the application.ini file and removes the necessary references to the SourceRepository which we need to identify the branch we are working on correctly. We can't do anything against that and would have to find a way to gracefully workaround that issue if possible.

For now lets take this patch and make it easier to investigate failures like those.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Landed as:
https://github.com/whimboo/mozmill-crowd/commit/10a8b2a478b647b2abad0c9c9661bf726056d9a0