{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/41546","repository_url":"https://api.github.com/repos/ansible/ansible","labels_url":"https://api.github.com/repos/ansible/ansible/issues/41546/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/41546/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/41546/events","html_url":"https://github.com/ansible/ansible/pull/41546","id":332451099,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk0OTMwOTc1","number":41546,"title":"Use string_types for isinstance checking in _check_conditional","user":{"login":"sivel","id":39340,"node_id":"MDQ6VXNlcjM5MzQw","avatar_url":"https://avatars3.githubusercontent.com/u/39340?v=4","gravatar_id":"","url":"https://api.github.com/users/sivel","html_url":"https://github.com/sivel","followers_url":"https://api.github.com/users/sivel/followers","following_url":"https://api.github.com/users/sivel/following{/other_user}","gists_url":"https://api.github.com/users/sivel/gists{/gist_id}","starred_url":"https://api.github.com/users/sivel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sivel/subscriptions","organizations_url":"https://api.github.com/users/sivel/orgs","repos_url":"https://api.github.com/users/sivel/repos","events_url":"https://api.github.com/users/sivel/events{/privacy}","received_events_url":"https://api.github.com/users/sivel/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":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":3,"created_at":"2018-06-14T15:24:01Z","updated_at":"2018-06-15T13:41:04Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/41546","html_url":"https://github.com/ansible/ansible/pull/41546","diff_url":"https://github.com/ansible/ansible/pull/41546.diff","patch_url":"https://github.com/ansible/ansible/pull/41546.patch"},"body":"##### SUMMARY\r\nUse `string_types` for `isinstance` checking in `_check_conditional`. Fixes #41475\r\n\r\n##### ISSUE TYPE\r\n - Bugfix Pull Request\r\n\r\n##### COMPONENT NAME\r\nlib/ansible/playbook/conditional.py\r\n\r\n##### ANSIBLE VERSION\r\n<!--- Paste verbatim output from \"ansible --version\" between quotes below -->\r\n```\r\n2.5\r\n2.6\r\n2.7\r\n```\r\n\r\n\r\n##### ADDITIONAL INFORMATION\r\n<!--- Include additional information to help people understand the change here.\r\nFor bugs that don't have a linked bug report, a step-by-step reproduction\r\nof the problem is helpful. -->\r\n\r\n<!--- Paste verbatim command output below, e.g. before and after your change -->\r\n```\r\n\r\n```"},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/397623501","html_url":"https://github.com/ansible/ansible/pull/41546#issuecomment-397623501","issue_url":"https://api.github.com/repos/ansible/ansible/issues/41546","id":397623501,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NzYyMzUwMQ==","user":{"login":"bcoca","id":836171,"node_id":"MDQ6VXNlcjgzNjE3MQ==","avatar_url":"https://avatars1.githubusercontent.com/u/836171?v=4","gravatar_id":"","url":"https://api.github.com/users/bcoca","html_url":"https://github.com/bcoca","followers_url":"https://api.github.com/users/bcoca/followers","following_url":"https://api.github.com/users/bcoca/following{/other_user}","gists_url":"https://api.github.com/users/bcoca/gists{/gist_id}","starred_url":"https://api.github.com/users/bcoca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bcoca/subscriptions","organizations_url":"https://api.github.com/users/bcoca/orgs","repos_url":"https://api.github.com/users/bcoca/repos","events_url":"https://api.github.com/users/bcoca/events{/privacy}","received_events_url":"https://api.github.com/users/bcoca/received_events","type":"User","site_admin":false},"created_at":"2018-06-15T13:41:04Z","updated_at":"2018-06-15T13:41:04Z","author_association":"MEMBER","body":"conditionals get populated from yaml/jinja2, both of which require utf8 ... unsure how this would be anything other than u'' at this point unless something is transforming them incorrectly in between input and this evaluation."}}
{"actor":{"display_login":"bcoca"}}