{"action":"closed","number":77,"pull_request":{"url":"https://api.github.com/repos/open-power/op-test-framework/pulls/77","id":88076047,"html_url":"https://github.com/open-power/op-test-framework/pull/77","diff_url":"https://github.com/open-power/op-test-framework/pull/77.diff","patch_url":"https://github.com/open-power/op-test-framework/pull/77.patch","issue_url":"https://api.github.com/repos/open-power/op-test-framework/issues/77","number":77,"state":"closed","locked":false,"title":"Effective execution of all the test cases.","user":{"login":"pridhiviraj","id":16423000,"avatar_url":"https://avatars.githubusercontent.com/u/16423000?v=3","gravatar_id":"","url":"https://api.github.com/users/pridhiviraj","html_url":"https://github.com/pridhiviraj","followers_url":"https://api.github.com/users/pridhiviraj/followers","following_url":"https://api.github.com/users/pridhiviraj/following{/other_user}","gists_url":"https://api.github.com/users/pridhiviraj/gists{/gist_id}","starred_url":"https://api.github.com/users/pridhiviraj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pridhiviraj/subscriptions","organizations_url":"https://api.github.com/users/pridhiviraj/orgs","repos_url":"https://api.github.com/users/pridhiviraj/repos","events_url":"https://api.github.com/users/pridhiviraj/events{/privacy}","received_events_url":"https://api.github.com/users/pridhiviraj/received_events","type":"User","site_admin":false},"body":"Some test cases will make the system to not to boot (https://github.com/open-power/op-build/issues/672)\r\nDue to that normal power off/on doesn't bring the system, we were end up in other simpler test cases to fail.\r\n\r\nThis patch makes existing function sys_bmc_power_on_validate_host to below changes.\r\nif system ping failed---> Do IPMI Power off--->Do BMC Cold reset----> Do IPMI Power ON---> wait for system to boot.\r\n\r\nAnd this sequence will executes only when the system is down and if it is up this function will be just\r\nbypassed.\r\n\r\nAnd this function added in all the test cases begining.\r\n\r\nSo now we have a effective execution of *most* of the test cases at the cost of *time*\r\n\r\nAny new test case addition just use this function at the beginning to execute that test case most of the times.\r\n\r\nStill if that function doesn't bring the system up, some thing goes wrong in BMC/Hostboot/HW level  ***bug***.\r\n\r\nSigned-off-by: Pridhiviraj Paidipeddi <ppaidipe@linux.vnet.ibm.com>\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/open-power/op-test-framework/77)\n<!-- Reviewable:end -->\n","created_at":"2016-10-05T13:19:42Z","updated_at":"2016-10-21T07:05:37Z","closed_at":"2016-10-21T07:05:36Z","merged_at":"2016-10-21T07:05:36Z","merge_commit_sha":"38f53d877042a44f80ac8474d2a4e0f13ed2aaf0","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/open-power/op-test-framework/pulls/77/commits","review_comments_url":"https://api.github.com/repos/open-power/op-test-framework/pulls/77/comments","review_comment_url":"https://api.github.com/repos/open-power/op-test-framework/pulls/comments{/number}","comments_url":"https://api.github.com/repos/open-power/op-test-framework/issues/77/comments","statuses_url":"https://api.github.com/repos/open-power/op-test-framework/statuses/c90edd82c17297e908fbc54acb1a11e690c13268","head":{"label":"pridhiviraj:fix_failures","ref":"fix_failures","sha":"c90edd82c17297e908fbc54acb1a11e690c13268","user":{"login":"pridhiviraj","id":16423000,"avatar_url":"https://avatars.githubusercontent.com/u/16423000?v=3","gravatar_id":"","url":"https://api.github.com/users/pridhiviraj","html_url":"https://github.com/pridhiviraj","followers_url":"https://api.github.com/users/pridhiviraj/followers","following_url":"https://api.github.com/users/pridhiviraj/following{/other_user}","gists_url":"https://api.github.com/users/pridhiviraj/gists{/gist_id}","starred_url":"https://api.github.com/users/pridhiviraj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pridhiviraj/subscriptions","organizations_url":"https://api.github.com/users/pridhiviraj/orgs","repos_url":"https://api.github.com/users/pridhiviraj/repos","events_url":"https://api.github.com/users/pridhiviraj/events{/privacy}","received_events_url":"https://api.github.com/users/pridhiviraj/received_events","type":"User","site_admin":false},"repo":{"id":48530608,"name":"op-test-framework","full_name":"pridhiviraj/op-test-framework","owner":{"login":"pridhiviraj","id":16423000,"avatar_url":"https://avatars.githubusercontent.com/u/16423000?v=3","gravatar_id":"","url":"https://api.github.com/users/pridhiviraj","html_url":"https://github.com/pridhiviraj","followers_url":"https://api.github.com/users/pridhiviraj/followers","following_url":"https://api.github.com/users/pridhiviraj/following{/other_user}","gists_url":"https://api.github.com/users/pridhiviraj/gists{/gist_id}","starred_url":"https://api.github.com/users/pridhiviraj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pridhiviraj/subscriptions","organizations_url":"https://api.github.com/users/pridhiviraj/orgs","repos_url":"https://api.github.com/users/pridhiviraj/repos","events_url":"https://api.github.com/users/pridhiviraj/events{/privacy}","received_events_url":"https://api.github.com/users/pridhiviraj/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/pridhiviraj/op-test-framework","description":null,"fork":true,"url":"https://api.github.com/repos/pridhiviraj/op-test-framework","forks_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/forks","keys_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/keys{/key_id}","collaborators_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/teams","hooks_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/hooks","issue_events_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/issues/events{/number}","events_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/events","assignees_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/assignees{/user}","branches_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/branches{/branch}","tags_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/tags","blobs_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/git/refs{/sha}","trees_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/git/trees{/sha}","statuses_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/statuses/{sha}","languages_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/languages","stargazers_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/stargazers","contributors_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/contributors","subscribers_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/subscribers","subscription_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/subscription","commits_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/commits{/sha}","git_commits_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/git/commits{/sha}","comments_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/comments{/number}","issue_comment_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/issues/comments{/number}","contents_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/contents/{+path}","compare_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/compare/{base}...{head}","merges_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/merges","archive_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/downloads","issues_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/issues{/number}","pulls_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/pulls{/number}","milestones_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/milestones{/number}","notifications_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/labels{/name}","releases_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/releases{/id}","deployments_url":"https://api.github.com/repos/pridhiviraj/op-test-framework/deployments","created_at":"2015-12-24T07:18:08Z","updated_at":"2016-01-14T07:44:38Z","pushed_at":"2016-10-21T06:40:51Z","git_url":"git://github.com/pridhiviraj/op-test-framework.git","ssh_url":"git@github.com:pridhiviraj/op-test-framework.git","clone_url":"https://github.com/pridhiviraj/op-test-framework.git","svn_url":"https://github.com/pridhiviraj/op-test-framework","homepage":null,"size":590,"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":"open-power:master","ref":"master","sha":"3d77a8a73c6e976e7c99889256761c93e205751b","user":{"login":"open-power","id":8000311,"avatar_url":"https://avatars.githubusercontent.com/u/8000311?v=3","gravatar_id":"","url":"https://api.github.com/users/open-power","html_url":"https://github.com/open-power","followers_url":"https://api.github.com/users/open-power/followers","following_url":"https://api.github.com/users/open-power/following{/other_user}","gists_url":"https://api.github.com/users/open-power/gists{/gist_id}","starred_url":"https://api.github.com/users/open-power/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/open-power/subscriptions","organizations_url":"https://api.github.com/users/open-power/orgs","repos_url":"https://api.github.com/users/open-power/repos","events_url":"https://api.github.com/users/open-power/events{/privacy}","received_events_url":"https://api.github.com/users/open-power/received_events","type":"Organization","site_admin":false},"repo":{"id":41101572,"name":"op-test-framework","full_name":"open-power/op-test-framework","owner":{"login":"open-power","id":8000311,"avatar_url":"https://avatars.githubusercontent.com/u/8000311?v=3","gravatar_id":"","url":"https://api.github.com/users/open-power","html_url":"https://github.com/open-power","followers_url":"https://api.github.com/users/open-power/followers","following_url":"https://api.github.com/users/open-power/following{/other_user}","gists_url":"https://api.github.com/users/open-power/gists{/gist_id}","starred_url":"https://api.github.com/users/open-power/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/open-power/subscriptions","organizations_url":"https://api.github.com/users/open-power/orgs","repos_url":"https://api.github.com/users/open-power/repos","events_url":"https://api.github.com/users/open-power/events{/privacy}","received_events_url":"https://api.github.com/users/open-power/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/open-power/op-test-framework","description":null,"fork":false,"url":"https://api.github.com/repos/open-power/op-test-framework","forks_url":"https://api.github.com/repos/open-power/op-test-framework/forks","keys_url":"https://api.github.com/repos/open-power/op-test-framework/keys{/key_id}","collaborators_url":"https://api.github.com/repos/open-power/op-test-framework/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/open-power/op-test-framework/teams","hooks_url":"https://api.github.com/repos/open-power/op-test-framework/hooks","issue_events_url":"https://api.github.com/repos/open-power/op-test-framework/issues/events{/number}","events_url":"https://api.github.com/repos/open-power/op-test-framework/events","assignees_url":"https://api.github.com/repos/open-power/op-test-framework/assignees{/user}","branches_url":"https://api.github.com/repos/open-power/op-test-framework/branches{/branch}","tags_url":"https://api.github.com/repos/open-power/op-test-framework/tags","blobs_url":"https://api.github.com/repos/open-power/op-test-framework/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/open-power/op-test-framework/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/open-power/op-test-framework/git/refs{/sha}","trees_url":"https://api.github.com/repos/open-power/op-test-framework/git/trees{/sha}","statuses_url":"https://api.github.com/repos/open-power/op-test-framework/statuses/{sha}","languages_url":"https://api.github.com/repos/open-power/op-test-framework/languages","stargazers_url":"https://api.github.com/repos/open-power/op-test-framework/stargazers","contributors_url":"https://api.github.com/repos/open-power/op-test-framework/contributors","subscribers_url":"https://api.github.com/repos/open-power/op-test-framework/subscribers","subscription_url":"https://api.github.com/repos/open-power/op-test-framework/subscription","commits_url":"https://api.github.com/repos/open-power/op-test-framework/commits{/sha}","git_commits_url":"https://api.github.com/repos/open-power/op-test-framework/git/commits{/sha}","comments_url":"https://api.github.com/repos/open-power/op-test-framework/comments{/number}","issue_comment_url":"https://api.github.com/repos/open-power/op-test-framework/issues/comments{/number}","contents_url":"https://api.github.com/repos/open-power/op-test-framework/contents/{+path}","compare_url":"https://api.github.com/repos/open-power/op-test-framework/compare/{base}...{head}","merges_url":"https://api.github.com/repos/open-power/op-test-framework/merges","archive_url":"https://api.github.com/repos/open-power/op-test-framework/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/open-power/op-test-framework/downloads","issues_url":"https://api.github.com/repos/open-power/op-test-framework/issues{/number}","pulls_url":"https://api.github.com/repos/open-power/op-test-framework/pulls{/number}","milestones_url":"https://api.github.com/repos/open-power/op-test-framework/milestones{/number}","notifications_url":"https://api.github.com/repos/open-power/op-test-framework/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/open-power/op-test-framework/labels{/name}","releases_url":"https://api.github.com/repos/open-power/op-test-framework/releases{/id}","deployments_url":"https://api.github.com/repos/open-power/op-test-framework/deployments","created_at":"2015-08-20T14:51:40Z","updated_at":"2016-09-27T05:38:50Z","pushed_at":"2016-10-21T07:05:37Z","git_url":"git://github.com/open-power/op-test-framework.git","ssh_url":"git@github.com:open-power/op-test-framework.git","clone_url":"https://github.com/open-power/op-test-framework.git","svn_url":"https://github.com/open-power/op-test-framework","homepage":null,"size":1103,"stargazers_count":6,"watchers_count":6,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":18,"mirror_url":null,"open_issues_count":7,"forks":18,"open_issues":7,"watchers":6,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/open-power/op-test-framework/pulls/77"},"html":{"href":"https://github.com/open-power/op-test-framework/pull/77"},"issue":{"href":"https://api.github.com/repos/open-power/op-test-framework/issues/77"},"comments":{"href":"https://api.github.com/repos/open-power/op-test-framework/issues/77/comments"},"review_comments":{"href":"https://api.github.com/repos/open-power/op-test-framework/pulls/77/comments"},"review_comment":{"href":"https://api.github.com/repos/open-power/op-test-framework/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/open-power/op-test-framework/pulls/77/commits"},"statuses":{"href":"https://api.github.com/repos/open-power/op-test-framework/statuses/c90edd82c17297e908fbc54acb1a11e690c13268"}},"merged":true,"mergeable":null,"mergeable_state":"unknown","merged_by":{"login":"stewart-ibm","id":6668672,"avatar_url":"https://avatars.githubusercontent.com/u/6668672?v=3","gravatar_id":"","url":"https://api.github.com/users/stewart-ibm","html_url":"https://github.com/stewart-ibm","followers_url":"https://api.github.com/users/stewart-ibm/followers","following_url":"https://api.github.com/users/stewart-ibm/following{/other_user}","gists_url":"https://api.github.com/users/stewart-ibm/gists{/gist_id}","starred_url":"https://api.github.com/users/stewart-ibm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stewart-ibm/subscriptions","organizations_url":"https://api.github.com/users/stewart-ibm/orgs","repos_url":"https://api.github.com/users/stewart-ibm/repos","events_url":"https://api.github.com/users/stewart-ibm/events{/privacy}","received_events_url":"https://api.github.com/users/stewart-ibm/received_events","type":"User","site_admin":false},"comments":1,"review_comments":0,"commits":2,"additions":111,"deletions":32,"changed_files":24}}
{"actor":{"display_login":"stewart-ibm"}}