{"action":"created","issue":{"url":"https://api.github.com/repos/apache/cloudstack/issues/1514","repository_url":"https://api.github.com/repos/apache/cloudstack","labels_url":"https://api.github.com/repos/apache/cloudstack/issues/1514/labels{/name}","comments_url":"https://api.github.com/repos/apache/cloudstack/issues/1514/comments","events_url":"https://api.github.com/repos/apache/cloudstack/issues/1514/events","html_url":"https://github.com/apache/cloudstack/pull/1514","id":150394055,"number":1514,"title":"CLOUDSTACK-6975: Prevent dnsmasq from starting on backup redundant RvR","user":{"login":"dsclose","id":103378,"avatar_url":"https://avatars.githubusercontent.com/u/103378?v=3","gravatar_id":"","url":"https://api.github.com/users/dsclose","html_url":"https://github.com/dsclose","followers_url":"https://api.github.com/users/dsclose/followers","following_url":"https://api.github.com/users/dsclose/following{/other_user}","gists_url":"https://api.github.com/users/dsclose/gists{/gist_id}","starred_url":"https://api.github.com/users/dsclose/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dsclose/subscriptions","organizations_url":"https://api.github.com/users/dsclose/orgs","repos_url":"https://api.github.com/users/dsclose/repos","events_url":"https://api.github.com/users/dsclose/events{/privacy}","received_events_url":"https://api.github.com/users/dsclose/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":21,"created_at":"2016-04-22T15:03:17Z","updated_at":"2016-05-23T12:17:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/cloudstack/pulls/1514","html_url":"https://github.com/apache/cloudstack/pull/1514","diff_url":"https://github.com/apache/cloudstack/pull/1514.diff","patch_url":"https://github.com/apache/cloudstack/pull/1514.patch"},"body":"Rebase of PR #1509 against the 4.7 branch as requested by @swill \r\n\r\nOne LGTM from @ustcweizhou carried from previous PR. Previous PR will be closed.\r\n\r\nDescription from PR #1509:\r\n\r\nCLOUDSTACK-6975 refers to service monitoring bringing up dnsmasq but this is no-longer accurate, as service monitoring is not active on the post-4.6 routers. These routers still suffer an essentially identical issue, however, because \"dnsmasq needs to be restarted each time configure.py is called in order to avoid lease problems.\" As such, dnsmasq is still running on backup RvRs, causing the issues described in CLOUDSTACK-6975.\r\n\r\nThis PR is based on a patch submitted by @ustcweizhou. The code now checks the redundant state of the router before restarting dnsmasq.\r\n\r\nRvR networks without this patch have dnsmasq running on both master and backup routers. RvR networks with this patch have dnsmasq running on only the master router."},"comment":{"url":"https://api.github.com/repos/apache/cloudstack/issues/comments/220964156","html_url":"https://github.com/apache/cloudstack/pull/1514#issuecomment-220964156","issue_url":"https://api.github.com/repos/apache/cloudstack/issues/1514","id":220964156,"user":{"login":"dsclose","id":103378,"avatar_url":"https://avatars.githubusercontent.com/u/103378?v=3","gravatar_id":"","url":"https://api.github.com/users/dsclose","html_url":"https://github.com/dsclose","followers_url":"https://api.github.com/users/dsclose/followers","following_url":"https://api.github.com/users/dsclose/following{/other_user}","gists_url":"https://api.github.com/users/dsclose/gists{/gist_id}","starred_url":"https://api.github.com/users/dsclose/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dsclose/subscriptions","organizations_url":"https://api.github.com/users/dsclose/orgs","repos_url":"https://api.github.com/users/dsclose/repos","events_url":"https://api.github.com/users/dsclose/events{/privacy}","received_events_url":"https://api.github.com/users/dsclose/received_events","type":"User","site_admin":false},"created_at":"2016-05-23T12:17:27Z","updated_at":"2016-05-23T12:17:27Z","body":"@luhaijiao I've seen the same error in our XenServer setup but wasn't sure what the cause is. I'd be surprised if this was the cause but it's worth exploring.\r\n\r\nI will try to boot some system VMs now without this patch to see if the password server starts properly and then apply the patch to see if it subsequently fails. I'll report back here shortly."}}
{"actor":{"display_login":"dsclose"}}