Closed Bug 786055 Opened 10 years ago Closed 10 years ago

bcontroller.py should use subprocess.Popen and PIPE to write to browser log

Categories

(Testing :: Talos, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: k0scist, Unassigned)

Details

(Whiteboard: [good first bug][mentor=jhammel][lang=py])

Attachments

(2 files, 3 obsolete files)

Currently bcontroller.py uses os.system and > to redirect browser
output to the log.  Instead, it should use subprocess.Popen and PIPE
to actually get the data in the browser log.
Whiteboard: [good first bug][mentor=jhammel][lang=py]
Under the talos folder, I didn't find the bcontroller.py.
I don't seem to have the file in my mozilla-central/testing/talos folder. So, I downloaded the raw file from the server and made the changes. Please check it. Its my first patch so please correct me if I have made any mistakes.
Comment on attachment 661485 [details] [diff] [review]
Changed os.system to subproces.Popen

>115c115,120
><       self.returncode = os.system(self.command + " > " + self.browser_log)
>---
>>       curProcess = subprocess.Popen(self.command, stdout=subprocess.PIPE)
>>       output, error = curProcess.communicate()
>>       self.returncode = curProcess.returncode
we never do anything with error, if we are going to store this variable lets use it

>>       output_file = open(self.browser_log, "w")
>>       output_file.write(output)
>>       output_file.close()
please wrap this in a try/except block.  On windows we have a lot of concurrency issues with browser_log.  It could be that we have it locked by os.system and this resolves our problems.  

Other comments are to make this a valid patch.  As this is your first patch, I am glad to see you figure out where the source is and how to make a valid diff for it!  The talos code lives in http://hg.mozilla.org/build/talos, and you can submit a patch that way.

When you do submit a patch, set the review flag while uploading the attachment.  Ask for :jhammel or :jmaher and we can review.  Once we give it a r+, we will run it through try server to ensure there are no corner cases we missed, then we can land it and schedule it for deployment.
Apart from changing it to subprocess.Popen, i've wrapped the file handling part in a try...catch block.
Attachment #662171 - Flags: review?
Attachment #662171 - Flags: review? → review?(jmaher)
Attachment #662171 - Flags: review?(jhammel)
Comment on attachment 662171 [details] [diff] [review]
Added try...catch block to file opeining

Review of attachment 662171 [details] [diff] [review]:
-----------------------------------------------------------------

I would be happy with just changing the one line here.

::: talos/bcontroller.py
@@ +119,5 @@
> +        output_file = open(self.browser_log, "w")
> +        output_file.write(output)
> +        output_file.close()
> +      except IOError, e:
> +        print e

please add some context here, maybe:
raise talosError("Unable to write output from test to log file due to: %s" % e)
Attachment #662171 - Flags: review?(jmaher) → review+
Attached patch Raised talosError exception. (obsolete) — Splinter Review
raised the talosError exception with the error info.
Attachment #661485 - Attachment is obsolete: true
Attachment #662171 - Attachment is obsolete: true
Attachment #662171 - Flags: review?(jhammel)
Attachment #662181 - Flags: review?(jmaher)
Comment on attachment 662181 [details] [diff] [review]
Raised talosError exception.

Review of attachment 662181 [details] [diff] [review]:
-----------------------------------------------------------------

thanks.
Attachment #662181 - Flags: review?(jmaher) → review+
what needs to be done after this?
i've pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=f726beef2044

If this goes well, then it can be checked in to http://hg.mozilla.org/build/talos
This fails on linux, https://tbpl.mozilla.org/php/getParsedLog.php?id=15311018&tree=Try&full=1#error0:

