{"action":"created","issue":{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/1495","labels_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/1495/labels{/name}","comments_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/1495/comments","events_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/1495/events","html_url":"https://github.com/RedHatQE/cfme_tests/pull/1495","id":53673800,"number":1495,"title":"Post results for failed builds","user":{"login":"seandst","id":441252,"avatar_url":"https://avatars.githubusercontent.com/u/441252?v=3","gravatar_id":"","url":"https://api.github.com/users/seandst","html_url":"https://github.com/seandst","followers_url":"https://api.github.com/users/seandst/followers","following_url":"https://api.github.com/users/seandst/following{/other_user}","gists_url":"https://api.github.com/users/seandst/gists{/gist_id}","starred_url":"https://api.github.com/users/seandst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seandst/subscriptions","organizations_url":"https://api.github.com/users/seandst/orgs","repos_url":"https://api.github.com/users/seandst/repos","events_url":"https://api.github.com/users/seandst/events{/privacy}","received_events_url":"https://api.github.com/users/seandst/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/labels/lint+ok","name":"lint ok","color":"328832"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-07T19:49:59Z","updated_at":"2015-01-08T12:03:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/pulls/1495","html_url":"https://github.com/RedHatQE/cfme_tests/pull/1495","diff_url":"https://github.com/RedHatQE/cfme_tests/pull/1495.diff","patch_url":"https://github.com/RedHatQE/cfme_tests/pull/1495.patch"},"body":"Merging this will break trackerbot's graph rendering, since it doesn't yet understand the idea of \"-1\" for the number of tests (and in fact, the API currently turns those values to 0, so the flot JS div0 errors). But if I get buy-in (mainly from @psav) on the \"-1 means the build failed\" idea, I can get trackerbot patched up pretty quickly to deal with failed builds and then we can merge this."},"comment":{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/comments/69170820","html_url":"https://github.com/RedHatQE/cfme_tests/pull/1495#issuecomment-69170820","issue_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/1495","id":69170820,"user":{"login":"psav","id":5639745,"avatar_url":"https://avatars.githubusercontent.com/u/5639745?v=3","gravatar_id":"","url":"https://api.github.com/users/psav","html_url":"https://github.com/psav","followers_url":"https://api.github.com/users/psav/followers","following_url":"https://api.github.com/users/psav/following{/other_user}","gists_url":"https://api.github.com/users/psav/gists{/gist_id}","starred_url":"https://api.github.com/users/psav/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/psav/subscriptions","organizations_url":"https://api.github.com/users/psav/orgs","repos_url":"https://api.github.com/users/psav/repos","events_url":"https://api.github.com/users/psav/events{/privacy}","received_events_url":"https://api.github.com/users/psav/received_events","type":"User","site_admin":false},"created_at":"2015-01-08T12:03:59Z","updated_at":"2015-01-08T12:03:59Z","body":"Yeh, I don't see an issue with the -1 for failed build, though a part of me wonders if we should have a \"build_result_code\" field in the db instead. At which point we could ignore builds for which that is set to non-zero?"}}