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)
Tree Management
Treeherder: API
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
Reporter | ||
Comment 1•10 years ago
|
||
Reporter | ||
Updated•10 years ago
|
Priority: -- → P1
Comment 2•10 years ago
|
||
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)
Assignee | ||
Comment 3•10 years ago
|
||
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)
Comment 4•10 years ago
|
||
(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)
Assignee | ||
Comment 5•10 years ago
|
||
Flags: needinfo?(mdoglio)
Attachment #8580766 -
Flags: review?(emorley)
Comment 6•10 years ago
|
||
Comment on attachment 8580766 [details]
Github PR #432 on treeherder-ui
Thanks! :-)
Attachment #8580766 -
Flags: review?(emorley) → review+
Updated•10 years ago
|
Assignee: nobody → mdoglio
Status: NEW → ASSIGNED
Comment 7•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder-ui
https://github.com/mozilla/treeherder-ui/commit/d3b9dbcbc225ecc19734430d680fc0a8c2b84562
Bug 1136869 - strip timezone indicator from last_modified
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
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.
Comment 9•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/f7258d24bef873e334aa9dec74832757a2716981
Bug 1136869 - strip timezone indicator from last_modified
You need to log in
before you can comment on or make changes to this bug.
Description
•