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

RESOLVED WONTFIX

Status

Testing
Talos
RESOLVED WONTFIX
5 years ago
5 years ago

People

(Reporter: Jeff Hammel, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

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

Attachments

(2 attachments, 3 obsolete attachments)

(Reporter)

Description

5 years ago
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.
(Reporter)

Updated

5 years ago
Whiteboard: [good first bug][mentor=jhammel][lang=py]

Comment 1

5 years ago
Under the talos folder, I didn't find the bcontroller.py.
you can find it here: 
http://hg.mozilla.org/build/talos/file/1c5976f92643/talos/bcontroller.py
Created attachment 661485 [details] [diff] [review]
Changed os.system to subproces.Popen

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.
Created attachment 662171 [details] [diff] [review]
Added try...catch block to file opeining

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+
Created attachment 662181 [details] [diff] [review]
Raised talosError exception.

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?
(Reporter)

Comment 10

5 years ago
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
(Reporter)

Comment 11

5 years ago
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

Comment 12

5 years ago
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
Created attachment 663336 [details] [diff] [review]
Passing argument as list instead of string.

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)
(Reporter)

Comment 14

5 years ago
> 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
(Reporter)

Comment 15

5 years ago
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+
(Reporter)

Comment 17

5 years ago
Created attachment 685029 [details]
use subprocess.Popen(..., shell=True)

this could be done a bit more elegantly in python 2.5+
Attachment #685029 - Flags: review?(jmaher)
(Reporter)

Comment 18

5 years ago
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+
(Reporter)

Comment 20

5 years ago
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.
(Reporter)

Comment 22

5 years ago
pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=20ceddd60cbb

Comment 23

5 years ago
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
(Reporter)

Comment 24

5 years ago
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
(Reporter)

Comment 25

5 years ago
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
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.