Exception in thread Thread-1:
Traceback (most recent call last):
  File "/home/cltbld/lib/python2.5/threading.py", line 486, in __bootstrap_inner
    self.run()
  File "/home/cltbld/talos-slave/talos-data/talos/bcontroller.py", line 117, in run
    curProcess = subprocess.Popen(self.command, stdout=subprocess.PIPE)
  File "/home/cltbld/lib/python2.5/subprocess.py", line 594, in __init__
    errread, errwrite)
  File "/home/cltbld/lib/python2.5/subprocess.py", line 1097, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory

This is because we are passing command as a string.  Instead, we should be passing a list of args
Try run for f726beef2044 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=f726beef2044
Results (out of 46 total builds):
    exception: 6
    success: 20
    failure: 20
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/jhammel@mozilla.com-f726beef2044
My bad! Changed the argument to a list. Hope this fixes it. Is there some way for me to test this out before posting the patch? Then I would be able to avoid such minor mistakes.
Attachment #662181 - Attachment is obsolete: true
Attachment #663336 - Flags: review?(jmaher)
Attachment #663336 - Flags: review?(jhammel)
> Is there some way for me to test this out before posting the patch? Then I would be able to avoid such minor mistakes.

You can certainly run some subset of the talos tests as a smoke-screen.  I often use

talos -n -d -a ts:tsvg --cycles 1 --develop
Comment on attachment 663336 [details] [diff] [review]
Passing argument as list instead of string.

+      curProcess = subprocess.Popen(self.command.split(' '), stdout=subprocess.PIPE)

This isn't very robust.  shlex.split() is the proper way to do this (on posix anyway).  However, my intention was to replace the entireity of self.command with a list, not just on calling.  :jmaher, do you have any thoughts here?  Could also just use the string with shell=True in the Popen instantiation.
Attachment #663336 - Flags: review?(jhammel) → review-
Comment on attachment 663336 [details] [diff] [review]
Passing argument as list instead of string.

Review of attachment 663336 [details] [diff] [review]:
-----------------------------------------------------------------

::: talos/bcontroller.py
@@ +113,5 @@
>          print "Error running etlparser: %s" % e
>          self.returncode = 1
>  
>      else:    #blocking call to system, non-remote device
> +      curProcess = subprocess.Popen(self.command.split(' '), stdout=subprocess.PIPE)

do we even need to split here?  Just above this in the code for the xperf block we do a subprocess.Popen(self.command).  I agree the split on ' ', isn't the most robust method.

Personally I think this gets us closer to using subprocess everywhere.  On a related but tangential note, for xperf and remote we write the log files by setting talos.logfile as a preference.  This ends up having pageloader write to a log file instead of stdout.  This is most problematic on windows, but works well on other operating systems.  That might be a good solution for this as well.
Attachment #663336 - Flags: review?(jmaher) → review+
this could be done a bit more elegantly in python 2.5+
Attachment #685029 - Flags: review?(jmaher)
After looking at this a bit, while it's probably a good idea to keep the firefox command line as a list vs string, this is probably more work than it is worth at this point.  It would be better to devote that work to e.g. switching to mozprocess. I propose we do one of two things:

1. use shell=True for subprocess.Popen as done in attachment 685029 [details]

2. WONTFIX this bug and keep the one call to os.system in bcontroller
Comment on attachment 685029 [details]
use subprocess.Popen(..., shell=True)

This patch helps us out in the short term.  Mozprocess might be a few months out, so lets make progress when we can.
Attachment #685029 - Flags: review?(jmaher) → review+
What sort of testing should be done for this patch, :jmaher?
I would say tp5 and ts on windows.  Sorry for not mentioning anything in a review.
Try run for 20ceddd60cbb is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=20ceddd60cbb
Results (out of 5 total builds):
    success: 3
    failure: 2
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/jhammel@mozilla.com-20ceddd60cbb
boo:

