Closed
Bug 1388817
Opened 7 years ago
Closed 7 years ago
AWFY task_info no longer contains output
Categories
(Testing Graveyard :: AWFY, enhancement)
Testing Graveyard
AWFY
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bc, Assigned: bc)
Details
Attachments
(1 file)
48 bytes,
patch
|
bbouvier
:
review+
|
Details | Diff | Splinter Review |
Beginning with https://arewefastyet.com/task_info.php?id=101190 the Output: section of the Task output is empty. https://arewefastyet.com/task_info.php?id=101189 is the last task which contains data in the Output: section.
This began sometime between 03:30 and 05:30 AM 2017-08-09 PT.
Assignee | ||
Comment 1•7 years ago
|
||
I think that we might be having an issue with the 302 Found response for http requests. We might as well convert them all to https and eliminate the possibility. If this fixes the issue, fine. If not, we can continue looking.
Attachment #8895698 -
Flags: review?(bbouvier)
Comment 2•7 years ago
|
||
Comment on attachment 8895698 [details] [diff] [review]
https://github.com/mozilla/arewefastyet/pull/155
Review of attachment 8895698 [details] [diff] [review]:
-----------------------------------------------------------------
Merged on github, thanks!
We might need to make sure the value of updateURL in all the awfy.config files on the workers use HTTPS too (there's one per worker, located in the awfy directory, at "slave/awfy.config").
Attachment #8895698 -
Attachment is patch: true
Attachment #8895698 -
Attachment mime type: text/x-github-pull-request → text/plain
Attachment #8895698 -
Flags: review?(bbouvier) → review+
Assignee | ||
Comment 3•7 years ago
|
||
Thanks! awfy-cmp-osx.corp.mtv2.mozilla.com needed the change. The others were ok.
Comment 4•7 years ago
|
||
From an higher prospective, it's another sign that we should use some python lib to do all the requests (like `requests`). The first sign was that when treeherder switched to gzip compression of HTTP requests, we had to *manually* implement gzip decompression in order to work: see fetch_json in https://github.com/mozilla/arewefastyet/blob/master/slave/utils.py#L203
Using a request lib would make this transparent and avoid us bad surprises in the future (say, if/when treeherder starts using brotli compression or redirects too).
Comment 5•7 years ago
|
||
(In reply to Bob Clary [:bc:] from comment #3)
> Thanks! awfy-cmp-osx.corp.mtv2.mozilla.com needed the change. The others
> were ok.
Wow, that was fast, thank you!
Assignee | ||
Comment 6•7 years ago
|
||
Looks like awfy-cmp-osx.corp.mtv2.mozilla.com had been rebooted. I restarted the task.
Assignee | ||
Comment 7•7 years ago
|
||
The dashboard is showing orange tasks which contain output again. We can call this fixed.
Assignee: nobody → bob
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•7 years ago
|
Component: General → AWFY
Updated•5 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•