{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/66897","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/66897/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/66897/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/66897/events","html_url":"https://github.com/kubernetes/kubernetes/pull/66897","id":346908312,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA1NjU0MjM4","number":66897,"title":"Log volume process failure event to pod.","user":{"login":"tsmetana","id":4759699,"node_id":"MDQ6VXNlcjQ3NTk2OTk=","avatar_url":"https://avatars3.githubusercontent.com/u/4759699?v=4","gravatar_id":"","url":"https://api.github.com/users/tsmetana","html_url":"https://github.com/tsmetana","followers_url":"https://api.github.com/users/tsmetana/followers","following_url":"https://api.github.com/users/tsmetana/following{/other_user}","gists_url":"https://api.github.com/users/tsmetana/gists{/gist_id}","starred_url":"https://api.github.com/users/tsmetana/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tsmetana/subscriptions","organizations_url":"https://api.github.com/users/tsmetana/orgs","repos_url":"https://api.github.com/users/tsmetana/repos","events_url":"https://api.github.com/users/tsmetana/events{/privacy}","received_events_url":"https://api.github.com/users/tsmetana/received_events","type":"User","site_admin":false},"labels":[{"id":451459590,"node_id":"MDU6TGFiZWw0NTE0NTk1OTA=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/area/kubeadm","name":"area/kubeadm","color":"0052cc","default":false},{"id":477397086,"node_id":"MDU6TGFiZWw0NzczOTcwODY=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/cncf-cla:%20yes","name":"cncf-cla: yes","color":"bfe5bf","default":false},{"id":267761362,"node_id":"MDU6TGFiZWwyNjc3NjEzNjI=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/kind/feature","name":"kind/feature","color":"c7def8","default":false},{"id":349530249,"node_id":"MDU6TGFiZWwzNDk1MzAyNDk=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/release-note-none","name":"release-note-none","color":"c2e0c6","default":false},{"id":173494222,"node_id":"MDU6TGFiZWwxNzM0OTQyMjI=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/sig/cluster-lifecycle","name":"sig/cluster-lifecycle","color":"d2b48c","default":false},{"id":173493665,"node_id":"MDU6TGFiZWwxNzM0OTM2NjU=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/sig/node","name":"sig/node","color":"d2b48c","default":false},{"id":253450978,"node_id":"MDU6TGFiZWwyNTM0NTA5Nzg=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/L","name":"size/L","color":"ee9900","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-08-02T08:21:38Z","updated_at":"2018-08-07T07:43:34Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/66897","html_url":"https://github.com/kubernetes/kubernetes/pull/66897","diff_url":"https://github.com/kubernetes/kubernetes/pull/66897.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/66897.patch"},"body":"When volume manager or attach/detach controller start to process\r\na volume and encounter errors, there is no error event recorded\r\nin Pod so that user have to wait until attach time out to realize\r\nthat something wrong with the volume. Besides, the time out event\r\ndoes not show the detail error information to the user.\r\n\r\nNeed to log volume process failure event to pod to show user the\r\ndetail volume handling error immediately once the error happens.\r\n\r\nFixes #58272\r\n\r\nNote to reviewers:\r\nThis is only a rebased PR #58273 by @xingzhou -- the original PR looks to be stalled, but we're still interested in the functionality so I'm re-trying to push it through again.\r\n\r\nThere is one aspect I'm still quite unsure about: it adds a new kubelet option to configure the interval to configure the failure event emission interval... I don't know of a better way to do this but adding new kubelet option is really an invasive change. I would prefer to remove that part.\r\n\r\n```release-note\r\nNONE\r\n```"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/410965303","html_url":"https://github.com/kubernetes/kubernetes/pull/66897#issuecomment-410965303","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/66897","id":410965303,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMDk2NTMwMw==","user":{"login":"tsmetana","id":4759699,"node_id":"MDQ6VXNlcjQ3NTk2OTk=","avatar_url":"https://avatars3.githubusercontent.com/u/4759699?v=4","gravatar_id":"","url":"https://api.github.com/users/tsmetana","html_url":"https://github.com/tsmetana","followers_url":"https://api.github.com/users/tsmetana/followers","following_url":"https://api.github.com/users/tsmetana/following{/other_user}","gists_url":"https://api.github.com/users/tsmetana/gists{/gist_id}","starred_url":"https://api.github.com/users/tsmetana/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tsmetana/subscriptions","organizations_url":"https://api.github.com/users/tsmetana/orgs","repos_url":"https://api.github.com/users/tsmetana/repos","events_url":"https://api.github.com/users/tsmetana/events{/privacy}","received_events_url":"https://api.github.com/users/tsmetana/received_events","type":"User","site_admin":false},"created_at":"2018-08-07T07:43:34Z","updated_at":"2018-08-07T07:43:34Z","author_association":"CONTRIBUTOR","body":"@msau42 -- I know you asked for the interval to be configurable... I wonder if it really is necessary. Especially since there is no simple way to do it and API change with new kubelet option seems to be really drastic change to me. Moreover changing the defaults means restarting kubelet which looks to be really inconvenient way to tweak the setting."}}
{"actor":{"display_login":"tsmetana"}}