{"action":"created","issue":{"url":"https://api.github.com/repos/juju/juju/issues/2281","labels_url":"https://api.github.com/repos/juju/juju/issues/2281/labels{/name}","comments_url":"https://api.github.com/repos/juju/juju/issues/2281/comments","events_url":"https://api.github.com/repos/juju/juju/issues/2281/events","html_url":"https://github.com/juju/juju/pull/2281","id":75263546,"number":2281,"title":"Juju LXC instances are routable after reboot","user":{"login":"dooferlad","id":1949960,"avatar_url":"https://avatars.githubusercontent.com/u/1949960?v=3","gravatar_id":"","url":"https://api.github.com/users/dooferlad","html_url":"https://github.com/dooferlad","followers_url":"https://api.github.com/users/dooferlad/followers","following_url":"https://api.github.com/users/dooferlad/following{/other_user}","gists_url":"https://api.github.com/users/dooferlad/gists{/gist_id}","starred_url":"https://api.github.com/users/dooferlad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dooferlad/subscriptions","organizations_url":"https://api.github.com/users/dooferlad/orgs","repos_url":"https://api.github.com/users/dooferlad/repos","events_url":"https://api.github.com/users/dooferlad/events{/privacy}","received_events_url":"https://api.github.com/users/dooferlad/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-11T15:38:59Z","updated_at":"2015-05-12T11:26:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/juju/juju/pulls/2281","html_url":"https://github.com/juju/juju/pull/2281","diff_url":"https://github.com/juju/juju/pull/2281.diff","patch_url":"https://github.com/juju/juju/pull/2281.patch"},"body":"This is a fix for https://bugs.launchpad.net/juju-core/+bug/1442012\r\n\r\nThis branch adds a maintenance step that is run on daemon start up for each instance that has already been provisioned. For all instance types other than LXC this does nothing.\r\n\r\nIf jujud finds an LXC instance it will run through the same network provisioning steps as it did when first creating the machine. This already checked to see if a route or IP Tables rule existed before adding them, so there is no harm in running it again, even if the network is set up correctly already. What we have found so far is that routes are not saved across reboots of host machines. We could have solved this by writing a route save and restore init task, but it is difficult to find if such a task already exists. This way we keep all Juju related network configuration inside Juju.\r\n\r\nThis has been live tested on MAAS by:\r\njuju bootstrap -e maas --upload-tools\r\njuju add-machine -e maas lxc:0\r\n\r\nwatch juju status # until 0/lxc/0 is in state running\r\njuju ssh 0/lxc/0 \"echo hello world\"\r\n\r\njuju ssh 0 \"sudo reboot\"\r\nwatch juju status # until 0/lxc/0 is in state running\r\njuju ssh 0/lxc/0 \"echo hello world\"\r\n\r\nRepeat with more containers on the same host.\n\n(Review request: http://reviews.vapour.ws/r/1642/)"},"comment":{"url":"https://api.github.com/repos/juju/juju/issues/comments/101245209","html_url":"https://github.com/juju/juju/pull/2281#issuecomment-101245209","issue_url":"https://api.github.com/repos/juju/juju/issues/2281","id":101245209,"user":{"login":"jujubot","id":7779494,"avatar_url":"https://avatars.githubusercontent.com/u/7779494?v=3","gravatar_id":"","url":"https://api.github.com/users/jujubot","html_url":"https://github.com/jujubot","followers_url":"https://api.github.com/users/jujubot/followers","following_url":"https://api.github.com/users/jujubot/following{/other_user}","gists_url":"https://api.github.com/users/jujubot/gists{/gist_id}","starred_url":"https://api.github.com/users/jujubot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jujubot/subscriptions","organizations_url":"https://api.github.com/users/jujubot/orgs","repos_url":"https://api.github.com/users/jujubot/repos","events_url":"https://api.github.com/users/jujubot/events{/privacy}","received_events_url":"https://api.github.com/users/jujubot/received_events","type":"User","site_admin":false},"created_at":"2015-05-12T11:26:44Z","updated_at":"2015-05-12T11:26:44Z","body":"Build failed: Does not match ['fixes-1453805']\n        build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3199\n"}}