{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/community/issues/172","repository_url":"https://api.github.com/repos/kubernetes/community","labels_url":"https://api.github.com/repos/kubernetes/community/issues/172/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/community/issues/172/comments","events_url":"https://api.github.com/repos/kubernetes/community/issues/172/events","html_url":"https://github.com/kubernetes/community/issues/172","id":195339807,"number":172,"title":"Proposal: Move kubedns addon code to its own repository under Kubernetes","user":{"login":"bowei","id":837034,"avatar_url":"https://avatars.githubusercontent.com/u/837034?v=3","gravatar_id":"","url":"https://api.github.com/users/bowei","html_url":"https://github.com/bowei","followers_url":"https://api.github.com/users/bowei/followers","following_url":"https://api.github.com/users/bowei/following{/other_user}","gists_url":"https://api.github.com/users/bowei/gists{/gist_id}","starred_url":"https://api.github.com/users/bowei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bowei/subscriptions","organizations_url":"https://api.github.com/users/bowei/orgs","repos_url":"https://api.github.com/users/bowei/repos","events_url":"https://api.github.com/users/bowei/events{/privacy}","received_events_url":"https://api.github.com/users/bowei/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-12-13T19:05:05Z","updated_at":"2016-12-15T21:23:40Z","closed_at":"2016-12-15T21:23:40Z","body":"# Overview\r\nThe kube-dns pod currently consists of the `kube-dns` daemon, `dnsmasq` and their healthcheck sidecars (`exec-healthz` and `dnsmasq-metrics`). The code is spread over both main `kubernetes/` and `contrib/` repositories. Images for each of the containers are currently versioned separately. For example, the `kube-dns` that ships with Kubernetes v1.5 is v1.9, `dnsmasq-metrics` v1.0 etc. \r\n\r\nNote: exec-healthz is being removed from the pod shortly.\r\n\r\nIt would be more coherent to move all of the related code into the same respository. `kube-dns` consumes only public APIs and the release of all Kubernetes maintained code can be unified.\r\n\r\n# Plan of action \r\n* Create a new repository in kubernetes called `dns`\r\n* Copy code (keeping history) of `kubernetes/pkg/dns` to `dns/pkg/dns`\r\n* Copy code (keeping history) of `kubernetes/cmd/kube-dns` to `dns/cmd/kube-dns`\r\n* Copy code (keeping history) of `contrib/dnsmasq-metrics` to `dns/cmd/kube-dns-sidecar`, adjusting `pkg` and `cmd` appropriately\r\n* Copy code (keeping history) `contrib/dnsmasq` to `dns/dnsmasq`\r\n* Use the [standard build template](https://github.com/thockin/go-build-template) for the verification and build infrastructure\r\n* e2e testing of DNS will remain in the Kubernetes repo. I would defer any potential movement of e2e tests until the [separation of e2e from the main repo](https://github.com/kubernetes/kubernetes/issues/36624) has been concretized.\r\n* `kubernetes/cluster/addons` will remain for now.\r\n* Add hook to run e2e against current kube-dns build. Note: this is not done today in the kubernetes repo itself.\r\n\r\n# Impacts\r\n* Kubernetes cluster/addon will refer to externally built image. This is already the case, as the CI system does not pick up changes to the kube-dns code. The referenced image on gcr.io is used instead.\r\n* kube-dns is a leaf project, output of `git grep 'k8s.io/kubernetes/\\(cmd\\|pkg\\)/dns/' | grep -v '^\\(pkg/dns\\|cmd/kube-dns\\)'` shows only one e2e test as an external dependency which will be removed."},"comment":{"url":"https://api.github.com/repos/kubernetes/community/issues/comments/267447922","html_url":"https://github.com/kubernetes/community/issues/172#issuecomment-267447922","issue_url":"https://api.github.com/repos/kubernetes/community/issues/172","id":267447922,"user":{"login":"thockin","id":5595220,"avatar_url":"https://avatars.githubusercontent.com/u/5595220?v=3","gravatar_id":"","url":"https://api.github.com/users/thockin","html_url":"https://github.com/thockin","followers_url":"https://api.github.com/users/thockin/followers","following_url":"https://api.github.com/users/thockin/following{/other_user}","gists_url":"https://api.github.com/users/thockin/gists{/gist_id}","starred_url":"https://api.github.com/users/thockin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thockin/subscriptions","organizations_url":"https://api.github.com/users/thockin/orgs","repos_url":"https://api.github.com/users/thockin/repos","events_url":"https://api.github.com/users/thockin/events{/privacy}","received_events_url":"https://api.github.com/users/thockin/received_events","type":"User","site_admin":false},"created_at":"2016-12-15T21:23:40Z","updated_at":"2016-12-15T21:23:40Z","body":"Repo created."}}
{"actor":{"display_login":"thockin"}}