It never gets past "Loading pushlog..." on the top. It was like this since this morning.
Getting this in the error console for Tracemonkey: Error: /([^<]+) /.exec(patch.author) is null Source File: http://tests.themasta.com/tinderboxpushlog/js/PushlogJSONParser.js Line: 20
Try TM after 8:30 tonight, and JM after 11:13 tonight - those thousand-changeset merges are hard on pushlog, and it tends not to manage to finish before tbpl times it out.
The same problem is happening on TryServer's TBPL now.
I think this is happening because author names don't always have in brackets (most but not all are "User Name <email>", some are just "email"): $ curl -s 'http://hg.mozilla.org/try/json-pushes?full=1&startdate=12+hours+ago&enddate=now' | grep author | grep -v '<' "author": "firstname.lastname@example.org", Regression caused by bug 584836?
Created attachment 467606 [details] [diff] [review] check for < in author name before using regex
Comment on attachment 467606 [details] [diff] [review] check for < in author name before using regex We can just tweak the regex to exclude the space. Pushed with that change (and a trim() around it to remove a possible trailing space): http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/1fd4bd863cc7
Assignee: nobody → robert
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
WFM locally using a pushlog JSON feed with missing "<".
Status: RESOLVED → VERIFIED
Thanks very much! This is a key time for the JS team, and I've been really relying on TBPL, so it was great to have a prompt fix.
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.