{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/24440","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/24440/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/24440/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/24440/events","html_url":"https://github.com/kubernetes/kubernetes/issues/24440","id":149307858,"number":24440,"title":"RFC: Deprecate cmd/integration/integration.go","user":{"login":"yujuhong","id":10524058,"avatar_url":"https://avatars.githubusercontent.com/u/10524058?v=3","gravatar_id":"","url":"https://api.github.com/users/yujuhong","html_url":"https://github.com/yujuhong","followers_url":"https://api.github.com/users/yujuhong/followers","following_url":"https://api.github.com/users/yujuhong/following{/other_user}","gists_url":"https://api.github.com/users/yujuhong/gists{/gist_id}","starred_url":"https://api.github.com/users/yujuhong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yujuhong/subscriptions","organizations_url":"https://api.github.com/users/yujuhong/orgs","repos_url":"https://api.github.com/users/yujuhong/repos","events_url":"https://api.github.com/users/yujuhong/events{/privacy}","received_events_url":"https://api.github.com/users/yujuhong/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/area/test","name":"area/test","color":"fad8c7"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/team/test-infra","name":"team/test-infra","color":"d2b48c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2016-04-19T00:14:04Z","updated_at":"2016-05-20T04:07:30Z","closed_at":null,"body":"The integration test is hard to debug -- it runs tests concurrently with interspersed logs. I doubt that it's catching any bug that neither unit test nor a e2e test can catch, and both unit and e2e tests are easier to debug.\r\nI'd vote for deprecating this file and remove/move all the tests. Any objections?\r\n\r\nHere is a list of tests that it curretnly runs: \r\nhttps://github.com/kubernetes/kubernetes/blob/f0c33d65b6fea84d791059d96c93a15e4b3de4ec/cmd/integration/integration.go#L1022\r\n```\r\n\t\trunReplicationControllerTest,\r\n\t\trunAtomicPutTest,\r\n\t\trunPatchTest,\r\n\t\trunServiceTest,\r\n\t\trunAPIVersionsTest,\r\n\t\trunMasterServiceTest,\r\n\t\tfunc(c *client.Client) {\r\n\t\t\trunSelfLinkTestOnNamespace(c, api.NamespaceDefault)\r\n\t\t\trunSelfLinkTestOnNamespace(c, \"other\")\r\n\t\t},\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/220512721","html_url":"https://github.com/kubernetes/kubernetes/issues/24440#issuecomment-220512721","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/24440","id":220512721,"user":{"login":"bprashanth","id":10927820,"avatar_url":"https://avatars.githubusercontent.com/u/10927820?v=3","gravatar_id":"","url":"https://api.github.com/users/bprashanth","html_url":"https://github.com/bprashanth","followers_url":"https://api.github.com/users/bprashanth/followers","following_url":"https://api.github.com/users/bprashanth/following{/other_user}","gists_url":"https://api.github.com/users/bprashanth/gists{/gist_id}","starred_url":"https://api.github.com/users/bprashanth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bprashanth/subscriptions","organizations_url":"https://api.github.com/users/bprashanth/orgs","repos_url":"https://api.github.com/users/bprashanth/repos","events_url":"https://api.github.com/users/bprashanth/events{/privacy}","received_events_url":"https://api.github.com/users/bprashanth/received_events","type":"User","site_admin":false},"created_at":"2016-05-20T04:07:30Z","updated_at":"2016-05-20T04:07:30Z","body":"This issue describes the circumstances leading up to that scheduler integration test, for anyone attempting to port it: https://github.com/kubernetes/kubernetes/issues/6023. Though IIRC we got rid of the modeler, so things might be different now."}}
{"actor":{"display_login":"bprashanth"}}