get_site_to_test in cesiumcron doesn't really get sites with more than one user

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
2 years ago

People

(Reporter: jbalogh, Assigned: jbalogh)

Tracking

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

9 years ago
Created attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test

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)
Attachment #397749 - Flags: review?(rdoherty) → review-
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

9 years ago
Attachment #397749 - Flags: review?(rdoherty)
Attachment #397749 - Flags: review?
Attachment #397749 - Flags: review-
(Assignee)

Comment 3

9 years ago
Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test

updated spawnff to handle generators
(Assignee)

Updated

9 years ago
Attachment #397749 - Flags: review?
Attachment #397749 - Flags: review?(rdoherty) → review+
(Assignee)

Comment 5

9 years ago
Thanks, committed in r50719.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.