{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14427","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14427/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14427/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14427/events","html_url":"https://github.com/kubernetes/kubernetes/pull/14427","id":107923022,"number":14427,"title":"Avoid assumption of a fixed list of cgroups in e2e conformance tests","user":{"login":"sttts","id":730123,"avatar_url":"https://avatars.githubusercontent.com/u/730123?v=3","gravatar_id":"","url":"https://api.github.com/users/sttts","html_url":"https://github.com/sttts","followers_url":"https://api.github.com/users/sttts/followers","following_url":"https://api.github.com/users/sttts/following{/other_user}","gists_url":"https://api.github.com/users/sttts/gists{/gist_id}","starred_url":"https://api.github.com/users/sttts/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sttts/subscriptions","organizations_url":"https://api.github.com/users/sttts/orgs","repos_url":"https://api.github.com/users/sttts/repos","events_url":"https://api.github.com/users/sttts/events{/privacy}","received_events_url":"https://api.github.com/users/sttts/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/cla:%20yes","name":"cla: yes","color":"bfe5bf"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/lgtm","name":"lgtm","color":"15dd18"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/M","name":"size/M","color":"eebb00"}],"state":"open","locked":false,"assignee":{"login":"quinton-hoole","id":10390785,"avatar_url":"https://avatars.githubusercontent.com/u/10390785?v=3","gravatar_id":"","url":"https://api.github.com/users/quinton-hoole","html_url":"https://github.com/quinton-hoole","followers_url":"https://api.github.com/users/quinton-hoole/followers","following_url":"https://api.github.com/users/quinton-hoole/following{/other_user}","gists_url":"https://api.github.com/users/quinton-hoole/gists{/gist_id}","starred_url":"https://api.github.com/users/quinton-hoole/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quinton-hoole/subscriptions","organizations_url":"https://api.github.com/users/quinton-hoole/orgs","repos_url":"https://api.github.com/users/quinton-hoole/repos","events_url":"https://api.github.com/users/quinton-hoole/events{/privacy}","received_events_url":"https://api.github.com/users/quinton-hoole/received_events","type":"User","site_admin":false},"milestone":null,"comments":12,"created_at":"2015-09-23T13:45:32Z","updated_at":"2015-09-23T18:41:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/14427","html_url":"https://github.com/kubernetes/kubernetes/pull/14427","diff_url":"https://github.com/kubernetes/kubernetes/pull/14427.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/14427.patch"},"body":"As a follow-up of the discussion with @gmarek today and that in https://github.com/mesosphere/kubernetes-mesos/issues/489 this PR reduces the assumptions of e2e tests about the list of cgroups (\"containers\") a kubelet is managing.\r\n\r\n**Problem**\r\n\r\nThe default list has been:\r\n```\r\n\"/\",\r\n\"/docker-daemon\",\r\n\"/kubelet\",\r\n\"/kube-proxy\",\r\n\"/system\",\r\n```\r\n\r\nIn setups other than gce/gke the kubelet will probably run in different cgroups hierarchies. In the case of Mesos the cgroup hierarchy is provided by the Mesos slave. K8s is configured to stay in this hierarchy. Especially the kubelet is not allowed to move any system processes around.\r\n\r\n**Solution**\r\nThe solution in this PR does two things:\r\n- it excludes @gmarek's \"Resource usage of system containers\" test from the conformance test suite\r\n- it reduces the list of containers to `\"/\"` only for provider other than gce and gke in the \"Kubelet monitors resource usage on node\" test.\r\n\r\n**Rational**\r\n\r\nAccording to @gmarek the \"Resource usage of system containers\" test was not meant as a general e2e test for every cluster setup. In addition to the list of cgroups it assumes fixed resource limits and checks these: \"it was supposed to run mainly on our soak cluster to look for regressions\".\r\n\r\nThe \"Kubelet monitors resource usage on node\" test should mainly check that the kubelet is able to monitor and to return proper logs. By reducing this to `\"/\"` only this test stays functional on every cluster.\r\n\r\nFixes half of https://github.com/mesosphere/kubernetes-mesos/issues/489: https://github.com/mesosphere/kubernetes-mesos/issues/436, https://github.com/mesosphere/kubernetes-mesos/issues/439"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/142692637","html_url":"https://github.com/kubernetes/kubernetes/pull/14427#issuecomment-142692637","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/14427","id":142692637,"user":{"login":"sttts","id":730123,"avatar_url":"https://avatars.githubusercontent.com/u/730123?v=3","gravatar_id":"","url":"https://api.github.com/users/sttts","html_url":"https://github.com/sttts","followers_url":"https://api.github.com/users/sttts/followers","following_url":"https://api.github.com/users/sttts/following{/other_user}","gists_url":"https://api.github.com/users/sttts/gists{/gist_id}","starred_url":"https://api.github.com/users/sttts/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sttts/subscriptions","organizations_url":"https://api.github.com/users/sttts/orgs","repos_url":"https://api.github.com/users/sttts/repos","events_url":"https://api.github.com/users/sttts/events{/privacy}","received_events_url":"https://api.github.com/users/sttts/received_events","type":"User","site_admin":false},"created_at":"2015-09-23T18:41:06Z","updated_at":"2015-09-23T18:41:06Z","body":"Evil auto responders."}}