{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14964","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14964/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14964/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14964/events","html_url":"https://github.com/kubernetes/kubernetes/pull/14964","id":109432754,"number":14964,"title":"Unrevert #14608 and decrease the latency of GCE load balancer deletions","user":{"login":"a-robinson","id":7085343,"avatar_url":"https://avatars.githubusercontent.com/u/7085343?v=3","gravatar_id":"","url":"https://api.github.com/users/a-robinson","html_url":"https://github.com/a-robinson","followers_url":"https://api.github.com/users/a-robinson/followers","following_url":"https://api.github.com/users/a-robinson/following{/other_user}","gists_url":"https://api.github.com/users/a-robinson/gists{/gist_id}","starred_url":"https://api.github.com/users/a-robinson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/a-robinson/subscriptions","organizations_url":"https://api.github.com/users/a-robinson/orgs","repos_url":"https://api.github.com/users/a-robinson/repos","events_url":"https://api.github.com/users/a-robinson/events{/privacy}","received_events_url":"https://api.github.com/users/a-robinson/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/cla:%20yes","name":"cla: yes","color":"bfe5bf"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/L","name":"size/L","color":"ee9900"}],"state":"open","locked":false,"assignee":{"login":"brendandburns","id":5751682,"avatar_url":"https://avatars.githubusercontent.com/u/5751682?v=3","gravatar_id":"","url":"https://api.github.com/users/brendandburns","html_url":"https://github.com/brendandburns","followers_url":"https://api.github.com/users/brendandburns/followers","following_url":"https://api.github.com/users/brendandburns/following{/other_user}","gists_url":"https://api.github.com/users/brendandburns/gists{/gist_id}","starred_url":"https://api.github.com/users/brendandburns/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brendandburns/subscriptions","organizations_url":"https://api.github.com/users/brendandburns/orgs","repos_url":"https://api.github.com/users/brendandburns/repos","events_url":"https://api.github.com/users/brendandburns/events{/privacy}","received_events_url":"https://api.github.com/users/brendandburns/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-10-02T05:23:46Z","updated_at":"2015-10-02T05:42:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/14964","html_url":"https://github.com/kubernetes/kubernetes/pull/14964","diff_url":"https://github.com/kubernetes/kubernetes/pull/14964.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/14964.patch"},"body":"This shouldn't be merged until my many requests for increased GCE static IP quota are filled, but I'm getting it out now so that it's ready once that happens.\r\n\r\nAs discussed in #14597, having load balancers use static IPs led to much greater static IP usage as well as leaks, since the IPs are deleted after everything else (meaning that if a target pool leaks, the IP is guaranteed to be leaked as well).\r\n\r\nIn addition to unreverting the original static IP change, this PR adds a utility function for running goroutines in parallel and coalescing the errors into the existing Aggregate type, then uses that function to delete the load balancer resources with as much parallelism as GCE's API allows. This should make it less likely that a master will still be waiting on resources to be deleted when it's torn down by jenkins.\r\n\r\nWe'd probably also find this pattern useful for decreasing the latency of LB creations, but that's less immediately necessary. If you don't think it's insane, I'll send that out separately after this.\r\n\r\n@thockin @brendandburns @cjcullen "},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/144930858","html_url":"https://github.com/kubernetes/kubernetes/pull/14964#issuecomment-144930858","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14964","id":144930858,"user":{"login":"k8s-merge-robot","id":13653959,"avatar_url":"https://avatars.githubusercontent.com/u/13653959?v=3","gravatar_id":"","url":"https://api.github.com/users/k8s-merge-robot","html_url":"https://github.com/k8s-merge-robot","followers_url":"https://api.github.com/users/k8s-merge-robot/followers","following_url":"https://api.github.com/users/k8s-merge-robot/following{/other_user}","gists_url":"https://api.github.com/users/k8s-merge-robot/gists{/gist_id}","starred_url":"https://api.github.com/users/k8s-merge-robot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/k8s-merge-robot/subscriptions","organizations_url":"https://api.github.com/users/k8s-merge-robot/orgs","repos_url":"https://api.github.com/users/k8s-merge-robot/repos","events_url":"https://api.github.com/users/k8s-merge-robot/events{/privacy}","received_events_url":"https://api.github.com/users/k8s-merge-robot/received_events","type":"User","site_admin":false},"created_at":"2015-10-02T05:42:50Z","updated_at":"2015-10-02T05:42:50Z","body":"Labelling this PR as size/L"}}