{"action":"created","issue":{"url":"https://api.github.com/repos/juju-solutions/kubernetes/issues/65","repository_url":"https://api.github.com/repos/juju-solutions/kubernetes","labels_url":"https://api.github.com/repos/juju-solutions/kubernetes/issues/65/labels{/name}","comments_url":"https://api.github.com/repos/juju-solutions/kubernetes/issues/65/comments","events_url":"https://api.github.com/repos/juju-solutions/kubernetes/issues/65/events","html_url":"https://github.com/juju-solutions/kubernetes/pull/65","id":184577498,"number":65,"title":"Add heapster, grafana, influxdb","user":{"login":"Cynerva","id":2251600,"avatar_url":"https://avatars.githubusercontent.com/u/2251600?v=3","gravatar_id":"","url":"https://api.github.com/users/Cynerva","html_url":"https://github.com/Cynerva","followers_url":"https://api.github.com/users/Cynerva/followers","following_url":"https://api.github.com/users/Cynerva/following{/other_user}","gists_url":"https://api.github.com/users/Cynerva/gists{/gist_id}","starred_url":"https://api.github.com/users/Cynerva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cynerva/subscriptions","organizations_url":"https://api.github.com/users/Cynerva/orgs","repos_url":"https://api.github.com/users/Cynerva/repos","events_url":"https://api.github.com/users/Cynerva/events{/privacy}","received_events_url":"https://api.github.com/users/Cynerva/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-21T21:49:19Z","updated_at":"2016-10-24T14:25:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/juju-solutions/kubernetes/pulls/65","html_url":"https://github.com/juju-solutions/kubernetes/pull/65","diff_url":"https://github.com/juju-solutions/kubernetes/pull/65.diff","patch_url":"https://github.com/juju-solutions/kubernetes/pull/65.patch"},"body":"WIP, do not merge.\r\n\r\nThis needs a lot of testing. I'll add more details to the PR on Monday."},"comment":{"url":"https://api.github.com/repos/juju-solutions/kubernetes/issues/comments/255755365","html_url":"https://github.com/juju-solutions/kubernetes/pull/65#issuecomment-255755365","issue_url":"https://api.github.com/repos/juju-solutions/kubernetes/issues/65","id":255755365,"user":{"login":"Cynerva","id":2251600,"avatar_url":"https://avatars.githubusercontent.com/u/2251600?v=3","gravatar_id":"","url":"https://api.github.com/users/Cynerva","html_url":"https://github.com/Cynerva","followers_url":"https://api.github.com/users/Cynerva/followers","following_url":"https://api.github.com/users/Cynerva/following{/other_user}","gists_url":"https://api.github.com/users/Cynerva/gists{/gist_id}","starred_url":"https://api.github.com/users/Cynerva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cynerva/subscriptions","organizations_url":"https://api.github.com/users/Cynerva/orgs","repos_url":"https://api.github.com/users/Cynerva/repos","events_url":"https://api.github.com/users/Cynerva/events{/privacy}","received_events_url":"https://api.github.com/users/Cynerva/received_events","type":"User","site_admin":false},"created_at":"2016-10-24T14:25:23Z","updated_at":"2016-10-24T14:25:23Z","body":"@chuckbutler This PR adds heapster, grafana, and influxdb to the kubernetes deployment when dashboard=true.\r\n\r\nThere are two things I want to highlight here.\r\n\r\n#### 1. We grabbed definitions from the heapster repo instead of the kubernetes one.\r\n\r\nWe did this because the [heapster definitions](https://github.com/kubernetes/heapster/tree/master/deploy/kube-config/influxdb) looked newer and easier to work with. The [kubernetes ones](https://github.com/kubernetes/kubernetes/tree/master/cluster/addons/cluster-monitoring/influxdb) require rendering some kind of template and there's no supporting build system for it that I can see.\r\n\r\n@mbruzek We will need to do further research on making addon updates less of a manual process. Right now it looks like we're gonna have to do some upstream work to make it doable. Sorry we didn't make any headway on that here.\r\n\r\n#### 2. We changed the way kubelet is initially brought up\r\n\r\nWe removed the [step where kubelet is brought up with no DNS config](https://github.com/juju-solutions/kubernetes/pull/65/files#diff-bdfd6f6b79f7f3171e9f0167164a7c98L138).\r\n\r\nIn order to make this work, we had to [send DNS info before we deploy kube-dns](https://github.com/juju-solutions/kubernetes/pull/65/files#diff-e6f5b987d0e976d7b76acd341abc74c0L157).\r\n\r\nThe reason we did this was to allow an early deploy of Heapster and friends. Without this change, they came up with missing DNS config and were unable to communicate with eachother."}}
{"actor":{"display_login":"Cynerva"}}