{"action":"created","issue":{"url":"https://api.github.com/repos/juju/juju/issues/5522","repository_url":"https://api.github.com/repos/juju/juju","labels_url":"https://api.github.com/repos/juju/juju/issues/5522/labels{/name}","comments_url":"https://api.github.com/repos/juju/juju/issues/5522/comments","events_url":"https://api.github.com/repos/juju/juju/issues/5522/events","html_url":"https://github.com/juju/juju/pull/5522","id":158204149,"number":5522,"title":"provider/maas: remove retry logic from bridge script","user":{"login":"frobware","id":943903,"avatar_url":"https://avatars.githubusercontent.com/u/943903?v=3","gravatar_id":"","url":"https://api.github.com/users/frobware","html_url":"https://github.com/frobware","followers_url":"https://api.github.com/users/frobware/followers","following_url":"https://api.github.com/users/frobware/following{/other_user}","gists_url":"https://api.github.com/users/frobware/gists{/gist_id}","starred_url":"https://api.github.com/users/frobware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/frobware/subscriptions","organizations_url":"https://api.github.com/users/frobware/orgs","repos_url":"https://api.github.com/users/frobware/repos","events_url":"https://api.github.com/users/frobware/events{/privacy}","received_events_url":"https://api.github.com/users/frobware/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-06-02T18:08:00Z","updated_at":"2016-06-14T09:30:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/juju/juju/pulls/5522","html_url":"https://github.com/juju/juju/pull/5522","diff_url":"https://github.com/juju/juju/pull/5522.diff","patch_url":"https://github.com/juju/juju/pull/5522.patch"},"body":"Remove the retry logic on various commands that have been introduced\r\nrecently for migrating the IP addresss and the routes from the parent\r\ndevice to the bridge device.\r\n\r\nI was seeing a lot of:\r\n\r\n  RTNETLINK: file exists\r\n\r\nwhen boostrapping.\r\n\r\nRemoving addresses from the parent is also done using a single call:\r\n\r\n  $ ip addr flush dev eth0\r\n\r\nand we now only run:\r\n\r\n  $ ip route replace {route} dev {bridge}\r\n\r\nif there is a 'default' route for the {parent}.\r\n\r\nPreviously we were doing 'ip route replace' for each bridge and it was\r\nthat command that was generating the 'RTNETLINK: file exists' messages\r\nbecause the following (which we issue) automatically adds a route for\r\nthe device:\r\n\r\n  $ ip addr add dev {bridge} {address}\r\n\r\nFixes [LP:#1588446](https://bugs.launchpad.net/juju-core/+bug/1588446)\n\n(Review request: http://reviews.vapour.ws/r/4969/)"},"comment":{"url":"https://api.github.com/repos/juju/juju/issues/comments/225828418","html_url":"https://github.com/juju/juju/pull/5522#issuecomment-225828418","issue_url":"https://api.github.com/repos/juju/juju/issues/5522","id":225828418,"user":{"login":"frobware","id":943903,"avatar_url":"https://avatars.githubusercontent.com/u/943903?v=3","gravatar_id":"","url":"https://api.github.com/users/frobware","html_url":"https://github.com/frobware","followers_url":"https://api.github.com/users/frobware/followers","following_url":"https://api.github.com/users/frobware/following{/other_user}","gists_url":"https://api.github.com/users/frobware/gists{/gist_id}","starred_url":"https://api.github.com/users/frobware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/frobware/subscriptions","organizations_url":"https://api.github.com/users/frobware/orgs","repos_url":"https://api.github.com/users/frobware/repos","events_url":"https://api.github.com/users/frobware/events{/privacy}","received_events_url":"https://api.github.com/users/frobware/received_events","type":"User","site_admin":false},"created_at":"2016-06-14T09:30:28Z","updated_at":"2016-06-14T09:30:28Z","body":"$$fixes-1588446$$"}}
{"actor":{"display_login":"frobware"}}