{"action":"created","issue":{"url":"https://api.github.com/repos/Azure/acs-engine/issues/3618","repository_url":"https://api.github.com/repos/Azure/acs-engine","labels_url":"https://api.github.com/repos/Azure/acs-engine/issues/3618/labels{/name}","comments_url":"https://api.github.com/repos/Azure/acs-engine/issues/3618/comments","events_url":"https://api.github.com/repos/Azure/acs-engine/issues/3618/events","html_url":"https://github.com/Azure/acs-engine/pull/3618","id":348051422,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA2NDkzMzE4","number":3618,"title":"Fix cni cleanup during kubelet start to be graceful & not loose any logs","user":{"login":"madhanrm","id":20309044,"node_id":"MDQ6VXNlcjIwMzA5MDQ0","avatar_url":"https://avatars1.githubusercontent.com/u/20309044?v=4","gravatar_id":"","url":"https://api.github.com/users/madhanrm","html_url":"https://github.com/madhanrm","followers_url":"https://api.github.com/users/madhanrm/followers","following_url":"https://api.github.com/users/madhanrm/following{/other_user}","gists_url":"https://api.github.com/users/madhanrm/gists{/gist_id}","starred_url":"https://api.github.com/users/madhanrm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/madhanrm/subscriptions","organizations_url":"https://api.github.com/users/madhanrm/orgs","repos_url":"https://api.github.com/users/madhanrm/repos","events_url":"https://api.github.com/users/madhanrm/events{/privacy}","received_events_url":"https://api.github.com/users/madhanrm/received_events","type":"User","site_admin":false},"labels":[{"id":936711751,"node_id":"MDU6TGFiZWw5MzY3MTE3NTE=","url":"https://api.github.com/repos/Azure/acs-engine/labels/size/S","name":"size/S","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-08-06T19:24:36Z","updated_at":"2018-08-06T21:21:14Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/Azure/acs-engine/pulls/3618","html_url":"https://github.com/Azure/acs-engine/pull/3618","diff_url":"https://github.com/Azure/acs-engine/pull/3618.diff","patch_url":"https://github.com/Azure/acs-engine/pull/3618.patch"},"body":"<!--  Thanks for sending a pull request!  Here are some tips for you:\r\n1. If you want *faster* PR reviews, read how: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#best-practices-for-faster-reviews\r\n-->\r\n\r\n**What this PR does / why we need it**:\r\n* Save the CNI Logs\r\n* Gracefully shutdown CNI instances & cleanup CNI json & locks if any\r\n\r\n**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #\r\n\r\n**Special notes for your reviewer**:\r\n\r\n**If applicable**:\r\n- [ ] documentation\r\n- [ ] unit tests\r\n- [ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)\r\n\r\n**Release note**:\r\n<!--  Steps to write your release note:\r\n1. Use the release-note-* labels to set the release note state (if you have access)\r\n2. Enter your extended release note in the below block; leaving it blank means using the PR title as the release note. If no release note is required, just write `NONE`.\r\n-->\r\n```release-note\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/Azure/acs-engine/issues/comments/410856836","html_url":"https://github.com/Azure/acs-engine/pull/3618#issuecomment-410856836","issue_url":"https://api.github.com/repos/Azure/acs-engine/issues/3618","id":410856836,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMDg1NjgzNg==","user":{"login":"PatrickLang","id":11345431,"node_id":"MDQ6VXNlcjExMzQ1NDMx","avatar_url":"https://avatars1.githubusercontent.com/u/11345431?v=4","gravatar_id":"","url":"https://api.github.com/users/PatrickLang","html_url":"https://github.com/PatrickLang","followers_url":"https://api.github.com/users/PatrickLang/followers","following_url":"https://api.github.com/users/PatrickLang/following{/other_user}","gists_url":"https://api.github.com/users/PatrickLang/gists{/gist_id}","starred_url":"https://api.github.com/users/PatrickLang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PatrickLang/subscriptions","organizations_url":"https://api.github.com/users/PatrickLang/orgs","repos_url":"https://api.github.com/users/PatrickLang/repos","events_url":"https://api.github.com/users/PatrickLang/events{/privacy}","received_events_url":"https://api.github.com/users/PatrickLang/received_events","type":"User","site_admin":false},"created_at":"2018-08-06T21:21:14Z","updated_at":"2018-08-06T21:21:14Z","author_association":"MEMBER","body":"Talked to @dineshgovindasamy on this. Looks good. Need to check test results before merge"}}
{"actor":{"display_login":"PatrickLang"}}