Closed
Bug 513376
Opened 15 years ago
Closed 15 years ago
get_site_to_test in cesiumcron doesn't really get sites with more than one user
Categories
(Webtools Graveyard :: Cesium, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jbalogh, Assigned: jbalogh)
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•15 years ago
|
||
This awesome query requires Django 1.1. I've updated the pip requirements doc, make sure you pip install -r again.
Attachment #397749 -
Flags: review?(rdoherty)
Updated•15 years ago
|
Attachment #397749 -
Flags: review?(rdoherty) → review-
Comment 2•15 years ago
|
||
Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test
r- for tracking purposes.
manage.py cesiumcron spits out errors.
File "/Users/rdoherty/Documents/Mozilla/projects/cesium/autoyslow/spawnff.py", line 16, in run_test
if len(page_list) == 0:
TypeError: object of type 'generator' has no len()
Assignee | ||
Updated•15 years ago
|
Attachment #397749 -
Flags: review?(rdoherty)
Attachment #397749 -
Flags: review?
Attachment #397749 -
Flags: review-
Assignee | ||
Comment 3•15 years ago
|
||
Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test
updated spawnff to handle generators
Assignee | ||
Updated•15 years ago
|
Attachment #397749 -
Flags: review?
Updated•15 years ago
|
Attachment #397749 -
Flags: review?(rdoherty) → review+
Comment 4•15 years ago
|
||
Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test
Looks good!
Assignee | ||
Comment 5•15 years ago
|
||
Thanks, committed in r50719.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•