Closed Bug 1136869 Opened 10 years ago Closed 10 years ago

job updates on vagrant fail with 500 error

Categories

(Tree Management :: Treeherder: API, defect, P1)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: camd, Assigned: mdoglio)

References

Details

Attachments

(2 files)

Get this error when sitting on the page and these job update endpoints are hit: http://local.treeherder.mozilla.org/api/project/try/jobs/?count=2000&last_modified__gt=2015-02-25T00:19:09.000Z&result_set_id__in=3,1,2,13,35,37,7,24,10,5&return_type=list Warning: Incorrect datetime value: '2015-02-25T00:19:09.000Z' for column 'last_modified' at row 1
Attached file full response stack trace (deleted) —
Priority: -- → P1
Blocks: 1097090
Mauro, would you mind taking a look at this? Both Cameron and my local vagrant environments no longer work after a vagrant destroy and up.
Flags: needinfo?(mdoglio)
Apparently mysql raises a warning because of the 'Z' bit in the last_modified filter. For some reasons Datasource (or MySQLdb) transforms it in an exception when you pass show_debug=True to the execute method. That means that on staging/production there's no such a problem, and the query is filtering correctly. The easiest way to solve it is to strip out the 'Z', I'll open a PR for it
Flags: needinfo?(mdoglio)
(In reply to Mauro Doglio [:mdoglio] from comment #3) > The easiest way to solve it is to strip out the 'Z', I'll open a PR for it
Flags: needinfo?(mdoglio)
Attached file Github PR #432 on treeherder-ui (deleted) —
Flags: needinfo?(mdoglio)
Attachment #8580766 - Flags: review?(emorley)
Comment on attachment 8580766 [details] Github PR #432 on treeherder-ui Thanks! :-)
Attachment #8580766 - Flags: review?(emorley) → review+
Assignee: nobody → mdoglio
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
I confirm the before and after change in the dev network console but I didn't see an error in comment 0 on Firefox pre-fix, which is strange (testing with local server, not vagrant). Maybe it would be helpful if camd re-tried his str on latest master to be sure of this fix in his env.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: