{"action":"closed","issue":{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/1860","labels_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/1860/labels{/name}","comments_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/1860/comments","events_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/1860/events","html_url":"https://github.com/GoogleCloudPlatform/kubernetes/issues/1860","id":46121802,"number":1860,"title":"Need better log or handling if newIPAllocator fails","user":{"login":"deads2k","id":8225098,"avatar_url":"https://avatars.githubusercontent.com/u/8225098?v=3","gravatar_id":"","url":"https://api.github.com/users/deads2k","html_url":"https://github.com/deads2k","followers_url":"https://api.github.com/users/deads2k/followers","following_url":"https://api.github.com/users/deads2k/following{/other_user}","gists_url":"https://api.github.com/users/deads2k/gists{/gist_id}","starred_url":"https://api.github.com/users/deads2k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deads2k/subscriptions","organizations_url":"https://api.github.com/users/deads2k/orgs","repos_url":"https://api.github.com/users/deads2k/repos","events_url":"https://api.github.com/users/deads2k/events{/privacy}","received_events_url":"https://api.github.com/users/deads2k/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/labels/area/cloudprovider","name":"area/cloudprovider","color":"f1fd21"},{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/labels/priority/P3","name":"priority/P3","color":"fef2c0"},{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/labels/team/cluster","name":"team/cluster","color":"d2b48c"}],"state":"closed","locked":false,"assignee":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2014-10-17T17:09:08Z","updated_at":"2015-07-09T05:38:00Z","closed_at":"2015-07-09T05:38:00Z","body":"service.newIPAllocator can return null.  If it does, then calls to service.REST.Create result in panics in the log due to nil pointer dereferences.  I would change it, but there's no clear way to return an error through the layers back into master.go.  Here is the stack:\r\n\r\nI1017 13:05:08.641713 17383 handlers.go:37] APIServer panic'd on POST /api/v1beta1/services?labels=: runtime error: invalid memory address or nil pointer dereference\r\n/home/deads/workspaces/openshift3/src/github.com/openshift/origin/Godeps/_workspace/src/github.com/GoogleCloudPlatform/kubernetes/pkg/apiserver/handlers.go:37 (0x6d6317)\r\n\tfunc.002: glog.Infof(\"APIServer panic'd on %v %v: %v\\n%s\\n\", req.Method, req.RequestURI, x, debug.Stack())\r\n/usr/local/go/src/pkg/runtime/panic.c:248 (0x40f48d)\r\n/usr/local/go/src/pkg/runtime/panic.c:552 (0x40ff3d)\r\n/usr/local/go/src/pkg/runtime/os_linux.c:237 (0x40eb2e)\r\n/home/deads/workspaces/openshift3/src/github.com/openshift/origin/Godeps/_workspace/src/github.com/GoogleCloudPlatform/kubernetes/pkg/registry/service/ip_allocator.go:71 (0x6edd9c)\r\n\t(*ipAllocator).AllocateNext: ipa.lock.Lock()\r\n/home/deads/workspaces/openshift3/src/github.com/openshift/origin/Godeps/_workspace/src/github.com/GoogleCloudPlatform/kubernetes/pkg/registry/service/rest.go:93 (0x6eede5)\r\n\t(*REST).Create: if ip, err := rs.portalMgr.AllocateNext(); err != nil {\r\n"}}