{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/9520","repository_url":"https://api.github.com/repos/openshift/origin","labels_url":"https://api.github.com/repos/openshift/origin/issues/9520/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/9520/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/9520/events","html_url":"https://github.com/openshift/origin/pull/9520","id":161990139,"number":9520,"title":"Merge openshift-sdn into origin","user":{"login":"danwinship","id":96040,"avatar_url":"https://avatars.githubusercontent.com/u/96040?v=3","gravatar_id":"","url":"https://api.github.com/users/danwinship","html_url":"https://github.com/danwinship","followers_url":"https://api.github.com/users/danwinship/followers","following_url":"https://api.github.com/users/danwinship/following{/other_user}","gists_url":"https://api.github.com/users/danwinship/gists{/gist_id}","starred_url":"https://api.github.com/users/danwinship/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danwinship/subscriptions","organizations_url":"https://api.github.com/users/danwinship/orgs","repos_url":"https://api.github.com/users/danwinship/repos","events_url":"https://api.github.com/users/danwinship/events{/privacy}","received_events_url":"https://api.github.com/users/danwinship/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openshift/origin/labels/needs-rebase","name":"needs-rebase","color":"cccccc"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":22,"created_at":"2016-06-23T18:24:05Z","updated_at":"2016-07-05T15:53:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/9520","html_url":"https://github.com/openshift/origin/pull/9520","diff_url":"https://github.com/openshift/origin/pull/9520.diff","patch_url":"https://github.com/openshift/origin/pull/9520.patch"},"body":"Background: openshift-sdn was originally a standalone program, but that stopped being true a long time ago. These days it can't be compiled on its own, can't be tested on its own, etc, and yet it is still a separate git module for historical reasons. (There was also a period of time when we thought that openshift-sdn was eventually going to go away [and be replaced with flannel], so we didn't want to move it into origin just to later get rid of it. But that's not the plan any more.)\r\n\r\nSo, this gets rid of openshift-sdn as a dependency of origin and moves the code directly into the origin tree instead:\r\n- openshift-sdn/plugins/osdn → origin/pkg/sdn/plugin\r\n- openshift-sdn/pkg/cmd/admin/network → origin/pkg/cmd/admin/network\r\n- openshift-sdn/pkg/{ipcmd,ovs,netutils} → origin/pkg/util/{ipcmd,ovs,netutils}\r\n\r\n(That leaves hack/debug.sh still in the old openshift-sdn module, and it needs to be figured out later, or maybe it will just stay in openshift-sdn until we manage to migrate all the code into the origin diagnostic tool.)\r\n\r\nThe commits just move the code directly from Godeps/ to the new locations, so that the files will at least still keep the history of imports from openshift-sdn (with commit messages that contain the relevant openshift-sdn commit hashes) rather than losing all history. Unfortunately, `hack/verify-upstream-commits.sh` doesn't like us moving things around like this. Maybe we can just ignore it?\r\n\r\n@openshift/networking PTAL. This is built on top of #9476 so ignore that part."},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/230520062","html_url":"https://github.com/openshift/origin/pull/9520#issuecomment-230520062","issue_url":"https://api.github.com/repos/openshift/origin/issues/9520","id":230520062,"user":{"login":"openshift-bot","id":1779249,"avatar_url":"https://avatars.githubusercontent.com/u/1779249?v=3","gravatar_id":"","url":"https://api.github.com/users/openshift-bot","html_url":"https://github.com/openshift-bot","followers_url":"https://api.github.com/users/openshift-bot/followers","following_url":"https://api.github.com/users/openshift-bot/following{/other_user}","gists_url":"https://api.github.com/users/openshift-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/openshift-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openshift-bot/subscriptions","organizations_url":"https://api.github.com/users/openshift-bot/orgs","repos_url":"https://api.github.com/users/openshift-bot/repos","events_url":"https://api.github.com/users/openshift-bot/events{/privacy}","received_events_url":"https://api.github.com/users/openshift-bot/received_events","type":"User","site_admin":false},"created_at":"2016-07-05T15:53:36Z","updated_at":"2016-07-05T15:53:36Z","body":"continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5787/)"}}
{"actor":{"display_login":"openshift-bot"}}