{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/760","labels_url":"https://api.github.com/repos/openshift/origin/issues/760/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/760/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/760/events","html_url":"https://github.com/openshift/origin/pull/760","id":55678793,"number":760,"title":"refactor build interfaces","user":{"login":"bparees","id":4974046,"avatar_url":"https://avatars.githubusercontent.com/u/4974046?v=3","gravatar_id":"","url":"https://api.github.com/users/bparees","html_url":"https://github.com/bparees","followers_url":"https://api.github.com/users/bparees/followers","following_url":"https://api.github.com/users/bparees/following{/other_user}","gists_url":"https://api.github.com/users/bparees/gists{/gist_id}","starred_url":"https://api.github.com/users/bparees/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bparees/subscriptions","organizations_url":"https://api.github.com/users/bparees/orgs","repos_url":"https://api.github.com/users/bparees/repos","events_url":"https://api.github.com/users/bparees/events{/privacy}","received_events_url":"https://api.github.com/users/bparees/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-01-27T21:43:51Z","updated_at":"2015-01-29T08:34:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/760","html_url":"https://github.com/openshift/origin/pull/760","diff_url":"https://github.com/openshift/origin/pull/760.diff","patch_url":"https://github.com/openshift/origin/pull/760.patch"},"body":"Attempting to bring some structure to the build interfaces we had scattered about the code...  the intent is to define all the interfaces and the \"standard\" implementation in the util.go file, define the mock impls in the test cases that use them, and reduce the number of interfaces (rather than having an interface per build operation, have an interface that aggregates all the build operations).\r\n\r\nthis makes it slightly harder to mock the interface since you have to provide more dummy implementations, but it makes it much easier (imho) to understand the set of interfaces involved in the build process and where they're implemented.\r\n"},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/71987751","html_url":"https://github.com/openshift/origin/pull/760#issuecomment-71987751","issue_url":"https://api.github.com/repos/openshift/origin/issues/760","id":71987751,"user":{"login":"openshift-bot","id":1779249,"avatar_url":"https://avatars.githubusercontent.com/u/1779249?v=3","gravatar_id":"","url":"https://api.github.com/users/openshift-bot","html_url":"https://github.com/openshift-bot","followers_url":"https://api.github.com/users/openshift-bot/followers","following_url":"https://api.github.com/users/openshift-bot/following{/other_user}","gists_url":"https://api.github.com/users/openshift-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/openshift-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openshift-bot/subscriptions","organizations_url":"https://api.github.com/users/openshift-bot/orgs","repos_url":"https://api.github.com/users/openshift-bot/repos","events_url":"https://api.github.com/users/openshift-bot/events{/privacy}","received_events_url":"https://api.github.com/users/openshift-bot/received_events","type":"User","site_admin":false},"created_at":"2015-01-29T08:34:50Z","updated_at":"2015-01-29T08:34:50Z","body":"continuous-integration/openshift-jenkins/test Waiting: Determining build queue position"}}