{"action":"created","comment":{"url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/comments/88139188","pull_request_review_id":8717257,"id":88139188,"diff_hunk":"@@ -249,14 +250,18 @@ def get_ip_address():\n         logger.info(\"Waiting for %s to be available via SSH\", connect_ip)\n         ssh_client = ssh.SSHClient(hostname=connect_ip, username=vm_analysis_data['username'],\n                                    password=vm_analysis_data['password'], port=22)\n-        wait_for(ssh_client.uptime, num_sec=3600, handle_exception=False)\n+        wait_for(ssh_client.uptime, num_sec=3600, handle_exception=True)\n         vm.ssh = ssh_client\n-\n     vm.system_type = detect_system_type(vm)\n     logger.info(\"Detected system type: %s\", vm.system_type)\n     vm.image = vm_analysis_data['image']\n     vm.connect_ip = connect_ip\n \n+    # TODO:  This is completely wrong and needs to be fixed","path":"cfme/tests/infrastructure/test_instance_analysis.py","position":222,"original_position":222,"commit_id":"f72f6aacb3c0bad6c83570fae57f3dd3ad9d8119","original_commit_id":"f72f6aacb3c0bad6c83570fae57f3dd3ad9d8119","user":{"login":"mshriver","id":21315076,"avatar_url":"https://avatars.githubusercontent.com/u/21315076?v=3","gravatar_id":"","url":"https://api.github.com/users/mshriver","html_url":"https://github.com/mshriver","followers_url":"https://api.github.com/users/mshriver/followers","following_url":"https://api.github.com/users/mshriver/following{/other_user}","gists_url":"https://api.github.com/users/mshriver/gists{/gist_id}","starred_url":"https://api.github.com/users/mshriver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mshriver/subscriptions","organizations_url":"https://api.github.com/users/mshriver/orgs","repos_url":"https://api.github.com/users/mshriver/repos","events_url":"https://api.github.com/users/mshriver/events{/privacy}","received_events_url":"https://api.github.com/users/mshriver/received_events","type":"User","site_admin":false},"body":"Sounds worthy of a card/issue","created_at":"2016-11-15T23:36:57Z","updated_at":"2016-11-15T23:39:36Z","html_url":"https://github.com/ManageIQ/integration_tests/pull/3796#discussion_r88139188","pull_request_url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796","_links":{"self":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/comments/88139188"},"html":{"href":"https://github.com/ManageIQ/integration_tests/pull/3796#discussion_r88139188"},"pull_request":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796"}}},"pull_request":{"url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796","id":93833082,"html_url":"https://github.com/ManageIQ/integration_tests/pull/3796","diff_url":"https://github.com/ManageIQ/integration_tests/pull/3796.diff","patch_url":"https://github.com/ManageIQ/integration_tests/pull/3796.patch","issue_url":"https://api.github.com/repos/ManageIQ/integration_tests/issues/3796","number":3796,"state":"open","locked":false,"title":"Ssa fixes","user":{"login":"dajohnso","id":477564,"avatar_url":"https://avatars.githubusercontent.com/u/477564?v=3","gravatar_id":"","url":"https://api.github.com/users/dajohnso","html_url":"https://github.com/dajohnso","followers_url":"https://api.github.com/users/dajohnso/followers","following_url":"https://api.github.com/users/dajohnso/following{/other_user}","gists_url":"https://api.github.com/users/dajohnso/gists{/gist_id}","starred_url":"https://api.github.com/users/dajohnso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dajohnso/subscriptions","organizations_url":"https://api.github.com/users/dajohnso/orgs","repos_url":"https://api.github.com/users/dajohnso/repos","events_url":"https://api.github.com/users/dajohnso/events{/privacy}","received_events_url":"https://api.github.com/users/dajohnso/received_events","type":"User","site_admin":false},"body":"Purpose or Intent\r\n=================\r\nA bunch of different changes related to SSA testing which is currently for the most part, completely busted.  There was several efforts in one...\r\n\r\n* move away from vm provisioning through the ui and opt for provider api provisioning for test vms\r\n* add openstack support\r\n* fix the tests\r\n\r\nThis PR is a move towards those goals but I believe its going to require another round of debug/fixing.","created_at":"2016-11-15T19:37:42Z","updated_at":"2016-11-15T23:39:36Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f718f086fd552f40137e3c87b098b4f1372fac84","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796/commits","review_comments_url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796/comments","review_comment_url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/comments{/number}","comments_url":"https://api.github.com/repos/ManageIQ/integration_tests/issues/3796/comments","statuses_url":"https://api.github.com/repos/ManageIQ/integration_tests/statuses/f72f6aacb3c0bad6c83570fae57f3dd3ad9d8119","head":{"label":"dajohnso:ssa_fixes","ref":"ssa_fixes","sha":"f72f6aacb3c0bad6c83570fae57f3dd3ad9d8119","user":{"login":"dajohnso","id":477564,"avatar_url":"https://avatars.githubusercontent.com/u/477564?v=3","gravatar_id":"","url":"https://api.github.com/users/dajohnso","html_url":"https://github.com/dajohnso","followers_url":"https://api.github.com/users/dajohnso/followers","following_url":"https://api.github.com/users/dajohnso/following{/other_user}","gists_url":"https://api.github.com/users/dajohnso/gists{/gist_id}","starred_url":"https://api.github.com/users/dajohnso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dajohnso/subscriptions","organizations_url":"https://api.github.com/users/dajohnso/orgs","repos_url":"https://api.github.com/users/dajohnso/repos","events_url":"https://api.github.com/users/dajohnso/events{/privacy}","received_events_url":"https://api.github.com/users/dajohnso/received_events","type":"User","site_admin":false},"repo":{"id":8909957,"name":"cfme_tests","full_name":"dajohnso/cfme_tests","owner":{"login":"dajohnso","id":477564,"avatar_url":"https://avatars.githubusercontent.com/u/477564?v=3","gravatar_id":"","url":"https://api.github.com/users/dajohnso","html_url":"https://github.com/dajohnso","followers_url":"https://api.github.com/users/dajohnso/followers","following_url":"https://api.github.com/users/dajohnso/following{/other_user}","gists_url":"https://api.github.com/users/dajohnso/gists{/gist_id}","starred_url":"https://api.github.com/users/dajohnso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dajohnso/subscriptions","organizations_url":"https://api.github.com/users/dajohnso/orgs","repos_url":"https://api.github.com/users/dajohnso/repos","events_url":"https://api.github.com/users/dajohnso/events{/privacy}","received_events_url":"https://api.github.com/users/dajohnso/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/dajohnso/cfme_tests","description":"ManageIQ Tests","fork":true,"url":"https://api.github.com/repos/dajohnso/cfme_tests","forks_url":"https://api.github.com/repos/dajohnso/cfme_tests/forks","keys_url":"https://api.github.com/repos/dajohnso/cfme_tests/keys{/key_id}","collaborators_url":"https://api.github.com/repos/dajohnso/cfme_tests/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/dajohnso/cfme_tests/teams","hooks_url":"https://api.github.com/repos/dajohnso/cfme_tests/hooks","issue_events_url":"https://api.github.com/repos/dajohnso/cfme_tests/issues/events{/number}","events_url":"https://api.github.com/repos/dajohnso/cfme_tests/events","assignees_url":"https://api.github.com/repos/dajohnso/cfme_tests/assignees{/user}","branches_url":"https://api.github.com/repos/dajohnso/cfme_tests/branches{/branch}","tags_url":"https://api.github.com/repos/dajohnso/cfme_tests/tags","blobs_url":"https://api.github.com/repos/dajohnso/cfme_tests/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/dajohnso/cfme_tests/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/dajohnso/cfme_tests/git/refs{/sha}","trees_url":"https://api.github.com/repos/dajohnso/cfme_tests/git/trees{/sha}","statuses_url":"https://api.github.com/repos/dajohnso/cfme_tests/statuses/{sha}","languages_url":"https://api.github.com/repos/dajohnso/cfme_tests/languages","stargazers_url":"https://api.github.com/repos/dajohnso/cfme_tests/stargazers","contributors_url":"https://api.github.com/repos/dajohnso/cfme_tests/contributors","subscribers_url":"https://api.github.com/repos/dajohnso/cfme_tests/subscribers","subscription_url":"https://api.github.com/repos/dajohnso/cfme_tests/subscription","commits_url":"https://api.github.com/repos/dajohnso/cfme_tests/commits{/sha}","git_commits_url":"https://api.github.com/repos/dajohnso/cfme_tests/git/commits{/sha}","comments_url":"https://api.github.com/repos/dajohnso/cfme_tests/comments{/number}","issue_comment_url":"https://api.github.com/repos/dajohnso/cfme_tests/issues/comments{/number}","contents_url":"https://api.github.com/repos/dajohnso/cfme_tests/contents/{+path}","compare_url":"https://api.github.com/repos/dajohnso/cfme_tests/compare/{base}...{head}","merges_url":"https://api.github.com/repos/dajohnso/cfme_tests/merges","archive_url":"https://api.github.com/repos/dajohnso/cfme_tests/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/dajohnso/cfme_tests/downloads","issues_url":"https://api.github.com/repos/dajohnso/cfme_tests/issues{/number}","pulls_url":"https://api.github.com/repos/dajohnso/cfme_tests/pulls{/number}","milestones_url":"https://api.github.com/repos/dajohnso/cfme_tests/milestones{/number}","notifications_url":"https://api.github.com/repos/dajohnso/cfme_tests/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/dajohnso/cfme_tests/labels{/name}","releases_url":"https://api.github.com/repos/dajohnso/cfme_tests/releases{/id}","deployments_url":"https://api.github.com/repos/dajohnso/cfme_tests/deployments","created_at":"2013-03-20T17:17:00Z","updated_at":"2015-04-29T18:57:57Z","pushed_at":"2016-11-15T19:55:30Z","git_url":"git://github.com/dajohnso/cfme_tests.git","ssh_url":"git@github.com:dajohnso/cfme_tests.git","clone_url":"https://github.com/dajohnso/cfme_tests.git","svn_url":"https://github.com/dajohnso/cfme_tests","homepage":null,"size":12582,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"ManageIQ:master","ref":"master","sha":"a2e9a580b1196fc7d50c4a6c6eba55cc08bee05e","user":{"login":"ManageIQ","id":1125575,"avatar_url":"https://avatars.githubusercontent.com/u/1125575?v=3","gravatar_id":"","url":"https://api.github.com/users/ManageIQ","html_url":"https://github.com/ManageIQ","followers_url":"https://api.github.com/users/ManageIQ/followers","following_url":"https://api.github.com/users/ManageIQ/following{/other_user}","gists_url":"https://api.github.com/users/ManageIQ/gists{/gist_id}","starred_url":"https://api.github.com/users/ManageIQ/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ManageIQ/subscriptions","organizations_url":"https://api.github.com/users/ManageIQ/orgs","repos_url":"https://api.github.com/users/ManageIQ/repos","events_url":"https://api.github.com/users/ManageIQ/events{/privacy}","received_events_url":"https://api.github.com/users/ManageIQ/received_events","type":"Organization","site_admin":false},"repo":{"id":8061086,"name":"integration_tests","full_name":"ManageIQ/integration_tests","owner":{"login":"ManageIQ","id":1125575,"avatar_url":"https://avatars.githubusercontent.com/u/1125575?v=3","gravatar_id":"","url":"https://api.github.com/users/ManageIQ","html_url":"https://github.com/ManageIQ","followers_url":"https://api.github.com/users/ManageIQ/followers","following_url":"https://api.github.com/users/ManageIQ/following{/other_user}","gists_url":"https://api.github.com/users/ManageIQ/gists{/gist_id}","starred_url":"https://api.github.com/users/ManageIQ/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ManageIQ/subscriptions","organizations_url":"https://api.github.com/users/ManageIQ/orgs","repos_url":"https://api.github.com/users/ManageIQ/repos","events_url":"https://api.github.com/users/ManageIQ/events{/privacy}","received_events_url":"https://api.github.com/users/ManageIQ/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/ManageIQ/integration_tests","description":"ManageIQ integration tests","fork":false,"url":"https://api.github.com/repos/ManageIQ/integration_tests","forks_url":"https://api.github.com/repos/ManageIQ/integration_tests/forks","keys_url":"https://api.github.com/repos/ManageIQ/integration_tests/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ManageIQ/integration_tests/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ManageIQ/integration_tests/teams","hooks_url":"https://api.github.com/repos/ManageIQ/integration_tests/hooks","issue_events_url":"https://api.github.com/repos/ManageIQ/integration_tests/issues/events{/number}","events_url":"https://api.github.com/repos/ManageIQ/integration_tests/events","assignees_url":"https://api.github.com/repos/ManageIQ/integration_tests/assignees{/user}","branches_url":"https://api.github.com/repos/ManageIQ/integration_tests/branches{/branch}","tags_url":"https://api.github.com/repos/ManageIQ/integration_tests/tags","blobs_url":"https://api.github.com/repos/ManageIQ/integration_tests/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ManageIQ/integration_tests/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ManageIQ/integration_tests/git/refs{/sha}","trees_url":"https://api.github.com/repos/ManageIQ/integration_tests/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ManageIQ/integration_tests/statuses/{sha}","languages_url":"https://api.github.com/repos/ManageIQ/integration_tests/languages","stargazers_url":"https://api.github.com/repos/ManageIQ/integration_tests/stargazers","contributors_url":"https://api.github.com/repos/ManageIQ/integration_tests/contributors","subscribers_url":"https://api.github.com/repos/ManageIQ/integration_tests/subscribers","subscription_url":"https://api.github.com/repos/ManageIQ/integration_tests/subscription","commits_url":"https://api.github.com/repos/ManageIQ/integration_tests/commits{/sha}","git_commits_url":"https://api.github.com/repos/ManageIQ/integration_tests/git/commits{/sha}","comments_url":"https://api.github.com/repos/ManageIQ/integration_tests/comments{/number}","issue_comment_url":"https://api.github.com/repos/ManageIQ/integration_tests/issues/comments{/number}","contents_url":"https://api.github.com/repos/ManageIQ/integration_tests/contents/{+path}","compare_url":"https://api.github.com/repos/ManageIQ/integration_tests/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ManageIQ/integration_tests/merges","archive_url":"https://api.github.com/repos/ManageIQ/integration_tests/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ManageIQ/integration_tests/downloads","issues_url":"https://api.github.com/repos/ManageIQ/integration_tests/issues{/number}","pulls_url":"https://api.github.com/repos/ManageIQ/integration_tests/pulls{/number}","milestones_url":"https://api.github.com/repos/ManageIQ/integration_tests/milestones{/number}","notifications_url":"https://api.github.com/repos/ManageIQ/integration_tests/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ManageIQ/integration_tests/labels{/name}","releases_url":"https://api.github.com/repos/ManageIQ/integration_tests/releases{/id}","deployments_url":"https://api.github.com/repos/ManageIQ/integration_tests/deployments","created_at":"2013-02-06T21:59:17Z","updated_at":"2016-10-25T08:25:21Z","pushed_at":"2016-11-15T22:15:39Z","git_url":"git://github.com/ManageIQ/integration_tests.git","ssh_url":"git@github.com:ManageIQ/integration_tests.git","clone_url":"https://github.com/ManageIQ/integration_tests.git","svn_url":"https://github.com/ManageIQ/integration_tests","homepage":"","size":18211,"stargazers_count":30,"watchers_count":30,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":69,"mirror_url":null,"open_issues_count":82,"forks":69,"open_issues":82,"watchers":30,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796"},"html":{"href":"https://github.com/ManageIQ/integration_tests/pull/3796"},"issue":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/issues/3796"},"comments":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/issues/3796/comments"},"review_comments":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796/comments"},"review_comment":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/pulls/3796/commits"},"statuses":{"href":"https://api.github.com/repos/ManageIQ/integration_tests/statuses/f72f6aacb3c0bad6c83570fae57f3dd3ad9d8119"}}}}
{"actor":{"display_login":"mshriver"}}