{"action":"created","issue":{"url":"https://api.github.com/repos/cf-tm-bot/garden/issues/49","repository_url":"https://api.github.com/repos/cf-tm-bot/garden","labels_url":"https://api.github.com/repos/cf-tm-bot/garden/issues/49/labels{/name}","comments_url":"https://api.github.com/repos/cf-tm-bot/garden/issues/49/comments","events_url":"https://api.github.com/repos/cf-tm-bot/garden/issues/49/events","html_url":"https://github.com/cf-tm-bot/garden/issues/49","id":178964610,"number":49,"title":"As a user I want to be able to set memory limits in garden-runc that runs in Xenial - Story Id: 119999729","user":{"login":"cf-tm-bot","id":20690070,"avatar_url":"https://avatars.githubusercontent.com/u/20690070?v=3","gravatar_id":"","url":"https://api.github.com/users/cf-tm-bot","html_url":"https://github.com/cf-tm-bot","followers_url":"https://api.github.com/users/cf-tm-bot/followers","following_url":"https://api.github.com/users/cf-tm-bot/following{/other_user}","gists_url":"https://api.github.com/users/cf-tm-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/cf-tm-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cf-tm-bot/subscriptions","organizations_url":"https://api.github.com/users/cf-tm-bot/orgs","repos_url":"https://api.github.com/users/cf-tm-bot/repos","events_url":"https://api.github.com/users/cf-tm-bot/events{/privacy}","received_events_url":"https://api.github.com/users/cf-tm-bot/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cf-tm-bot/garden/labels/feature","name":"feature","color":"ededed"},{"url":"https://api.github.com/repos/cf-tm-bot/garden/labels/needs-acceptance","name":"needs-acceptance","color":"ededed"},{"url":"https://api.github.com/repos/cf-tm-bot/garden/labels/unscheduled","name":"unscheduled","color":"c8cbd0"},{"url":"https://api.github.com/repos/cf-tm-bot/garden/labels/xenial","name":"xenial","color":"ededed"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-09-23T20:17:31Z","updated_at":"2016-09-23T20:17:47Z","closed_at":null,"body":"Ubuntu 16.04 (Xenial) runs on Linux Kernel 4.4 that is compiled without the `CONFIG_MEMCG_SWAP_ENABLED` build flag. That means that in Xenial's Cgroups memory subsystem there is no `memory.memsw.limit_in_bytes` file, which normally represents a memory limit that includes swap and main memory. In garden-runc when memory limits are set we set the `memsw` limit as well, which in Xenial does not work,\n\n---\n\nMirrors: [story 119999729](https://www.pivotaltracker.com/story/show/119999729) submitted on May 20, 2016 UTC\n- **Requester**: George Lestaris\n- **Estimate**: 0.0"},"comment":{"url":"https://api.github.com/repos/cf-tm-bot/garden/issues/comments/249292022","html_url":"https://github.com/cf-tm-bot/garden/issues/49#issuecomment-249292022","issue_url":"https://api.github.com/repos/cf-tm-bot/garden/issues/49","id":249292022,"user":{"login":"cf-tm-bot","id":20690070,"avatar_url":"https://avatars.githubusercontent.com/u/20690070?v=3","gravatar_id":"","url":"https://api.github.com/users/cf-tm-bot","html_url":"https://github.com/cf-tm-bot","followers_url":"https://api.github.com/users/cf-tm-bot/followers","following_url":"https://api.github.com/users/cf-tm-bot/following{/other_user}","gists_url":"https://api.github.com/users/cf-tm-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/cf-tm-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cf-tm-bot/subscriptions","organizations_url":"https://api.github.com/users/cf-tm-bot/orgs","repos_url":"https://api.github.com/users/cf-tm-bot/repos","events_url":"https://api.github.com/users/cf-tm-bot/events{/privacy}","received_events_url":"https://api.github.com/users/cf-tm-bot/received_events","type":"User","site_admin":false},"created_at":"2016-09-23T20:17:47Z","updated_at":"2016-09-23T20:17:47Z","body":"Julian Friedman [commented](https://www.pivotaltracker.com/n/projects/1158420/stories/119999729/comments/134678951) on May 21, 2016, 7.50 AM UTC:\n\nhad a quick google, looks like this should actually be fine, we just need to pass the `swapaccount=1` parameter to the kernel on boot up (which has the same effect as the compilation flag above). Even better looks like the trusty stemcell builder [already does this](https://github.com/cloudfoundry/bosh/blob/695bed549bd93dd263285b6da08b4c23fe24e98c/bosh-stemcell/spec/stemcells/ubuntu_trusty_spec.rb#L15) which is why it works fine on trusty, guessing when we swapped to the custom kernel in the spike the argument got dropped.\n\n<!--\nComment Id: 134678951\n-->"}}
{"actor":{"display_login":"cf-tm-bot"}}