{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/41195","repository_url":"https://api.github.com/repos/ansible/ansible","labels_url":"https://api.github.com/repos/ansible/ansible/issues/41195/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/41195/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/41195/events","html_url":"https://github.com/ansible/ansible/pull/41195","id":329925972,"node_id":"MDExOlB1bGxSZXF1ZXN0MTkzMDU4MzQy","number":41195,"title":"dig lookup plugin: stop masking resolution errors","user":{"login":"fghaas","id":610707,"node_id":"MDQ6VXNlcjYxMDcwNw==","avatar_url":"https://avatars1.githubusercontent.com/u/610707?v=4","gravatar_id":"","url":"https://api.github.com/users/fghaas","html_url":"https://github.com/fghaas","followers_url":"https://api.github.com/users/fghaas/followers","following_url":"https://api.github.com/users/fghaas/following{/other_user}","gists_url":"https://api.github.com/users/fghaas/gists{/gist_id}","starred_url":"https://api.github.com/users/fghaas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fghaas/subscriptions","organizations_url":"https://api.github.com/users/fghaas/orgs","repos_url":"https://api.github.com/users/fghaas/repos","events_url":"https://api.github.com/users/fghaas/events{/privacy}","received_events_url":"https://api.github.com/users/fghaas/received_events","type":"User","site_admin":false},"labels":[{"id":947190508,"node_id":"MDU6TGFiZWw5NDcxOTA1MDg=","url":"https://api.github.com/repos/ansible/ansible/labels/affects_2.7","name":"affects_2.7","color":"ededed","default":false},{"id":785271699,"node_id":"MDU6TGFiZWw3ODUyNzE2OTk=","url":"https://api.github.com/repos/ansible/ansible/labels/bug","name":"bug","color":"fbca04","default":true},{"id":61436281,"node_id":"MDU6TGFiZWw2MTQzNjI4MQ==","url":"https://api.github.com/repos/ansible/ansible/labels/needs_revision","name":"needs_revision","color":"eb6450","default":false},{"id":581543918,"node_id":"MDU6TGFiZWw1ODE1NDM5MTg=","url":"https://api.github.com/repos/ansible/ansible/labels/stale_ci","name":"stale_ci","color":"fef2c0","default":false},{"id":636352796,"node_id":"MDU6TGFiZWw2MzYzNTI3OTY=","url":"https://api.github.com/repos/ansible/ansible/labels/support:core","name":"support:core","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2018-06-06T15:43:57Z","updated_at":"2018-06-29T19:09:41Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/41195","html_url":"https://github.com/ansible/ansible/pull/41195","diff_url":"https://github.com/ansible/ansible/pull/41195.diff","patch_url":"https://github.com/ansible/ansible/pull/41195.patch"},"body":"##### SUMMARY\r\n\r\nWhen users use the `dig` lookup plugin, it is reasonable to assume that they would like the lookup to return meaningful data. If that data is not forthcoming from the DNS, then it's more prudent to just fail the playbook run, rather than carry on and return data of questionable utility (like the empty string, or the literal `NXDOMAIN`).\r\n\r\nFixes #41192.\r\n\r\n##### ISSUE TYPE\r\n - Bugfix Pull Request\r\n\r\n##### COMPONENT NAME\r\ndig\r\n\r\n##### ANSIBLE VERSION\r\n```\r\nansible 2.5.4\r\n  config file = /home/florian/.ansible.cfg\r\n  configured module search path = [u'/home/florian/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']\r\n  ansible python module location = /usr/lib/python2.7/dist-packages/ansible\r\n  executable location = /usr/bin/ansible\r\n  python version = 2.7.15rc1 (default, Apr 15 2018, 21:51:34) [GCC 7.3.0]\r\n\r\n```\r\n\r\n\r\n##### ADDITIONAL INFORMATION\r\nThis is a first stab at the issue; please let me know if I'm headed in the right direction.\r\n"},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/401447299","html_url":"https://github.com/ansible/ansible/pull/41195#issuecomment-401447299","issue_url":"https://api.github.com/repos/ansible/ansible/issues/41195","id":401447299,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMTQ0NzI5OQ==","user":{"login":"fghaas","id":610707,"node_id":"MDQ6VXNlcjYxMDcwNw==","avatar_url":"https://avatars1.githubusercontent.com/u/610707?v=4","gravatar_id":"","url":"https://api.github.com/users/fghaas","html_url":"https://github.com/fghaas","followers_url":"https://api.github.com/users/fghaas/followers","following_url":"https://api.github.com/users/fghaas/following{/other_user}","gists_url":"https://api.github.com/users/fghaas/gists{/gist_id}","starred_url":"https://api.github.com/users/fghaas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fghaas/subscriptions","organizations_url":"https://api.github.com/users/fghaas/orgs","repos_url":"https://api.github.com/users/fghaas/repos","events_url":"https://api.github.com/users/fghaas/events{/privacy}","received_events_url":"https://api.github.com/users/fghaas/received_events","type":"User","site_admin":false},"created_at":"2018-06-29T19:09:41Z","updated_at":"2018-06-29T19:09:41Z","author_association":"CONTRIBUTOR","body":"@bcoca That is exactly what @jpmens originally suggested, to which I explained why I thought why this wasn't a good idea, to which he then answered he had come to a similar conclusion. I still think it isn't a good idea."}}
{"actor":{"display_login":"fghaas"}}