{"action":"created","issue":{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/5019","labels_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/5019/labels{/name}","comments_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/5019/comments","events_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/5019/events","html_url":"https://github.com/GoogleCloudPlatform/kubernetes/pull/5019","id":59765558,"number":5019,"title":"kubelet: reject pods on host port conflict","user":{"login":"yujuhong","id":10524058,"avatar_url":"https://avatars.githubusercontent.com/u/10524058?v=3","gravatar_id":"","url":"https://api.github.com/users/yujuhong","html_url":"https://github.com/yujuhong","followers_url":"https://api.github.com/users/yujuhong/followers","following_url":"https://api.github.com/users/yujuhong/following{/other_user}","gists_url":"https://api.github.com/users/yujuhong/gists{/gist_id}","starred_url":"https://api.github.com/users/yujuhong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yujuhong/subscriptions","organizations_url":"https://api.github.com/users/yujuhong/orgs","repos_url":"https://api.github.com/users/yujuhong/repos","events_url":"https://api.github.com/users/yujuhong/events{/privacy}","received_events_url":"https://api.github.com/users/yujuhong/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/labels/cla:%20yes","name":"cla: yes","color":"ECF7EC"}],"state":"open","locked":false,"assignee":{"login":"dchen1107","id":7740897,"avatar_url":"https://avatars.githubusercontent.com/u/7740897?v=3","gravatar_id":"","url":"https://api.github.com/users/dchen1107","html_url":"https://github.com/dchen1107","followers_url":"https://api.github.com/users/dchen1107/followers","following_url":"https://api.github.com/users/dchen1107/following{/other_user}","gists_url":"https://api.github.com/users/dchen1107/gists{/gist_id}","starred_url":"https://api.github.com/users/dchen1107/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dchen1107/subscriptions","organizations_url":"https://api.github.com/users/dchen1107/orgs","repos_url":"https://api.github.com/users/dchen1107/repos","events_url":"https://api.github.com/users/dchen1107/events{/privacy}","received_events_url":"https://api.github.com/users/dchen1107/received_events","type":"User","site_admin":false},"milestone":null,"comments":6,"created_at":"2015-03-04T08:32:31Z","updated_at":"2015-03-05T01:17:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/pulls/5019","html_url":"https://github.com/GoogleCloudPlatform/kubernetes/pull/5019","diff_url":"https://github.com/GoogleCloudPlatform/kubernetes/pull/5019.diff","patch_url":"https://github.com/GoogleCloudPlatform/kubernetes/pull/5019.patch"},"body":"When a host port conflict is detected, kubelet should set the pod status to\r\nfail. The failed status will then be polled by other components at a later time,\r\nwhich allows replication controller to create a new pod if necessary.\r\n\r\nTo achieve this, this change stores the pod status information in a status map\r\nupon the detecton of port conflict. GetPodStatus() consults this status map\r\nbefore attempting to query docker. The entries in the status map will be removed\r\nwhen the pod is no longer associated with the node.\r\n\r\nThis fixes #4623."},"comment":{"url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/comments/77287090","html_url":"https://github.com/GoogleCloudPlatform/kubernetes/pull/5019#issuecomment-77287090","issue_url":"https://api.github.com/repos/GoogleCloudPlatform/kubernetes/issues/5019","id":77287090,"user":{"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},"created_at":"2015-03-05T01:17:08Z","updated_at":"2015-03-05T01:17:08Z","body":"It probably is, I just wanted to keep that consideration in mind \r\n\r\n\r\n> On Mar 4, 2015, at 8:03 PM, Yu-Ju Hong <notifications@github.com> wrote:\r\n> \r\n> Agreed that creationTimestamp doesn't provide a total ordering. The question becomes that whether we need the total ordering to resolve the conflict. Perhaps it's enough to reject ONE pod, irregardless which pod it is?\r\n> \r\n> —\r\n> Reply to this email directly or view it on GitHub.\r\n> \r\n"}}