NOISE: __start_tp_report
NOISE: _x_x_mozilla_page_load
NOISE: _x_x_mozilla_page_load_details
NOISE: |i|pagename|runs|
NOISE: |0;thesartorialist.blogspot.com/thesartorialist.blogspot.com/index.html;2249;100;109;106;106;129;106;106;107;93;107;106;106;106;107;106;103;105;106;92;104;109;106;106;104
NOISE: |1;cakewrecks.blogspot.com/cakewrecks.blogspot.com/index.html;578;215;216;207;206;210;208;212;209;212;212;208;211;220;208;221;209;215;207;215;202;207;206;209;211
NOISE: |2;baidu.com/www.baidu.com/s@wd=mozilla.html;1250;88;59;74;58;79;63;78;64;64;63;64;63;63;64;64;63;62;63;79;63;63;67;78;63
NOISE: |3;en.wikipedia.org/en.wikipedia.org/wiki/Rorschach_test.html;874;441;444;445;453;441;446;445;443;443;454;455;449;446;454;450;446;449;480;451;446;450;454;449;452
NOISE: |4;twitter.com/twitter.com/ICHCheezburger.html;461;180;223;223;224;227;216;224;223;221;223;225;226;220;224;215;222;216;218;230;227;225;224;222;224
NOISE: |5;msn.com/www.msn.com/index.html;333;220;218;220;221;228;225;219;223;220;223;223;221;220;221;220;217;224;222;222;220;223;218;220;217
NOISE: |6;yahoo.co.jp/www.yahoo.co.jp/index.html;224;99;81;88;81;86;82;83;84;85;84;92;69;84;83;84;83;84;82;98;69;84;67;89;79
NOISE: |7;amazon.com/www.amazon.com/Kindle-Wireless-Reader-Wifi-Graphite/dp/B002Y27P3M/507846.html;1436;843;873;844;846;853;846;852;853;862;852;850;862;848;849;871;895;853;915;850;957;870;849;868;854
NOISE: |8;linkedin.com/www.linkedin.com/in/christopherblizzard@goback=.nppvan_%252Flemuelf.html;419;139;114;123;115;119;116;113;117;117;115;116;113;114;121;113;117;119;113;113;115;115;116;114;113
NOISE: |9;bing.com/www.bing.com/search@q=mozilla&go=&form=QBLH&qs=n&sk=&sc=8-0.html;111;80;78;79;80;81;84;79;79;80;83;78;80;78;78;79;79;80;79;79;79;78;79;79;80
NOISE: |10;icanhascheezburger.com/icanhascheezburger.com/index.html;953;394;397;401;397;416;399;407;402;394;402;399;393;394;396;392;401;407;387;393;396;396;397;393;396
NOISE: |11;yandex.ru/yandex.ru/yandsearch@text=mozilla&lr=21215.html;359;195;175;204;200;200;197;202;203;205;203;204;203;201;200;205;199;201;206;217;428;206;194;202;199
NOISE: |12;cgi.ebay.com/cgi.ebay.com/ALL-NEW-KINDLE-3-eBOOK-WIRELESS-READING-DEVICE-W-WIFI-/130496077314@pt=LH_DefaultDomain_0&hash=item1e622c1e02.html;776;342;320;326;309;326;316;326;327;324;321;326;333;326;318;326;324;318;326;322;312;326;326;328;328
NOISE: |13;163.com/www.163.com/index.html;876;541;526;519;534;515;521;499;525;525;502;478;528;526;536;511;521;534;513;521;508;503;520;527;532
NOISE: |14;mail.ru/mail.ru/index.html;462;248;236;248;274;245;237;245;241;255;245;243;243;248;249;250;242;257;247;252;242;244;246;250;240
NOISE: |15;bbc.co.uk/www.bbc.co.uk/news/index.html;594;288;291;289;293;308;289;288;291;292;291;289;292;293;292;293;292;290;292;289;292;297;294;292;289
NOISE: |16;store.apple.com/store.apple.com/us@mco=Nzc1MjMwNA.html;604;376;391;369;373;371;370;369;372;373;370;377;372;374;377;375;372;374;369;374;371;372;370;373;372
NOISE: |17;imdb.com/www.imdb.com/title/tt1099212/index.html;667;374;334;361;380;371;362;365;365;364;360;367;367;366;366;363;369;368;376;363;334;371;361;360;364
NOISE: |18;mozilla.com/www.mozilla.com/en-US/firefox/all-older.html;972;290;291;288;285;395;367;290;290;302;374;289;294;293;294;289;292;295;288;296;313;290;291;293;304
NOISE: |19;ask.com/www.ask.com/web@q=What%27s+the+difference+between+brown+and+white+eggs%253F&gc=1&qsrc=3045&o=0&l=dir.html;564;196;198;192;195;199;195;193;195;193;197;194;196;196;199;197;198;197;198;192;195;193;194;193;191
NOISE: |20;cnn.com/www.cnn.com/index.html;747;443;452;443;447;441;446;446;448;457;453;442;449;451;442;453;448;449;508;445;453;457;449;456;449
NOISE: |21;sohu.com/www.sohu.com/index.html;754;460;465;463;437;448;451;463;435;461;454;453;472;429;462;452;459;448;458;473;453;458;464;456;464
NOISE: |22;vkontakte.ru/vkontakte.ru/help.php@page=about.html;328;117;121;112;123;108;109;111;122;138;113;123;118;112;118;120;122;127;117;119;111;111;111;118;111
NOISE: |23;youku.com/www.youku.com/index.html;882;468;445;452;454;447;448;445;446;452;455;477;450;445;465;445;447;447;446;446;451;448;445;446;444
NOISE: |24;myparentswereawesome.tumblr.com/myparentswereawesome.tumblr.com/index.html;388;131;147;146;144;144;144;146;147;148;169;144;147;144;146;145;151;148;146;145;143;149;147;148;147
NOISE: |25;ifeng.com/ifeng.com/index.html;986;404;381;386;386;380;382;382;380;385;381;370;380;379;386;383;386;393;384;389;380;381;380;388;432
NOISE: |26;ameblo.jp/ameblo.jp/index.html;639;116;123;124;126;127;130;127;124;123;126;124;124;130;127;128;129;129;125;124;128;130;127;123;128
NOISE: |27;tudou.com/www.tudou.com/index.html;562;275;272;275;273;278;289;281;273;280;274;279;274;274;288;300;279;274;278;278;275;278;274;273;275
NOISE: |28;chemistry.about.com/chemistry.about.com/index.html;414;155;165;158;158;159;162;159;164;160;158;159;163;164;162;160;159;158;168;163;160;157;163;159;159
NOISE: |29;beatonna.livejournal.com/beatonna.livejournal.com/index.html;464;112;109;113;110;110;110;111;97;109;105;112;108;109;107;114;114;95;107;110;112;95;108;106;96
NOISE: |30;hao123.com/hao123.com/index.html;378;133;136;136;137;131;133;137;134;136;133;137;131;135;134;133;134;133;135;133;136;133;133;135;133
NOISE: |31;rakuten.co.jp/www.rakuten.co.jp/index.html;771;382;392;387;409;392;415;413;379;406;391;414;386;395;385;388;396;389;386;385;387;385;388;389;389
NOISE: |32;alibaba.com/www.alibaba.com/product-tp/101509462/World_s_Cheapest_Laptop.html;194;127;124;127;126;123;130;110;113;111;127;114;130;109;124;127;114;126;113;126;115;114;113;125;112
NOISE: |33;uol.com.br/www.uol.com.br/index.html;785;477;486;485;486;485;480;479;486;468;476;482;484;483;485;488;485;485;485;480;483;478;486;486;487
NOISE: |34;cnet.com/www.cnet.com/index.html;670;406;435;408;413;408;410;411;408;413;415;408;409;416;410;412;410;416;420;406;405;417;410;406;407
NOISE: |35;ehow.com/www.ehow.com/how_4575878_prevent-fire-home.html;379;172;137;137;169;136;166;132;164;137;137;140;137;137;136;133;166;136;167;137;163;136;171;136;173
NOISE: |36;thepiratebay.org/thepiratebay.org/top/201.html;368;219;223;255;230;221;229;227;221;222;224;221;224;195;203;222;223;225;231;226;222;202;226;221;231
NOISE: |37;page.renren.com/page.renren.com/index.html;501;242;251;253;252;251;252;245;254;230;254;248;252;248;252;249;251;248;251;248;252;251;249;249;250
NOISE: |38;chinaz.com/chinaz.com/index.html;471;230;224;227;225;226;227;264;231;225;233;227;227;225;227;225;227;232;231;232;228;241;227;227;227
NOISE: |39;globo.com/www.globo.com/index.html;815;471;472;487;471;473;499;472;476;481;481;459;474;472;483;474;481;471;470;474;480;468;481;474;519
NOISE: |40;spiegel.de/www.spiegel.de/index.html;647;412;397;402;403;404;399;403;400;404;402;402;403;405;404;397;399;401;408;433;408;401;398;404;402
NOISE: |41;dailymotion.com/www.dailymotion.com/us.html;572;333;328;335;327;330;339;339;341;332;330;348;335;340;334;334;375;331;333;328;333;329;335;328;327
NOISE: |42;goo.ne.jp/goo.ne.jp/index.html;275;107;143;137;107;129;125;130;107;109;108;129;108;130;108;134;127;132;126;128;109;130;137;136;103
NOISE: |43;alipay.com/www.alipay.com/index.html;313;130;160;157;153;156;154;157;155;158;154;157;154;160;151;153;154;159;152;155;156;154;162;155;153
NOISE: |44;stackoverflow.com/stackoverflow.com/questions/184618/what-is-the-best-comment-in-source-code-you-have-ever-encountered.html;528;373;384;373;383;383;383;369;372;372;372;373;371;373;378;382;373;375;392;401;379;370;371;386;409
NOISE: |45;nicovideo.jp/www.nicovideo.jp/index.html;444;197;211;209;208;214;208;215;210;214;215;214;209;214;209;213;209;217;209;218;213;216;213;215;214
NOISE: |46;ezinearticles.com/ezinearticles.com/index.html@Migraine-Ocular---The-Eye-Migraines&id=4684133.html;211;168;169;199;153;164;169;168;146;164;167;166;166;178;173;172;182;167;167;172;169;170;168;172;168
NOISE: |47;taringa.net/www.taringa.net/index.html;572;302;351;353;351;354;377;338;359;351;346;341;346;355;309;293;291;350;341;351;354;296;353;347;349
NOISE: |48;tmall.com/www.tmall.com/index.html@ver=2010s.html;1767;1544;1513;1515;1549;1512;1508;1549;1516;1513;1535;1512;1546;1515;1513;1515;1547;1512;1584;1511;1507;1515;1513;1507;1513
NOISE: |49;huffingtonpost.com/www.huffingtonpost.com/index.html;976;797;818;802;788;777;798;802;776;802;781;777;817;787;774;789;812;802;777;779;818;784;791;802;802
NOISE: |50;deviantart.com/www.deviantart.com/index.html;493;228;227;229;227;226;226;225;229;229;219;223;223;226;228;226;226;226;226;227;230;223;225;223;227
NOISE: |51;media.photobucket.com/media.photobucket.com/image/funny%20gif/findstuff22/Best%20Images/Funny/funny-gif1.jpg@o=1.html;493;258;257;257;268;256;258;256;263;268;258;263;255;262;263;267;267;256;266;259;269;262;268;266;255
NOISE: |52;douban.com/www.douban.com/index.html;549;248;241;249;246;240;239;240;244;239;241;249;243;239;237;244;240;238;245;245;238;237;237;240;239
NOISE: |53;imgur.com/imgur.com/gallery/index.html;530;244;253;271;253;272;255;254;263;253;253;254;258;257;250;252;257;262;254;252;233;259;271;257;260
NOISE: |54;reddit.com/www.reddit.com/index.html;333;242;243;236;240;238;239;236;235;236;234;235;236;235;238;286;235;242;241;242;248;242;242;237;246
NOISE: |55;digg.com/digg.com/news/story/New_logo_for_Mozilla_Firefox_browser.html;496;401;392;402;407;395;399;409;401;403;401;401;405;385;406;394;393;393;398;402;396;398;391;402;399
NOISE: |56;filestube.com/www.filestube.com/t/the+vampire+diaries.html;366;219;212;212;219;221;227;212;219;227;227;263;213;221;220;219;227;221;225;223;233;214;226;215;227
NOISE: |57;dailymail.co.uk/www.dailymail.co.uk/ushome/index.html;1137;684;677;662;673;667;672;658;688;691;670;676;688;670;669;662;679;690;684;674;670;727;669;679;671
NOISE: |58;whois.domaintools.com/whois.domaintools.com/mozilla.com.html;241;97;115;109;110;112;112;113;114;110;116;112;113;112;113;111;113;104;115;109;113;115;113;105;103
NOISE: |59;indiatimes.com/www.indiatimes.com/index.html;517;384;337;340;342;331;340;342;342;327;341;329;335;333;340;344;337;334;332;333;332;344;340;337;326
NOISE: |60;rambler.ru/www.rambler.ru/index.html;392;202;197;201;201;202;204;210;210;202;178;202;204;202;203;207;210;202;204;203;212;201;202;205;202
NOISE: |61;torrentz.eu/torrentz.eu/search@q=movies.html;239;171;189;171;154;169;156;163;158;160;163;166;165;164;157;165;139;164;156;163;156;160;156;165;160
NOISE: |62;reuters.com/www.reuters.com/index.html;728;463;471;491;465;511;464;478;461;467;465;456;465;470;469;466;468;462;461;465;466;479;473;471;497
NOISE: |63;foxnews.com/www.foxnews.com/index.html;677;301;305;308;304;308;307;305;304;309;306;307;306;306;308;305;304;323;303;304;302;302;344;305;307
NOISE: |64;xinhuanet.com/xinhuanet.com/index.html;1420;652;670;673;673;681;679;657;665;667;688;689;711;679;658;669;675;671;654;677;678;677;684;667;659
NOISE: |65;56.com/www.56.com/index.html;941;622;615;579;577;552;552;553;577;594;567;563;572;548;554;548;561;612;608;563;563;563;599;569;563
NOISE: |66;bild.de/www.bild.de/index.html;1564;1011;1010;1011;1008;1007;1052;1016;1010;1016;1005;1009;1006;1018;1009;1013;1010;1056;1009;1013;1008;1011;1023;1012;1026
NOISE: |67;guardian.co.uk/www.guardian.co.uk/index.html;623;326;308;302;299;297;343;303;299;298;327;325;300;297;305;325;311;326;298;326;297;334;325;322;327
NOISE: |68;w3schools.com/www.w3schools.com/html/default.asp.html;340;217;204;225;213;261;225;219;221;222;225;219;223;226;218;217;239;222;224;219;225;218;216;217;224
NOISE: |69;naver.com/www.naver.com/index.html;895;299;296;305;298;294;292;296;295;299;302;298;304;297;298;298;301;299;303;295;271;297;291;302;296
NOISE: |70;blogfa.com/blogfa.com/index.html;346;77;55;76;72;68;68;72;69;74;70;72;76;69;69;72;73;74;70;72;72;74;70;69;73
NOISE: |71;terra.com.br/www.terra.com.br/portal/index.html;521;325;340;329;341;329;339;328;343;331;341;330;340;334;330;336;327;335;340;328;339;328;339;337;338
NOISE: |72;ucoz.ru/www.ucoz.ru/index.html;311;173;167;182;179;178;175;171;179;170;177;175;180;184;167;215;172;171;177;176;178;177;179;173;172
NOISE: |73;yelp.com/www.yelp.com/biz/alexanders-steakhouse-cupertino.html;676;460;466;466;460;462;463;464;460;465;486;463;462;465;466;462;466;462;462;462;471;466;458;469;467
NOISE: |74;wsj.com/online.wsj.com/home-page.html;827;558;557;559;557;555;541;561;558;565;556;556;556;560;555;556;557;563;553;556;557;567;556;552;554
NOISE: |75;noimpactman.typepad.com/noimpactman.typepad.com/index.html;473;294;288;286;286;282;302;286;284;284;283;288;287;290;285;285;281;288;283;288;289;329;291;294;281
NOISE: |76;myspace.com/www.myspace.com/albumart.html;906;585;583;582;584;585;583;586;584;584;584;588;584;631;587;590;583;590;584;585;583;585;583;589;582
NOISE: |77;google.com/www.google.com/search@q=mozilla.html;142;102;77;93;91;100;95;77;82;76;90;86;97;94;90;96;81;83;91;79;96;95;95;96;80
NOISE: |78;orange.fr/www.orange.fr/index.html;267;115;102;99;98;97;98;101;100;100;98;99;99;99;101;100;98;98;99;99;98;99;99;96;100
NOISE: |79;php.net/php.net/index.html;161;109;108;98;107;104;109;102;105;106;104;106;123;104;105;103;105;104;104;87;104;104;105;105;108
NOISE: |80;zol.com.cn/www.zol.com.cn/index.html;1159;718;703;760;702;715;702;714;700;703;712;714;702;703;719;700;716;700;687;716;699;714;699;700;715
NOISE: |81;mashable.com/mashable.com/index.html;799;506;488;483;479;486;480;488;485;468;484;483;475;487;491;485;481;481;480;480;482;486;486;478;487
NOISE: |82;etsy.com/www.etsy.com/category/geekery/videogame.html;291;136;139;180;137;137;137;138;154;161;135;137;136;153;136;138;154;138;152;157;152;138;137;136;153
NOISE: |83;gmx.net/www.gmx.net/index.html;409;217;222;220;224;202;217;226;221;277;221;228;220;233;222;220;232;223;222;220;221;231;224;207;223
NOISE: |84;csdn.net/csdn.net/index.html;645;208;215;207;215;216;213;212;214;210;214;209;210;208;213;211;210;211;215;209;211;211;211;211;218
NOISE: |85;xunlei.com/xunlei.com/index.html;820;677;655;671;683;650;648;661;649;652;677;652;653;649;657;660;660;658;658;662;659;648;657;662;663
NOISE: |86;hatena.ne.jp/www.hatena.ne.jp/index.html;624;158;138;138;135;137;161;138;161;138;138;136;139;174;159;137;138;136;142;142;138;138;136;136;142
NOISE: |87;icious.com/www.delicious.com/index.html;325;229;274;235;231;229;235;235;230;232;229;225;235;224;227;231;230;231;233;225;232;227;226;228;231
NOISE: |88;repubblica.it/www.repubblica.it/index.html;816;348;354;351;343;348;354;347;353;346;348;351;352;348;355;347;354;347;350;347;353;346;350;348;355
NOISE: |89;web.de/web.de/index.html;456;239;273;224;232;220;244;227;228;221;233;226;231;232;227;222;228;227;229;228;233;221;227;227;229
NOISE: |90;slideshare.net/www.slideshare.net/jameswillamor/lolcats-in-popular-culture-a-historical-perspective.html;475;313;323;316;365;312;310;320;317;318;327;314;310;317;318;318;315;319;319;307;317;318;318;318;321
NOISE: |91;telegraph.co.uk/www.telegraph.co.uk/index.html;559;314;302;291;298;300;300;301;296;301;297;297;296;303;297;303;298;300;300;296;313;299;301;296;303
NOISE: |92;seesaa.net/blog.seesaa.jp/index.html;776;253;250;252;254;254;253;254;251;256;252;251;253;250;254;253;254;254;256;252;252;255;251;251;252
NOISE: |93;wp.pl/www.wp.pl/index.html;435;161;224;214;225;241;225;219;227;218;225;225;224;224;226;225;227;223;225;226;225;220;228;220;231
NOISE: |94;aljazeera.net/aljazeera.net/portal.html;838;356;416;362;366;358;364;372;357;367;354;357;368;356;360;360;362;366;363;359;359;356;370;358;402
NOISE: |95;w3.org/www.w3.org/standards/webdesign/htmlcss.html;195;142;128;142;129;133;129;144;130;133;131;129;131;142;131;129;138;131;133;133;137;141;129;134;128
NOISE: |96;homeway.com.cn/www.hexun.com/index.html;685;363;367;371;426;373;365;372;378;369;375;367;371;379;369;369;376;368;385;371;377;374;377;366;360
NOISE: |97;facebook.com/www.facebook.com/Google.html;241;117;112;107;113;107;113;106;112;142;109;126;141;109;140;115;116;114;120;120;134;130;142;154;112
NOISE: |98;youtube.com/www.youtube.com/music.html;497;269;279;281;280;282;318;279;280;277;282;283;280;283;278;282;282;280;282;281;283;277;274;283;274
NOISE: |99;people.com.cn/people.com.cn/index.html;608;338;339;337;345;341;376;333;337;338;336;342;335;343;341;342;336;340;340;341;340;340;341;340;337
NOISE: __end_tp_report
NOISE: __start_cc_report
NOISE: _x_x_mozilla_cycle_collect,161163
NOISE: __end_cc_report
NOISE: __startTimestamp1353963640994__endTimestamp
NOISE: 
NOISE: MOZ_EVENT_TRACE sample 1353963641078 383
NOISE: MOZ_EVENT_TRACE stop 1353963641078
Failed tp5n: 
		Stopped Mon, 26 Nov 2012 13:07:21
