{"action":"created","issue":{"url":"https://api.github.com/repos/xaque208/puppet-unbound/issues/95","labels_url":"https://api.github.com/repos/xaque208/puppet-unbound/issues/95/labels{/name}","comments_url":"https://api.github.com/repos/xaque208/puppet-unbound/issues/95/comments","events_url":"https://api.github.com/repos/xaque208/puppet-unbound/issues/95/events","html_url":"https://github.com/xaque208/puppet-unbound/pull/95","id":92878676,"number":95,"title":"Update control functionality","user":{"login":"b4ldr","id":1110206,"avatar_url":"https://avatars.githubusercontent.com/u/1110206?v=3","gravatar_id":"","url":"https://api.github.com/users/b4ldr","html_url":"https://github.com/b4ldr","followers_url":"https://api.github.com/users/b4ldr/followers","following_url":"https://api.github.com/users/b4ldr/following{/other_user}","gists_url":"https://api.github.com/users/b4ldr/gists{/gist_id}","starred_url":"https://api.github.com/users/b4ldr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/b4ldr/subscriptions","organizations_url":"https://api.github.com/users/b4ldr/orgs","repos_url":"https://api.github.com/users/b4ldr/repos","events_url":"https://api.github.com/users/b4ldr/events{/privacy}","received_events_url":"https://api.github.com/users/b4ldr/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-03T13:39:15Z","updated_at":"2015-07-06T11:21:31Z","closed_at":"2015-07-06T11:21:31Z","pull_request":{"url":"https://api.github.com/repos/xaque208/puppet-unbound/pulls/95","html_url":"https://github.com/xaque208/puppet-unbound/pull/95","diff_url":"https://github.com/xaque208/puppet-unbound/pull/95.diff","patch_url":"https://github.com/xaque208/puppet-unbound/pull/95.patch"},"body":"The last update broke configurations that dont use the unbuond::remote class as control-enable was disabled resulting in the following error\r\n\r\n```\r\nError: /Stage[main]/Unbound/Service[unbound]: Failed to call refresh: Could not restart Service[unbound]: Execution of 'unbound-control reload' returned 1: [1435927171] unbound-control[7677:0] warning: control-enable is 'no' in the config file.\r\n[1435927171] unbound-control[7677:0] error: connect: Connection refused for 127.0.0.1\r\nError: /Stage[main]/Unbound/Service[unbound]: Could not restart Service[unbound]: Execution of 'unbound-control reload' returned 1: [1435927171] unbound-control[7677:0] warning: control-enable is 'no' in the config file.\r\n[1435927171] unbound-control[7677:0] error: connect: Connection refused for 127.0.0.1\r\nWrapped exception:\r\nExecution of 'unbound-control reload' returned 1: [1435927171] unbound-control[7677:0] warning: control-enable is 'no' in the config file.\r\n[1435927171] unbound-control[7677:0] error: connect: Connection refused for 127.0.0.1\r\n```\r\n\r\nThis attach allows for both configurations.  An error is still created when one switches from control-enable false -> true.  This happens because the catalog has control enable set to true, therefore the service reoload is set to use unbound-control however unbuond has not been restarted so it is not listening on the control port.  This results in the following error\r\n\r\n```\r\nError: /Stage[main]/Unbound/Service[unbound]: Failed to call refresh: Could not restart Service[unbound]: Execution of 'unbound-control reload' returned 1: [1435929699] unbound-control[18018:0] error: connect: Connection refused for 127.0.0.1\r\nError: /Stage[main]/Unbound/Service[unbound]: Could not restart Service[unbound]: Execution of 'unbound-control reload' returned 1: [1435929699] unbound-control[18018:0] error: connect: Connection refused for 127.0.0.1\r\nWrapped exception:\r\nExecution of 'unbound-control reload' returned 1: [1435929699] unbound-control[18018:0] error: connect: Connection refused for 127.0.0.1\r\n```\r\n\r\nFixing this last error is somewhat chicken egg situation and any fix i could think of looked pretty messy "},"comment":{"url":"https://api.github.com/repos/xaque208/puppet-unbound/issues/comments/118823384","html_url":"https://github.com/xaque208/puppet-unbound/pull/95#issuecomment-118823384","issue_url":"https://api.github.com/repos/xaque208/puppet-unbound/issues/95","id":118823384,"user":{"login":"b4ldr","id":1110206,"avatar_url":"https://avatars.githubusercontent.com/u/1110206?v=3","gravatar_id":"","url":"https://api.github.com/users/b4ldr","html_url":"https://github.com/b4ldr","followers_url":"https://api.github.com/users/b4ldr/followers","following_url":"https://api.github.com/users/b4ldr/following{/other_user}","gists_url":"https://api.github.com/users/b4ldr/gists{/gist_id}","starred_url":"https://api.github.com/users/b4ldr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/b4ldr/subscriptions","organizations_url":"https://api.github.com/users/b4ldr/orgs","repos_url":"https://api.github.com/users/b4ldr/repos","events_url":"https://api.github.com/users/b4ldr/events{/privacy}","received_events_url":"https://api.github.com/users/b4ldr/received_events","type":"User","site_admin":false},"created_at":"2015-07-06T11:21:31Z","updated_at":"2015-07-06T11:21:31Z","body":"Yes it is, i tried to split them but i must have made a mistake when branching #97. Ill close this as you seem happy with #97 "}}