{"action":"closed","number":34662,"pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/34662","id":161969663,"node_id":"MDExOlB1bGxSZXF1ZXN0MTYxOTY5NjYz","html_url":"https://github.com/ansible/ansible/pull/34662","diff_url":"https://github.com/ansible/ansible/pull/34662.diff","patch_url":"https://github.com/ansible/ansible/pull/34662.patch","issue_url":"https://api.github.com/repos/ansible/ansible/issues/34662","number":34662,"state":"closed","locked":false,"title":"Fixes python keyerror if no port is specified and default should be used","user":{"login":"grumpymatt","id":32645437,"node_id":"MDQ6VXNlcjMyNjQ1NDM3","avatar_url":"https://avatars3.githubusercontent.com/u/32645437?v=4","gravatar_id":"","url":"https://api.github.com/users/grumpymatt","html_url":"https://github.com/grumpymatt","followers_url":"https://api.github.com/users/grumpymatt/followers","following_url":"https://api.github.com/users/grumpymatt/following{/other_user}","gists_url":"https://api.github.com/users/grumpymatt/gists{/gist_id}","starred_url":"https://api.github.com/users/grumpymatt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grumpymatt/subscriptions","organizations_url":"https://api.github.com/users/grumpymatt/orgs","repos_url":"https://api.github.com/users/grumpymatt/repos","events_url":"https://api.github.com/users/grumpymatt/events{/privacy}","received_events_url":"https://api.github.com/users/grumpymatt/received_events","type":"User","site_admin":false},"body":"##### SUMMARY\r\n<!--- Describe the change, including rationale and design decisions -->\r\nReceived python keyerror when port value was missing. Used python dict get method to properly default the port value\r\n\r\n<!---\r\nIf you are fixing an existing issue, please include \"Fixes #nnn\" in your\r\ncommit message and your description; but you should still explain what\r\nthe change does.\r\n-->\r\n\r\n##### ISSUE TYPE\r\n<!--- Pick one below and delete the rest: -->\r\n - Bugfix Pull Request\r\n\r\n##### COMPONENT NAME\r\n<!--- Name of the module, plugin, module or task -->\r\n\r\n##### ANSIBLE VERSION\r\n<!--- Paste verbatim output from \"ansible --version\" between quotes below -->\r\n```\r\nansible 2.5.0 (devel d1846425db) last updated 2018/01/09 19:53:23 (GMT +000)\r\n  config file = /home/itmxg01/ansible/vmware-provisioning/ansible.cfg\r\n  configured module search path = [u'/home/itmxg01/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']\r\n  ansible python module location = /home/itmxg01/ansible/ansible/lib/ansible\r\n  executable location = /home/itmxg01/ansible/ansible/bin/ansible\r\n  python version = 2.7.5 (default, Aug  4 2017, 00:39:18) [GCC 4.8.5 20150623 (Red Hat 4.8.5-16)]\r\n\r\n```\r\n\r\n\r\n##### ADDITIONAL INFORMATION\r\n<!---\r\nInclude 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\r\n<!--- Paste verbatim command output below, e.g. before and after your change -->\r\n```\r\n\r\n```\r\n","created_at":"2018-01-09T20:37:06Z","updated_at":"2018-07-02T12:47:49Z","closed_at":"2018-07-02T12:47:49Z","merged_at":null,"merge_commit_sha":"65bc86fb6994f50dd0b739793c2ef71f50ca94e4","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[{"id":686214011,"node_id":"MDU6TGFiZWw2ODYyMTQwMTE=","url":"https://api.github.com/repos/ansible/ansible/labels/affects_2.5","name":"affects_2.5","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":723150340,"node_id":"MDU6TGFiZWw3MjMxNTAzNDA=","url":"https://api.github.com/repos/ansible/ansible/labels/new_contributor","name":"new_contributor","color":"c2e0c6","default":false},{"id":970086458,"node_id":"MDU6TGFiZWw5NzAwODY0NTg=","url":"https://api.github.com/repos/ansible/ansible/labels/small_patch","name":"small_patch","color":"ededed","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":636327968,"node_id":"MDU6TGFiZWw2MzYzMjc5Njg=","url":"https://api.github.com/repos/ansible/ansible/labels/support:community","name":"support:community","color":"ededed","default":false},{"id":427121291,"node_id":"MDU6TGFiZWw0MjcxMjEyOTE=","url":"https://api.github.com/repos/ansible/ansible/labels/vmware","name":"vmware","color":"d93f0b","default":false}],"milestone":null,"commits_url":"https://api.github.com/repos/ansible/ansible/pulls/34662/commits","review_comments_url":"https://api.github.com/repos/ansible/ansible/pulls/34662/comments","review_comment_url":"https://api.github.com/repos/ansible/ansible/pulls/comments{/number}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/34662/comments","statuses_url":"https://api.github.com/repos/ansible/ansible/statuses/4e4ea2508c65e2bfb7b4920ccb7e40d2f16cfc86","head":{"label":"grumpymatt:fix_py_keyerror","ref":"fix_py_keyerror","sha":"4e4ea2508c65e2bfb7b4920ccb7e40d2f16cfc86","user":{"login":"grumpymatt","id":32645437,"node_id":"MDQ6VXNlcjMyNjQ1NDM3","avatar_url":"https://avatars3.githubusercontent.com/u/32645437?v=4","gravatar_id":"","url":"https://api.github.com/users/grumpymatt","html_url":"https://github.com/grumpymatt","followers_url":"https://api.github.com/users/grumpymatt/followers","following_url":"https://api.github.com/users/grumpymatt/following{/other_user}","gists_url":"https://api.github.com/users/grumpymatt/gists{/gist_id}","starred_url":"https://api.github.com/users/grumpymatt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grumpymatt/subscriptions","organizations_url":"https://api.github.com/users/grumpymatt/orgs","repos_url":"https://api.github.com/users/grumpymatt/repos","events_url":"https://api.github.com/users/grumpymatt/events{/privacy}","received_events_url":"https://api.github.com/users/grumpymatt/received_events","type":"User","site_admin":false},"repo":{"id":116866590,"node_id":"MDEwOlJlcG9zaXRvcnkxMTY4NjY1OTA=","name":"ansible","full_name":"grumpymatt/ansible","owner":{"login":"grumpymatt","id":32645437,"node_id":"MDQ6VXNlcjMyNjQ1NDM3","avatar_url":"https://avatars3.githubusercontent.com/u/32645437?v=4","gravatar_id":"","url":"https://api.github.com/users/grumpymatt","html_url":"https://github.com/grumpymatt","followers_url":"https://api.github.com/users/grumpymatt/followers","following_url":"https://api.github.com/users/grumpymatt/following{/other_user}","gists_url":"https://api.github.com/users/grumpymatt/gists{/gist_id}","starred_url":"https://api.github.com/users/grumpymatt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grumpymatt/subscriptions","organizations_url":"https://api.github.com/users/grumpymatt/orgs","repos_url":"https://api.github.com/users/grumpymatt/repos","events_url":"https://api.github.com/users/grumpymatt/events{/privacy}","received_events_url":"https://api.github.com/users/grumpymatt/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/grumpymatt/ansible","description":"Ansible is a radically simple IT automation platform that makes your applications and systems easier to deploy. Avoid writing scripts or custom code to deploy and update your applications— automate in a language that approaches plain English, using SSH, with no agents to install on remote systems.","fork":true,"url":"https://api.github.com/repos/grumpymatt/ansible","forks_url":"https://api.github.com/repos/grumpymatt/ansible/forks","keys_url":"https://api.github.com/repos/grumpymatt/ansible/keys{/key_id}","collaborators_url":"https://api.github.com/repos/grumpymatt/ansible/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/grumpymatt/ansible/teams","hooks_url":"https://api.github.com/repos/grumpymatt/ansible/hooks","issue_events_url":"https://api.github.com/repos/grumpymatt/ansible/issues/events{/number}","events_url":"https://api.github.com/repos/grumpymatt/ansible/events","assignees_url":"https://api.github.com/repos/grumpymatt/ansible/assignees{/user}","branches_url":"https://api.github.com/repos/grumpymatt/ansible/branches{/branch}","tags_url":"https://api.github.com/repos/grumpymatt/ansible/tags","blobs_url":"https://api.github.com/repos/grumpymatt/ansible/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/grumpymatt/ansible/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/grumpymatt/ansible/git/refs{/sha}","trees_url":"https://api.github.com/repos/grumpymatt/ansible/git/trees{/sha}","statuses_url":"https://api.github.com/repos/grumpymatt/ansible/statuses/{sha}","languages_url":"https://api.github.com/repos/grumpymatt/ansible/languages","stargazers_url":"https://api.github.com/repos/grumpymatt/ansible/stargazers","contributors_url":"https://api.github.com/repos/grumpymatt/ansible/contributors","subscribers_url":"https://api.github.com/repos/grumpymatt/ansible/subscribers","subscription_url":"https://api.github.com/repos/grumpymatt/ansible/subscription","commits_url":"https://api.github.com/repos/grumpymatt/ansible/commits{/sha}","git_commits_url":"https://api.github.com/repos/grumpymatt/ansible/git/commits{/sha}","comments_url":"https://api.github.com/repos/grumpymatt/ansible/comments{/number}","issue_comment_url":"https://api.github.com/repos/grumpymatt/ansible/issues/comments{/number}","contents_url":"https://api.github.com/repos/grumpymatt/ansible/contents/{+path}","compare_url":"https://api.github.com/repos/grumpymatt/ansible/compare/{base}...{head}","merges_url":"https://api.github.com/repos/grumpymatt/ansible/merges","archive_url":"https://api.github.com/repos/grumpymatt/ansible/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/grumpymatt/ansible/downloads","issues_url":"https://api.github.com/repos/grumpymatt/ansible/issues{/number}","pulls_url":"https://api.github.com/repos/grumpymatt/ansible/pulls{/number}","milestones_url":"https://api.github.com/repos/grumpymatt/ansible/milestones{/number}","notifications_url":"https://api.github.com/repos/grumpymatt/ansible/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/grumpymatt/ansible/labels{/name}","releases_url":"https://api.github.com/repos/grumpymatt/ansible/releases{/id}","deployments_url":"https://api.github.com/repos/grumpymatt/ansible/deployments","created_at":"2018-01-09T20:21:40Z","updated_at":"2018-04-17T20:47:12Z","pushed_at":"2018-04-17T20:57:58Z","git_url":"git://github.com/grumpymatt/ansible.git","ssh_url":"git@github.com:grumpymatt/ansible.git","clone_url":"https://github.com/grumpymatt/ansible.git","svn_url":"https://github.com/grumpymatt/ansible","homepage":"https://www.ansible.com/","size":108425,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"gpl-3.0","name":"GNU General Public License v3.0","spdx_id":"GPL-3.0","url":"https://api.github.com/licenses/gpl-3.0","node_id":"MDc6TGljZW5zZTk="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"devel"}},"base":{"label":"ansible:devel","ref":"devel","sha":"70e49b9243ffb3ce24d51676c348d8d7531b9610","user":{"login":"ansible","id":1507452,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE1MDc0NTI=","avatar_url":"https://avatars2.githubusercontent.com/u/1507452?v=4","gravatar_id":"","url":"https://api.github.com/users/ansible","html_url":"https://github.com/ansible","followers_url":"https://api.github.com/users/ansible/followers","following_url":"https://api.github.com/users/ansible/following{/other_user}","gists_url":"https://api.github.com/users/ansible/gists{/gist_id}","starred_url":"https://api.github.com/users/ansible/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ansible/subscriptions","organizations_url":"https://api.github.com/users/ansible/orgs","repos_url":"https://api.github.com/users/ansible/repos","events_url":"https://api.github.com/users/ansible/events{/privacy}","received_events_url":"https://api.github.com/users/ansible/received_events","type":"Organization","site_admin":false},"repo":{"id":3638964,"node_id":"MDEwOlJlcG9zaXRvcnkzNjM4OTY0","name":"ansible","full_name":"ansible/ansible","owner":{"login":"ansible","id":1507452,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE1MDc0NTI=","avatar_url":"https://avatars2.githubusercontent.com/u/1507452?v=4","gravatar_id":"","url":"https://api.github.com/users/ansible","html_url":"https://github.com/ansible","followers_url":"https://api.github.com/users/ansible/followers","following_url":"https://api.github.com/users/ansible/following{/other_user}","gists_url":"https://api.github.com/users/ansible/gists{/gist_id}","starred_url":"https://api.github.com/users/ansible/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ansible/subscriptions","organizations_url":"https://api.github.com/users/ansible/orgs","repos_url":"https://api.github.com/users/ansible/repos","events_url":"https://api.github.com/users/ansible/events{/privacy}","received_events_url":"https://api.github.com/users/ansible/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/ansible/ansible","description":"Ansible is a radically simple IT automation platform that makes your applications and systems easier to deploy. Avoid writing scripts or custom code to deploy and update your applications — automate in a language that approaches plain English, using SSH, with no agents to install on remote systems. https://docs.ansible.com/ansible/","fork":false,"url":"https://api.github.com/repos/ansible/ansible","forks_url":"https://api.github.com/repos/ansible/ansible/forks","keys_url":"https://api.github.com/repos/ansible/ansible/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ansible/ansible/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ansible/ansible/teams","hooks_url":"https://api.github.com/repos/ansible/ansible/hooks","issue_events_url":"https://api.github.com/repos/ansible/ansible/issues/events{/number}","events_url":"https://api.github.com/repos/ansible/ansible/events","assignees_url":"https://api.github.com/repos/ansible/ansible/assignees{/user}","branches_url":"https://api.github.com/repos/ansible/ansible/branches{/branch}","tags_url":"https://api.github.com/repos/ansible/ansible/tags","blobs_url":"https://api.github.com/repos/ansible/ansible/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ansible/ansible/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ansible/ansible/git/refs{/sha}","trees_url":"https://api.github.com/repos/ansible/ansible/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ansible/ansible/statuses/{sha}","languages_url":"https://api.github.com/repos/ansible/ansible/languages","stargazers_url":"https://api.github.com/repos/ansible/ansible/stargazers","contributors_url":"https://api.github.com/repos/ansible/ansible/contributors","subscribers_url":"https://api.github.com/repos/ansible/ansible/subscribers","subscription_url":"https://api.github.com/repos/ansible/ansible/subscription","commits_url":"https://api.github.com/repos/ansible/ansible/commits{/sha}","git_commits_url":"https://api.github.com/repos/ansible/ansible/git/commits{/sha}","comments_url":"https://api.github.com/repos/ansible/ansible/comments{/number}","issue_comment_url":"https://api.github.com/repos/ansible/ansible/issues/comments{/number}","contents_url":"https://api.github.com/repos/ansible/ansible/contents/{+path}","compare_url":"https://api.github.com/repos/ansible/ansible/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ansible/ansible/merges","archive_url":"https://api.github.com/repos/ansible/ansible/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ansible/ansible/downloads","issues_url":"https://api.github.com/repos/ansible/ansible/issues{/number}","pulls_url":"https://api.github.com/repos/ansible/ansible/pulls{/number}","milestones_url":"https://api.github.com/repos/ansible/ansible/milestones{/number}","notifications_url":"https://api.github.com/repos/ansible/ansible/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ansible/ansible/labels{/name}","releases_url":"https://api.github.com/repos/ansible/ansible/releases{/id}","deployments_url":"https://api.github.com/repos/ansible/ansible/deployments","created_at":"2012-03-06T14:58:02Z","updated_at":"2018-07-02T12:43:39Z","pushed_at":"2018-07-02T12:45:37Z","git_url":"git://github.com/ansible/ansible.git","ssh_url":"git@github.com:ansible/ansible.git","clone_url":"https://github.com/ansible/ansible.git","svn_url":"https://github.com/ansible/ansible","homepage":"https://www.ansible.com/","size":128572,"stargazers_count":31062,"watchers_count":31062,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":11818,"mirror_url":null,"archived":false,"open_issues_count":5141,"license":{"key":"gpl-3.0","name":"GNU General Public License v3.0","spdx_id":"GPL-3.0","url":"https://api.github.com/licenses/gpl-3.0","node_id":"MDc6TGljZW5zZTk="},"forks":11818,"open_issues":5141,"watchers":31062,"default_branch":"devel"}},"_links":{"self":{"href":"https://api.github.com/repos/ansible/ansible/pulls/34662"},"html":{"href":"https://github.com/ansible/ansible/pull/34662"},"issue":{"href":"https://api.github.com/repos/ansible/ansible/issues/34662"},"comments":{"href":"https://api.github.com/repos/ansible/ansible/issues/34662/comments"},"review_comments":{"href":"https://api.github.com/repos/ansible/ansible/pulls/34662/comments"},"review_comment":{"href":"https://api.github.com/repos/ansible/ansible/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/ansible/ansible/pulls/34662/commits"},"statuses":{"href":"https://api.github.com/repos/ansible/ansible/statuses/4e4ea2508c65e2bfb7b4920ccb7e40d2f16cfc86"}},"author_association":"NONE","merged":false,"mergeable":true,"rebaseable":true,"mergeable_state":"clean","merged_by":null,"comments":7,"review_comments":0,"maintainer_can_modify":false,"commits":1,"additions":1,"deletions":1,"changed_files":1}}
{"actor":{"display_login":"Akasurde"}}