{"action":"created","comment":{"url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/comments/123798906","pull_request_review_id":46040619,"id":123798906,"diff_hunk":"@@ -185,7 +195,8 @@ def choose_cluster(self, platform):\n             raise RuntimeError('No clusters found for platform {}!'\n                                .format(platform))\n \n-        selected = min(clusters, key=lambda c: c.load)\n+        reachable_clusters = filter(lambda x: x.load != self.UNREACHABLE_CLUSTER_LOAD, clusters)","path":"atomic_reactor/plugins/build_orchestrate_build.py","position":45,"original_position":45,"commit_id":"c9aef1c11575f0d53240f3d8f204e55a55abdcec","original_commit_id":"c9aef1c11575f0d53240f3d8f204e55a55abdcec","user":{"login":"lcarva","id":5272931,"avatar_url":"https://avatars2.githubusercontent.com/u/5272931?v=3","gravatar_id":"","url":"https://api.github.com/users/lcarva","html_url":"https://github.com/lcarva","followers_url":"https://api.github.com/users/lcarva/followers","following_url":"https://api.github.com/users/lcarva/following{/other_user}","gists_url":"https://api.github.com/users/lcarva/gists{/gist_id}","starred_url":"https://api.github.com/users/lcarva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lcarva/subscriptions","organizations_url":"https://api.github.com/users/lcarva/orgs","repos_url":"https://api.github.com/users/lcarva/repos","events_url":"https://api.github.com/users/lcarva/events{/privacy}","received_events_url":"https://api.github.com/users/lcarva/received_events","type":"User","site_admin":false},"body":"Let's add a test case for this scenario as well.","created_at":"2017-06-23T17:07:52Z","updated_at":"2017-06-23T17:07:52Z","html_url":"https://github.com/projectatomic/atomic-reactor/pull/712#discussion_r123798906","pull_request_url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712","_links":{"self":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/comments/123798906"},"html":{"href":"https://github.com/projectatomic/atomic-reactor/pull/712#discussion_r123798906"},"pull_request":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712"}}},"pull_request":{"url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712","id":127181245,"html_url":"https://github.com/projectatomic/atomic-reactor/pull/712","diff_url":"https://github.com/projectatomic/atomic-reactor/pull/712.diff","patch_url":"https://github.com/projectatomic/atomic-reactor/pull/712.patch","issue_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/712","number":712,"state":"open","locked":false,"title":"Orchestration: exclude a cluster if builds cannot be listed","user":{"login":"vrutkovs","id":114501,"avatar_url":"https://avatars0.githubusercontent.com/u/114501?v=3","gravatar_id":"","url":"https://api.github.com/users/vrutkovs","html_url":"https://github.com/vrutkovs","followers_url":"https://api.github.com/users/vrutkovs/followers","following_url":"https://api.github.com/users/vrutkovs/following{/other_user}","gists_url":"https://api.github.com/users/vrutkovs/gists{/gist_id}","starred_url":"https://api.github.com/users/vrutkovs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vrutkovs/subscriptions","organizations_url":"https://api.github.com/users/vrutkovs/orgs","repos_url":"https://api.github.com/users/vrutkovs/repos","events_url":"https://api.github.com/users/vrutkovs/events{/privacy}","received_events_url":"https://api.github.com/users/vrutkovs/received_events","type":"User","site_admin":false},"body":"~The test is a bit weird - I can't find a way to throw exception when a specific worker's `list_build` is called, so I throw it once - and it always happens for `eggs` worker~\r\n\r\nSigned-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>","created_at":"2017-06-23T10:45:47Z","updated_at":"2017-06-23T17:07:52Z","closed_at":null,"merged_at":null,"merge_commit_sha":"9c092ad74d5afb528e27b8c4c5ce4a1fab349d73","assignee":null,"assignees":[],"requested_reviewers":[{"login":"rcerven","id":23337371,"avatar_url":"https://avatars3.githubusercontent.com/u/23337371?v=3","gravatar_id":"","url":"https://api.github.com/users/rcerven","html_url":"https://github.com/rcerven","followers_url":"https://api.github.com/users/rcerven/followers","following_url":"https://api.github.com/users/rcerven/following{/other_user}","gists_url":"https://api.github.com/users/rcerven/gists{/gist_id}","starred_url":"https://api.github.com/users/rcerven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rcerven/subscriptions","organizations_url":"https://api.github.com/users/rcerven/orgs","repos_url":"https://api.github.com/users/rcerven/repos","events_url":"https://api.github.com/users/rcerven/events{/privacy}","received_events_url":"https://api.github.com/users/rcerven/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712/commits","review_comments_url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712/comments","review_comment_url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/comments{/number}","comments_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/712/comments","statuses_url":"https://api.github.com/repos/projectatomic/atomic-reactor/statuses/c9aef1c11575f0d53240f3d8f204e55a55abdcec","head":{"label":"vrutkovs:orchestrate-improve","ref":"orchestrate-improve","sha":"c9aef1c11575f0d53240f3d8f204e55a55abdcec","user":{"login":"vrutkovs","id":114501,"avatar_url":"https://avatars0.githubusercontent.com/u/114501?v=3","gravatar_id":"","url":"https://api.github.com/users/vrutkovs","html_url":"https://github.com/vrutkovs","followers_url":"https://api.github.com/users/vrutkovs/followers","following_url":"https://api.github.com/users/vrutkovs/following{/other_user}","gists_url":"https://api.github.com/users/vrutkovs/gists{/gist_id}","starred_url":"https://api.github.com/users/vrutkovs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vrutkovs/subscriptions","organizations_url":"https://api.github.com/users/vrutkovs/orgs","repos_url":"https://api.github.com/users/vrutkovs/repos","events_url":"https://api.github.com/users/vrutkovs/events{/privacy}","received_events_url":"https://api.github.com/users/vrutkovs/received_events","type":"User","site_admin":false},"repo":{"id":52874388,"name":"atomic-reactor","full_name":"vrutkovs/atomic-reactor","owner":{"login":"vrutkovs","id":114501,"avatar_url":"https://avatars0.githubusercontent.com/u/114501?v=3","gravatar_id":"","url":"https://api.github.com/users/vrutkovs","html_url":"https://github.com/vrutkovs","followers_url":"https://api.github.com/users/vrutkovs/followers","following_url":"https://api.github.com/users/vrutkovs/following{/other_user}","gists_url":"https://api.github.com/users/vrutkovs/gists{/gist_id}","starred_url":"https://api.github.com/users/vrutkovs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vrutkovs/subscriptions","organizations_url":"https://api.github.com/users/vrutkovs/orgs","repos_url":"https://api.github.com/users/vrutkovs/repos","events_url":"https://api.github.com/users/vrutkovs/events{/privacy}","received_events_url":"https://api.github.com/users/vrutkovs/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/vrutkovs/atomic-reactor","description":"Simple python library for building docker images.","fork":true,"url":"https://api.github.com/repos/vrutkovs/atomic-reactor","forks_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/forks","keys_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/keys{/key_id}","collaborators_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/teams","hooks_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/hooks","issue_events_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/issues/events{/number}","events_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/events","assignees_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/assignees{/user}","branches_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/branches{/branch}","tags_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/tags","blobs_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/git/refs{/sha}","trees_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/git/trees{/sha}","statuses_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/statuses/{sha}","languages_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/languages","stargazers_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/stargazers","contributors_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/contributors","subscribers_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/subscribers","subscription_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/subscription","commits_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/commits{/sha}","git_commits_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/git/commits{/sha}","comments_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/comments{/number}","issue_comment_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/issues/comments{/number}","contents_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/contents/{+path}","compare_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/compare/{base}...{head}","merges_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/merges","archive_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/downloads","issues_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/issues{/number}","pulls_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/pulls{/number}","milestones_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/milestones{/number}","notifications_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/labels{/name}","releases_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/releases{/id}","deployments_url":"https://api.github.com/repos/vrutkovs/atomic-reactor/deployments","created_at":"2016-03-01T12:15:50Z","updated_at":"2016-03-01T12:15:51Z","pushed_at":"2017-06-23T12:00:12Z","git_url":"git://github.com/vrutkovs/atomic-reactor.git","ssh_url":"git@github.com:vrutkovs/atomic-reactor.git","clone_url":"https://github.com/vrutkovs/atomic-reactor.git","svn_url":"https://github.com/vrutkovs/atomic-reactor","homepage":"","size":1978,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":3,"forks":0,"open_issues":3,"watchers":0,"default_branch":"master"}},"base":{"label":"projectatomic:master","ref":"master","sha":"d8606549331605e964d99ca7f84f5a71222e5ae0","user":{"login":"projectatomic","id":6852258,"avatar_url":"https://avatars2.githubusercontent.com/u/6852258?v=3","gravatar_id":"","url":"https://api.github.com/users/projectatomic","html_url":"https://github.com/projectatomic","followers_url":"https://api.github.com/users/projectatomic/followers","following_url":"https://api.github.com/users/projectatomic/following{/other_user}","gists_url":"https://api.github.com/users/projectatomic/gists{/gist_id}","starred_url":"https://api.github.com/users/projectatomic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projectatomic/subscriptions","organizations_url":"https://api.github.com/users/projectatomic/orgs","repos_url":"https://api.github.com/users/projectatomic/repos","events_url":"https://api.github.com/users/projectatomic/events{/privacy}","received_events_url":"https://api.github.com/users/projectatomic/received_events","type":"Organization","site_admin":false},"repo":{"id":25527782,"name":"atomic-reactor","full_name":"projectatomic/atomic-reactor","owner":{"login":"projectatomic","id":6852258,"avatar_url":"https://avatars2.githubusercontent.com/u/6852258?v=3","gravatar_id":"","url":"https://api.github.com/users/projectatomic","html_url":"https://github.com/projectatomic","followers_url":"https://api.github.com/users/projectatomic/followers","following_url":"https://api.github.com/users/projectatomic/following{/other_user}","gists_url":"https://api.github.com/users/projectatomic/gists{/gist_id}","starred_url":"https://api.github.com/users/projectatomic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projectatomic/subscriptions","organizations_url":"https://api.github.com/users/projectatomic/orgs","repos_url":"https://api.github.com/users/projectatomic/repos","events_url":"https://api.github.com/users/projectatomic/events{/privacy}","received_events_url":"https://api.github.com/users/projectatomic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/projectatomic/atomic-reactor","description":"Simple python library for building docker images.","fork":false,"url":"https://api.github.com/repos/projectatomic/atomic-reactor","forks_url":"https://api.github.com/repos/projectatomic/atomic-reactor/forks","keys_url":"https://api.github.com/repos/projectatomic/atomic-reactor/keys{/key_id}","collaborators_url":"https://api.github.com/repos/projectatomic/atomic-reactor/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/projectatomic/atomic-reactor/teams","hooks_url":"https://api.github.com/repos/projectatomic/atomic-reactor/hooks","issue_events_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/events{/number}","events_url":"https://api.github.com/repos/projectatomic/atomic-reactor/events","assignees_url":"https://api.github.com/repos/projectatomic/atomic-reactor/assignees{/user}","branches_url":"https://api.github.com/repos/projectatomic/atomic-reactor/branches{/branch}","tags_url":"https://api.github.com/repos/projectatomic/atomic-reactor/tags","blobs_url":"https://api.github.com/repos/projectatomic/atomic-reactor/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/projectatomic/atomic-reactor/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/projectatomic/atomic-reactor/git/refs{/sha}","trees_url":"https://api.github.com/repos/projectatomic/atomic-reactor/git/trees{/sha}","statuses_url":"https://api.github.com/repos/projectatomic/atomic-reactor/statuses/{sha}","languages_url":"https://api.github.com/repos/projectatomic/atomic-reactor/languages","stargazers_url":"https://api.github.com/repos/projectatomic/atomic-reactor/stargazers","contributors_url":"https://api.github.com/repos/projectatomic/atomic-reactor/contributors","subscribers_url":"https://api.github.com/repos/projectatomic/atomic-reactor/subscribers","subscription_url":"https://api.github.com/repos/projectatomic/atomic-reactor/subscription","commits_url":"https://api.github.com/repos/projectatomic/atomic-reactor/commits{/sha}","git_commits_url":"https://api.github.com/repos/projectatomic/atomic-reactor/git/commits{/sha}","comments_url":"https://api.github.com/repos/projectatomic/atomic-reactor/comments{/number}","issue_comment_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/comments{/number}","contents_url":"https://api.github.com/repos/projectatomic/atomic-reactor/contents/{+path}","compare_url":"https://api.github.com/repos/projectatomic/atomic-reactor/compare/{base}...{head}","merges_url":"https://api.github.com/repos/projectatomic/atomic-reactor/merges","archive_url":"https://api.github.com/repos/projectatomic/atomic-reactor/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/projectatomic/atomic-reactor/downloads","issues_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues{/number}","pulls_url":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls{/number}","milestones_url":"https://api.github.com/repos/projectatomic/atomic-reactor/milestones{/number}","notifications_url":"https://api.github.com/repos/projectatomic/atomic-reactor/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/projectatomic/atomic-reactor/labels{/name}","releases_url":"https://api.github.com/repos/projectatomic/atomic-reactor/releases{/id}","deployments_url":"https://api.github.com/repos/projectatomic/atomic-reactor/deployments","created_at":"2014-10-21T14:45:11Z","updated_at":"2017-05-26T22:19:41Z","pushed_at":"2017-06-23T17:03:02Z","git_url":"git://github.com/projectatomic/atomic-reactor.git","ssh_url":"git@github.com:projectatomic/atomic-reactor.git","clone_url":"https://github.com/projectatomic/atomic-reactor.git","svn_url":"https://github.com/projectatomic/atomic-reactor","homepage":"","size":1902,"stargazers_count":88,"watchers_count":88,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":35,"mirror_url":null,"open_issues_count":63,"forks":35,"open_issues":63,"watchers":88,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712"},"html":{"href":"https://github.com/projectatomic/atomic-reactor/pull/712"},"issue":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/712"},"comments":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/712/comments"},"review_comments":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712/comments"},"review_comment":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/pulls/712/commits"},"statuses":{"href":"https://api.github.com/repos/projectatomic/atomic-reactor/statuses/c9aef1c11575f0d53240f3d8f204e55a55abdcec"}}}}
{"actor":{"display_login":"lcarva"}}