{"action":"created","issue":{"url":"https://api.github.com/repos/influxdb/telegraf/issues/133","labels_url":"https://api.github.com/repos/influxdb/telegraf/issues/133/labels{/name}","comments_url":"https://api.github.com/repos/influxdb/telegraf/issues/133/comments","events_url":"https://api.github.com/repos/influxdb/telegraf/issues/133/events","html_url":"https://github.com/influxdb/telegraf/pull/133","id":102567127,"number":133,"title":"Fix errors not being logged for plugin.Gather","user":{"login":"nickscript0","id":9680595,"avatar_url":"https://avatars.githubusercontent.com/u/9680595?v=3","gravatar_id":"","url":"https://api.github.com/users/nickscript0","html_url":"https://github.com/nickscript0","followers_url":"https://api.github.com/users/nickscript0/followers","following_url":"https://api.github.com/users/nickscript0/following{/other_user}","gists_url":"https://api.github.com/users/nickscript0/gists{/gist_id}","starred_url":"https://api.github.com/users/nickscript0/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickscript0/subscriptions","organizations_url":"https://api.github.com/users/nickscript0/orgs","repos_url":"https://api.github.com/users/nickscript0/repos","events_url":"https://api.github.com/users/nickscript0/events{/privacy}","received_events_url":"https://api.github.com/users/nickscript0/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-08-22T21:02:04Z","updated_at":"2015-08-24T17:35:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/influxdb/telegraf/pulls/133","html_url":"https://github.com/influxdb/telegraf/pull/133","diff_url":"https://github.com/influxdb/telegraf/pull/133.diff","patch_url":"https://github.com/influxdb/telegraf/pull/133.patch"},"body":"Currently if plugin.Gather returns an error it does not show up in the telegraf log. This PR adds logging to show an error in both crankParallel and crankSeparate cases.\r\n\r\nFor example a misconfigured plugin as follows currently show no errors:\r\n```\r\n[mysql]\r\nservers = [\"notlocalhost\"]\r\n```\r\n\r\nAfter this fix it shows an error as expected:\r\n```\r\n./telegraf -config /etc/opt/telegraf/telegraf.conf\r\n2015/08/22 20:48:07 Starting Telegraf\r\n2015/08/22 20:48:07 Loaded outputs: influxdb\r\n2015/08/22 20:48:07 Loaded plugins: cpu mysql\r\n2015/08/22 20:48:07 Tags enabled: dc=test1 host=a11790\r\n2015/08/22 20:48:07 Error in plugins: Invalid DSN: Missing the slash separating the database name\r\n```"},"comment":{"url":"https://api.github.com/repos/influxdb/telegraf/issues/comments/134310891","html_url":"https://github.com/influxdb/telegraf/pull/133#issuecomment-134310891","issue_url":"https://api.github.com/repos/influxdb/telegraf/issues/133","id":134310891,"user":{"login":"nickscript0","id":9680595,"avatar_url":"https://avatars.githubusercontent.com/u/9680595?v=3","gravatar_id":"","url":"https://api.github.com/users/nickscript0","html_url":"https://github.com/nickscript0","followers_url":"https://api.github.com/users/nickscript0/followers","following_url":"https://api.github.com/users/nickscript0/following{/other_user}","gists_url":"https://api.github.com/users/nickscript0/gists{/gist_id}","starred_url":"https://api.github.com/users/nickscript0/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickscript0/subscriptions","organizations_url":"https://api.github.com/users/nickscript0/orgs","repos_url":"https://api.github.com/users/nickscript0/repos","events_url":"https://api.github.com/users/nickscript0/events{/privacy}","received_events_url":"https://api.github.com/users/nickscript0/received_events","type":"User","site_admin":false},"created_at":"2015-08-24T17:35:20Z","updated_at":"2015-08-24T17:35:20Z","body":"Addressed your comments: added plugin name and updated CHANGELOG. Should be good to merge now."}}