Closed
Bug 1059847
Opened 10 years ago
Closed 10 years ago
logviewer: Clicking an error line for a log deleted from ftp shows perpetual "Loading..."
Categories
(Tree Management :: Treeherder, defect, P2)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: automatedtester)
References
Details
Attachments
(1 file)
1) Find a Try job that ran >30 days ago (since that's the timeframe for logs being purged from ftp for try jobs). eg https://treeherder.mozilla.org/ui/logviewer.html#?job_id=1013218&repo=try 2) Click on one of the errors in the overview Expected: Error message of "Log not found" Actual: Perpetual "Loading..." Webconsole shows: GET https://treeherder.mozilla.org/api/project/try/logslice/ [HTTP/1.1 404 NOT FOUND 216ms] And https://treeherder.mozilla.org/api/project/try/logslice/?end_line=7050&job_id=1013218&start_line=6800 returns: {"detail": "log file not found"}
Assignee | ||
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder-ui https://github.com/mozilla/treeherder-ui/commit/3dab70bf5b80cce36791111eb116ce0fa65fd3c1 Bug 1059847: Show 'Log not found error' instead of 'loading' when the log is no longer available https://github.com/mozilla/treeherder-ui/commit/acaa14ebca07b2123069043cc69c1b280b8e2e9a Merge pull request #173 from AutomatedTester/1059847 Bug 1059847: Show 'Log not found error' instead of 'loading' when the log is no longer available
Updated•10 years ago
|
Attachment #8488264 -
Flags: review?(mdoglio) → review+
Reporter | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 3•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/68feb17759eb6b8d8b6b2b71ced9def549b8faf3 Bug 1059847: Show 'Log not found error' instead of 'loading' when the log is no longer available https://github.com/mozilla/treeherder/commit/0b222480ac757b2fa90afebcdfa7b6d662fd5a4a Merge pull request #173 from AutomatedTester/1059847 Bug 1059847: Show 'Log not found error' instead of 'loading' when the log is no longer available
You need to log in
before you can comment on or make changes to this bug.
Description
•