payload: {"action":"created","issue":{"url":"https://api.github.com/repos/Azure/sonic-mgmt/issues/1172","repository_url":"https://api.github.com/repos/Azure/sonic-mgmt","labels_url":"https://api.github.com/repos/Azure/sonic-mgmt/issues/1172/labels{/name}","comments_url":"https://api.github.com/repos/Azure/sonic-mgmt/issues/1172/comments","events_url":"https://api.github.com/repos/Azure/sonic-mgmt/issues/1172/events","html_url":"https://github.com/Azure/sonic-mgmt/pull/1172","id":510676825,"node_id":"MDExOlB1bGxSZXF1ZXN0MzMxMDA5NDgx","number":1172,"title":"[tests] add sym link to ansible.cfg for pytest-tests","user":{"login":"stepanblyschak","id":38952541,"node_id":"MDQ6VXNlcjM4OTUyNTQx","avatar_url":"https://avatars0.githubusercontent.com/u/38952541?v=4","gravatar_id":"","url":"https://api.github.com/users/stepanblyschak","html_url":"https://github.com/stepanblyschak","followers_url":"https://api.github.com/users/stepanblyschak/followers","following_url":"https://api.github.com/users/stepanblyschak/following{/other_user}","gists_url":"https://api.github.com/users/stepanblyschak/gists{/gist_id}","starred_url":"https://api.github.com/users/stepanblyschak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stepanblyschak/subscriptions","organizations_url":"https://api.github.com/users/stepanblyschak/orgs","repos_url":"https://api.github.com/users/stepanblyschak/repos","events_url":"https://api.github.com/users/stepanblyschak/events{/privacy}","received_events_url":"https://api.github.com/users/stepanblyschak/received_events","type":"User","site_admin":false},"labels":[{"id":1198996779,"node_id":"MDU6TGFiZWwxMTk4OTk2Nzc5","url":"https://api.github.com/repos/Azure/sonic-mgmt/labels/Request%20for%20201811%20branch%20:shipit:","name":"Request for 201811 branch :shipit:","color":"b5a4f9","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2019-10-22T14:00:07Z","updated_at":"2019-10-24T16:23:40Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/Azure/sonic-mgmt/pulls/1172","html_url":"https://github.com/Azure/sonic-mgmt/pull/1172","diff_url":"https://github.com/Azure/sonic-mgmt/pull/1172.diff","patch_url":"https://github.com/Azure/sonic-mgmt/pull/1172.patch"},"body":"Need to make pytest-ansible to use ansible.cfg, otherwise if hosts\r\nare not in known_hosts we will get the following error:\r\n\r\n```\r\n16:02:49         \"    def _run(self, *module_args, **complex_args):\",\r\n16:02:49         \"        module_ignore_errors = complex_args.pop('module_ignore_errors', False)\",\r\n16:02:49         \"        module_async = complex_args.pop('module_async', False)\",\r\n16:02:49         \"    \",\r\n16:02:49         \"        if module_async:\",\r\n16:02:49         \"            def run_module(module_args, complex_args):\",\r\n16:02:49         \"                return self.module(*module_args, **complex_args)[self.hostname]\",\r\n16:02:49         \"            pool = ThreadPool()\",\r\n16:02:49         \"            result = pool.apply_async(run_module, (module_args, complex_args))\",\r\n16:02:49         \"            return pool, result\",\r\n16:02:49         \"    \",\r\n16:02:49         \"        res = self.module(*module_args, **complex_args)[self.hostname]\",\r\n16:02:49         \"        if res.is_failed and not module_ignore_errors:\",\r\n16:02:49         \">           raise RunAnsibleModuleFail(\\\"run module {} failed, errmsg {}\\\".format(self.module_name, res))\",\r\n16:02:49         \"E           RunAnsibleModuleFail: run module command failed, errmsg {'msg': u\\\"ERROR! Using a SSH password instead of a key is not possible because Host Key checking is enabled and sshpass does not support this.  Please add this host's fingerprint to your known_hosts file to manage this host.\\\", 'failed': True}\",\r\n16:02:49         \"\",\r\n```\r\n\r\nSigned-off-by: Stepan Blyschak <stepanb@mellanox.com>\r\n\r\n<!--\r\nPlease make sure you've read and understood our contributing guidelines;\r\nhttps://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md\r\n\r\nPlease provide following information to help code review process a bit easier:\r\n-->\r\n### Description of PR\r\n<!--\r\n- Please include a summary of the change and which issue is fixed.\r\n- Please also include relevant motivation and context. Where should reviewer start? background contaxt?\r\n- List any dependencies that are required for this change.\r\n-->\r\n\r\nSummary:\r\nFixes # (issue)\r\n\r\n### Type of change\r\n\r\n- [x] Bug fix\r\n- [] Testbed and Framework(new/improvement)\r\n- [] Test case(new/improvement)\r\n\r\n### Approach\r\n#### How did you do it?\r\n\r\n#### How did you verify/test it?\r\n\r\n#### Any platform specific information?\r\n\r\n#### Supported testbed topology if it's a new test case?\r\n\r\n### Documentation \r\n<!--\r\n(If it's a new feature, new test case)\r\nDid you update documentation/Wiki relevant to your implementation?\r\nLink to the wiki page?\r\n-->\r\n"},"comment":{"url":"https://api.github.com/repos/Azure/sonic-mgmt/issues/comments/545996036","html_url":"https://github.com/Azure/sonic-mgmt/pull/1172#issuecomment-545996036","issue_url":"https://api.github.com/repos/Azure/sonic-mgmt/issues/1172","id":545996036,"node_id":"MDEyOklzc3VlQ29tbWVudDU0NTk5NjAzNg==","user":{"login":"stepanblyschak","id":38952541,"node_id":"MDQ6VXNlcjM4OTUyNTQx","avatar_url":"https://avatars0.githubusercontent.com/u/38952541?v=4","gravatar_id":"","url":"https://api.github.com/users/stepanblyschak","html_url":"https://github.com/stepanblyschak","followers_url":"https://api.github.com/users/stepanblyschak/followers","following_url":"https://api.github.com/users/stepanblyschak/following{/other_user}","gists_url":"https://api.github.com/users/stepanblyschak/gists{/gist_id}","starred_url":"https://api.github.com/users/stepanblyschak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stepanblyschak/subscriptions","organizations_url":"https://api.github.com/users/stepanblyschak/orgs","repos_url":"https://api.github.com/users/stepanblyschak/repos","events_url":"https://api.github.com/users/stepanblyschak/events{/privacy}","received_events_url":"https://api.github.com/users/stepanblyschak/received_events","type":"User","site_admin":false},"created_at":"2019-10-24T16:23:40Z","updated_at":"2019-10-24T16:23:40Z","author_association":"COLLABORATOR","body":"@lguohan\r\nWhy they should be different if they pytest still uses ansible transport and ansible modules?\r\nBesides, we and I assume some one else have different ansible.cfg tailored for our environment.\r\nCreating a copy based on upstream may produce a difference for us between ansible and pytest transport/module settings and we will have to maintain both files."}}
other: {"actor":{"display_login":"stepanblyschak"}}