Traceback (most recent call last):
  File "run_tests.py", line 261, in run_tests
    talos_results.add(mytest.runTest(browser_config, test))
  File "c:\talos-slave\talos-data\talos\ttest.py", line 414, in runTest
    test_results.add(browser_log_filename, counter_results=counter_results)
  File "c:\talos-slave\talos-data\talos\results.py", line 120, in add
    browserLog = BrowserLogResults(filename=results, counter_results=counter_results, global_counters=self.global_counters)
  File "c:\talos-slave\talos-data\talos\results.py", line 319, in __init__
    self.parse()
  File "c:\talos-slave\talos-data\talos\results.py", line 356, in parse
    self.error("Could not find %s in browser output: (tokens: %s)" % (attr, tokens))
  File "c:\talos-slave\talos-data\talos\results.py", line 331, in error
    raise utils.talosError(message)
talosError: "Could not find beforeLaunchTime in browser output: (tokens: ('__startBeforeLaunchTimestamp', '__endBeforeLaunchTimestamp')) [browser_output.txt]"
Traceback (most recent call last):
  File "run_tests.py", line 311, in ?
    main()
  File "run_tests.py", line 308, in main
    run_tests(parser)
  File "run_tests.py", line 269, in run_tests
    raise e
utils.talosError: "Could not find beforeLaunchTime in browser output: (tokens: ('__startBeforeLaunchTimestamp', '__endBeforeLaunchTimestamp')) [browser_output.txt]"
program finished with exit code 1
elapsedTime=2046.671000

Not sure why this would happen o_O
I'm leaning towards WONTFIXing this until we can use > python 2.5 and/or just switching to mozprocess. This is a lot of work for not much reward
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.