{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/treeherder/issues/859","labels_url":"https://api.github.com/repos/mozilla/treeherder/issues/859/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/treeherder/issues/859/comments","events_url":"https://api.github.com/repos/mozilla/treeherder/issues/859/events","html_url":"https://github.com/mozilla/treeherder/pull/859","id":100349313,"number":859,"title":"Bug 1192917 - Handle 'unknown' steps and jobs with improved UI","user":{"login":"tojonmz","id":3660661,"avatar_url":"https://avatars.githubusercontent.com/u/3660661?v=3","gravatar_id":"","url":"https://api.github.com/users/tojonmz","html_url":"https://github.com/tojonmz","followers_url":"https://api.github.com/users/tojonmz/followers","following_url":"https://api.github.com/users/tojonmz/following{/other_user}","gists_url":"https://api.github.com/users/tojonmz/gists{/gist_id}","starred_url":"https://api.github.com/users/tojonmz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tojonmz/subscriptions","organizations_url":"https://api.github.com/users/tojonmz/orgs","repos_url":"https://api.github.com/users/tojonmz/repos","events_url":"https://api.github.com/users/tojonmz/events{/privacy}","received_events_url":"https://api.github.com/users/tojonmz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-08-11T16:11:16Z","updated_at":"2015-08-14T15:06:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozilla/treeherder/pulls/859","html_url":"https://github.com/mozilla/treeherder/pull/859","diff_url":"https://github.com/mozilla/treeherder/pull/859.diff","patch_url":"https://github.com/mozilla/treeherder/pull/859.patch"},"body":"This work fixes Bugzilla bug [1192917](https://bugzilla.mozilla.org/show_bug.cgi?id=1192917).\r\n\r\nThis provides a new color treatment for unknown jobs (formerly black), and raises visibility in the UI when the log artifact has a similar result, which we see fairly frequently. We use Bootstrap [warning](http://getbootstrap.com/components/#alerts) alert colors.\r\n\r\nUnknown jobs (mocked on running jobs, since they are relatively rare):\r\n![unknownjobs](https://cloud.githubusercontent.com/assets/3660661/9202200/b64be4f6-401e-11e5-9f36-be2fd067df8c.jpg)\r\n\r\nUnknown Logviewer step result:\r\n![unknownstep](https://cloud.githubusercontent.com/assets/3660661/9202224/d5b2931c-401e-11e5-9473-45a6bca88c37.jpg)\r\n\r\nAnd a corresponding tooltip to spell it out to the consumer:\r\n![tooltip](https://cloud.githubusercontent.com/assets/3660661/9202239/e639ce6c-401e-11e5-94dd-e5a1666b6fb9.jpg)\r\n\r\nEverything seems fine on both browsers, testing a variety of jobs, successful, failed, busted, etc.\r\n\r\nWhile this job exists on stage, this is a suitable logviewer path for branch testing. But there are many other Gaia Python Integration Tests and other tests available for the failure case.\r\n`/logviewer.html#?job_id=9397555&repo=mozilla-inbound`\r\n\r\nTested on OSX 10.10.3:\r\nNightly **42.0a1 (2015-08-10)**\r\nChrome Latest Release **44.0.2403.130 (64-bit)**\r\n\r\nAdding @camd for review.\n\n<!-- Reviewable:start -->\n[<img src=\"https://reviewable.io/review_button.png\" height=40 alt=\"Review on Reviewable\"/>](https://reviewable.io/reviews/mozilla/treeherder/859)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/mozilla/treeherder/issues/comments/131140107","html_url":"https://github.com/mozilla/treeherder/pull/859#issuecomment-131140107","issue_url":"https://api.github.com/repos/mozilla/treeherder/issues/859","id":131140107,"user":{"login":"tojonmz","id":3660661,"avatar_url":"https://avatars.githubusercontent.com/u/3660661?v=3","gravatar_id":"","url":"https://api.github.com/users/tojonmz","html_url":"https://github.com/tojonmz","followers_url":"https://api.github.com/users/tojonmz/followers","following_url":"https://api.github.com/users/tojonmz/following{/other_user}","gists_url":"https://api.github.com/users/tojonmz/gists{/gist_id}","starred_url":"https://api.github.com/users/tojonmz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tojonmz/subscriptions","organizations_url":"https://api.github.com/users/tojonmz/orgs","repos_url":"https://api.github.com/users/tojonmz/repos","events_url":"https://api.github.com/users/tojonmz/events{/privacy}","received_events_url":"https://api.github.com/users/tojonmz/received_events","type":"User","site_admin":false},"created_at":"2015-08-14T15:06:02Z","updated_at":"2015-08-14T15:06:02Z","body":"I also tried fa-warning for the unknown step container. I'm still inclined to just go with the text '(!)' unless anyone feels strongly. Since this is something that doesn't impact the log or its validity unlike a missing log artifact (which we flag with fa-warning adjacent to the raw log button).\r\n\r\n![exlamation](https://cloud.githubusercontent.com/assets/3660661/9276738/baf20128-4273-11e5-9043-a1cdf515e120.jpg)\r\n\r\n![fawarning](https://cloud.githubusercontent.com/assets/3660661/9276741/bf63f4e6-4273-11e5-9aad-93d98d0af07d.jpg)\r\n"}}