{"action":"created","comment":{"url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/comments/89276442","pull_request_review_id":9822319,"id":89276442,"diff_hunk":"@@ -167,17 +169,20 @@ def product_upgrade(product):\n     if check_necessary_env_variables_for_upgrade(product):\n         sat_host, cap_hosts, clients6, clients7 = setup_products_for_upgrade(\n             product, os.environ.get('OS'))\n-        execute(satellite6_upgrade, host=sat_host)\n-        # Generate foreman debug on satellite after upgrade\n-        execute(foreman_debug, 'satellite_{}'.format(sat_host), host=sat_host)\n-        if product == 'capsule':\n-            for cap_host in cap_hosts:\n-                execute(satellite6_capsule_upgrade, cap_host, host=cap_host)\n-                # Generate foreman debug on capsule after upgrade\n-                execute(\n-                    foreman_debug,\n-                    'capsule_{}'.format(cap_host),\n-                    host=cap_host)\n-        if product == 'client':\n-            satellite6_client_upgrade('rhel6', clients6)\n-            satellite6_client_upgrade('rhel7', clients7)\n+        with LogAnalyzer(sat_host):","path":"automation_tools/satellite6/upgrade/__init__.py","position":44,"original_position":44,"commit_id":"cb0dec3d1455eb4455dac3e2fd356638f876d699","original_commit_id":"cb0dec3d1455eb4455dac3e2fd356638f876d699","user":{"login":"jyejare","id":11752425,"avatar_url":"https://avatars.githubusercontent.com/u/11752425?v=3","gravatar_id":"","url":"https://api.github.com/users/jyejare","html_url":"https://github.com/jyejare","followers_url":"https://api.github.com/users/jyejare/followers","following_url":"https://api.github.com/users/jyejare/following{/other_user}","gists_url":"https://api.github.com/users/jyejare/gists{/gist_id}","starred_url":"https://api.github.com/users/jyejare/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jyejare/subscriptions","organizations_url":"https://api.github.com/users/jyejare/orgs","repos_url":"https://api.github.com/users/jyejare/repos","events_url":"https://api.github.com/users/jyejare/events{/privacy}","received_events_url":"https://api.github.com/users/jyejare/received_events","type":"User","site_admin":false},"body":"Repeat the LogAnalyzer context manager on sat_host(which you are doing already) and on cap_host.\r\nI will take care about clients as they are situated in Docker and the whole process is bit complex.","created_at":"2016-11-23T09:12:05Z","updated_at":"2016-11-23T09:12:05Z","html_url":"https://github.com/SatelliteQE/automation-tools/pull/459#discussion_r89276442","pull_request_url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459","_links":{"self":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/comments/89276442"},"html":{"href":"https://github.com/SatelliteQE/automation-tools/pull/459#discussion_r89276442"},"pull_request":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459"}}},"pull_request":{"url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459","id":94888785,"html_url":"https://github.com/SatelliteQE/automation-tools/pull/459","diff_url":"https://github.com/SatelliteQE/automation-tools/pull/459.diff","patch_url":"https://github.com/SatelliteQE/automation-tools/pull/459.patch","issue_url":"https://api.github.com/repos/SatelliteQE/automation-tools/issues/459","number":459,"state":"open","locked":false,"title":"Added LogAnalyzer for checking changes on Satellite log files","user":{"login":"renzon","id":3457115,"avatar_url":"https://avatars.githubusercontent.com/u/3457115?v=3","gravatar_id":"","url":"https://api.github.com/users/renzon","html_url":"https://github.com/renzon","followers_url":"https://api.github.com/users/renzon/followers","following_url":"https://api.github.com/users/renzon/following{/other_user}","gists_url":"https://api.github.com/users/renzon/gists{/gist_id}","starred_url":"https://api.github.com/users/renzon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/renzon/subscriptions","organizations_url":"https://api.github.com/users/renzon/orgs","repos_url":"https://api.github.com/users/renzon/repos","events_url":"https://api.github.com/users/renzon/events{/privacy}","received_events_url":"https://api.github.com/users/renzon/received_events","type":"User","site_admin":false},"body":"close #447\r\n\r\nContext Manager was added to upgrade task","created_at":"2016-11-22T20:09:10Z","updated_at":"2016-11-23T09:12:05Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e4c6f2913855c2e99eb1c8abf7e9f2af5dc5eebe","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459/commits","review_comments_url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459/comments","review_comment_url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/comments{/number}","comments_url":"https://api.github.com/repos/SatelliteQE/automation-tools/issues/459/comments","statuses_url":"https://api.github.com/repos/SatelliteQE/automation-tools/statuses/cb0dec3d1455eb4455dac3e2fd356638f876d699","head":{"label":"renzon:447","ref":"447","sha":"cb0dec3d1455eb4455dac3e2fd356638f876d699","user":{"login":"renzon","id":3457115,"avatar_url":"https://avatars.githubusercontent.com/u/3457115?v=3","gravatar_id":"","url":"https://api.github.com/users/renzon","html_url":"https://github.com/renzon","followers_url":"https://api.github.com/users/renzon/followers","following_url":"https://api.github.com/users/renzon/following{/other_user}","gists_url":"https://api.github.com/users/renzon/gists{/gist_id}","starred_url":"https://api.github.com/users/renzon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/renzon/subscriptions","organizations_url":"https://api.github.com/users/renzon/orgs","repos_url":"https://api.github.com/users/renzon/repos","events_url":"https://api.github.com/users/renzon/events{/privacy}","received_events_url":"https://api.github.com/users/renzon/received_events","type":"User","site_admin":false},"repo":{"id":74361173,"name":"automation-tools","full_name":"renzon/automation-tools","owner":{"login":"renzon","id":3457115,"avatar_url":"https://avatars.githubusercontent.com/u/3457115?v=3","gravatar_id":"","url":"https://api.github.com/users/renzon","html_url":"https://github.com/renzon","followers_url":"https://api.github.com/users/renzon/followers","following_url":"https://api.github.com/users/renzon/following{/other_user}","gists_url":"https://api.github.com/users/renzon/gists{/gist_id}","starred_url":"https://api.github.com/users/renzon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/renzon/subscriptions","organizations_url":"https://api.github.com/users/renzon/orgs","repos_url":"https://api.github.com/users/renzon/repos","events_url":"https://api.github.com/users/renzon/events{/privacy}","received_events_url":"https://api.github.com/users/renzon/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/renzon/automation-tools","description":"A set of tools to help automating testing Foreman with Robottelo (https://github.com/SatelliteQE/robottelo)","fork":true,"url":"https://api.github.com/repos/renzon/automation-tools","forks_url":"https://api.github.com/repos/renzon/automation-tools/forks","keys_url":"https://api.github.com/repos/renzon/automation-tools/keys{/key_id}","collaborators_url":"https://api.github.com/repos/renzon/automation-tools/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/renzon/automation-tools/teams","hooks_url":"https://api.github.com/repos/renzon/automation-tools/hooks","issue_events_url":"https://api.github.com/repos/renzon/automation-tools/issues/events{/number}","events_url":"https://api.github.com/repos/renzon/automation-tools/events","assignees_url":"https://api.github.com/repos/renzon/automation-tools/assignees{/user}","branches_url":"https://api.github.com/repos/renzon/automation-tools/branches{/branch}","tags_url":"https://api.github.com/repos/renzon/automation-tools/tags","blobs_url":"https://api.github.com/repos/renzon/automation-tools/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/renzon/automation-tools/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/renzon/automation-tools/git/refs{/sha}","trees_url":"https://api.github.com/repos/renzon/automation-tools/git/trees{/sha}","statuses_url":"https://api.github.com/repos/renzon/automation-tools/statuses/{sha}","languages_url":"https://api.github.com/repos/renzon/automation-tools/languages","stargazers_url":"https://api.github.com/repos/renzon/automation-tools/stargazers","contributors_url":"https://api.github.com/repos/renzon/automation-tools/contributors","subscribers_url":"https://api.github.com/repos/renzon/automation-tools/subscribers","subscription_url":"https://api.github.com/repos/renzon/automation-tools/subscription","commits_url":"https://api.github.com/repos/renzon/automation-tools/commits{/sha}","git_commits_url":"https://api.github.com/repos/renzon/automation-tools/git/commits{/sha}","comments_url":"https://api.github.com/repos/renzon/automation-tools/comments{/number}","issue_comment_url":"https://api.github.com/repos/renzon/automation-tools/issues/comments{/number}","contents_url":"https://api.github.com/repos/renzon/automation-tools/contents/{+path}","compare_url":"https://api.github.com/repos/renzon/automation-tools/compare/{base}...{head}","merges_url":"https://api.github.com/repos/renzon/automation-tools/merges","archive_url":"https://api.github.com/repos/renzon/automation-tools/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/renzon/automation-tools/downloads","issues_url":"https://api.github.com/repos/renzon/automation-tools/issues{/number}","pulls_url":"https://api.github.com/repos/renzon/automation-tools/pulls{/number}","milestones_url":"https://api.github.com/repos/renzon/automation-tools/milestones{/number}","notifications_url":"https://api.github.com/repos/renzon/automation-tools/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/renzon/automation-tools/labels{/name}","releases_url":"https://api.github.com/repos/renzon/automation-tools/releases{/id}","deployments_url":"https://api.github.com/repos/renzon/automation-tools/deployments","created_at":"2016-11-21T12:25:12Z","updated_at":"2016-11-21T12:25:14Z","pushed_at":"2016-11-22T20:15:47Z","git_url":"git://github.com/renzon/automation-tools.git","ssh_url":"git@github.com:renzon/automation-tools.git","clone_url":"https://github.com/renzon/automation-tools.git","svn_url":"https://github.com/renzon/automation-tools","homepage":"","size":811,"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":"SatelliteQE:master","ref":"master","sha":"4297a6291478cfadd6bf0c857414c5cf74c4275b","user":{"login":"SatelliteQE","id":8754276,"avatar_url":"https://avatars.githubusercontent.com/u/8754276?v=3","gravatar_id":"","url":"https://api.github.com/users/SatelliteQE","html_url":"https://github.com/SatelliteQE","followers_url":"https://api.github.com/users/SatelliteQE/followers","following_url":"https://api.github.com/users/SatelliteQE/following{/other_user}","gists_url":"https://api.github.com/users/SatelliteQE/gists{/gist_id}","starred_url":"https://api.github.com/users/SatelliteQE/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SatelliteQE/subscriptions","organizations_url":"https://api.github.com/users/SatelliteQE/orgs","repos_url":"https://api.github.com/users/SatelliteQE/repos","events_url":"https://api.github.com/users/SatelliteQE/events{/privacy}","received_events_url":"https://api.github.com/users/SatelliteQE/received_events","type":"Organization","site_admin":false},"repo":{"id":22513327,"name":"automation-tools","full_name":"SatelliteQE/automation-tools","owner":{"login":"SatelliteQE","id":8754276,"avatar_url":"https://avatars.githubusercontent.com/u/8754276?v=3","gravatar_id":"","url":"https://api.github.com/users/SatelliteQE","html_url":"https://github.com/SatelliteQE","followers_url":"https://api.github.com/users/SatelliteQE/followers","following_url":"https://api.github.com/users/SatelliteQE/following{/other_user}","gists_url":"https://api.github.com/users/SatelliteQE/gists{/gist_id}","starred_url":"https://api.github.com/users/SatelliteQE/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SatelliteQE/subscriptions","organizations_url":"https://api.github.com/users/SatelliteQE/orgs","repos_url":"https://api.github.com/users/SatelliteQE/repos","events_url":"https://api.github.com/users/SatelliteQE/events{/privacy}","received_events_url":"https://api.github.com/users/SatelliteQE/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/SatelliteQE/automation-tools","description":"A set of tools to help automating testing Foreman with Robottelo (https://github.com/SatelliteQE/robottelo)","fork":false,"url":"https://api.github.com/repos/SatelliteQE/automation-tools","forks_url":"https://api.github.com/repos/SatelliteQE/automation-tools/forks","keys_url":"https://api.github.com/repos/SatelliteQE/automation-tools/keys{/key_id}","collaborators_url":"https://api.github.com/repos/SatelliteQE/automation-tools/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/SatelliteQE/automation-tools/teams","hooks_url":"https://api.github.com/repos/SatelliteQE/automation-tools/hooks","issue_events_url":"https://api.github.com/repos/SatelliteQE/automation-tools/issues/events{/number}","events_url":"https://api.github.com/repos/SatelliteQE/automation-tools/events","assignees_url":"https://api.github.com/repos/SatelliteQE/automation-tools/assignees{/user}","branches_url":"https://api.github.com/repos/SatelliteQE/automation-tools/branches{/branch}","tags_url":"https://api.github.com/repos/SatelliteQE/automation-tools/tags","blobs_url":"https://api.github.com/repos/SatelliteQE/automation-tools/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/SatelliteQE/automation-tools/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/SatelliteQE/automation-tools/git/refs{/sha}","trees_url":"https://api.github.com/repos/SatelliteQE/automation-tools/git/trees{/sha}","statuses_url":"https://api.github.com/repos/SatelliteQE/automation-tools/statuses/{sha}","languages_url":"https://api.github.com/repos/SatelliteQE/automation-tools/languages","stargazers_url":"https://api.github.com/repos/SatelliteQE/automation-tools/stargazers","contributors_url":"https://api.github.com/repos/SatelliteQE/automation-tools/contributors","subscribers_url":"https://api.github.com/repos/SatelliteQE/automation-tools/subscribers","subscription_url":"https://api.github.com/repos/SatelliteQE/automation-tools/subscription","commits_url":"https://api.github.com/repos/SatelliteQE/automation-tools/commits{/sha}","git_commits_url":"https://api.github.com/repos/SatelliteQE/automation-tools/git/commits{/sha}","comments_url":"https://api.github.com/repos/SatelliteQE/automation-tools/comments{/number}","issue_comment_url":"https://api.github.com/repos/SatelliteQE/automation-tools/issues/comments{/number}","contents_url":"https://api.github.com/repos/SatelliteQE/automation-tools/contents/{+path}","compare_url":"https://api.github.com/repos/SatelliteQE/automation-tools/compare/{base}...{head}","merges_url":"https://api.github.com/repos/SatelliteQE/automation-tools/merges","archive_url":"https://api.github.com/repos/SatelliteQE/automation-tools/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/SatelliteQE/automation-tools/downloads","issues_url":"https://api.github.com/repos/SatelliteQE/automation-tools/issues{/number}","pulls_url":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls{/number}","milestones_url":"https://api.github.com/repos/SatelliteQE/automation-tools/milestones{/number}","notifications_url":"https://api.github.com/repos/SatelliteQE/automation-tools/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/SatelliteQE/automation-tools/labels{/name}","releases_url":"https://api.github.com/repos/SatelliteQE/automation-tools/releases{/id}","deployments_url":"https://api.github.com/repos/SatelliteQE/automation-tools/deployments","created_at":"2014-08-01T14:27:34Z","updated_at":"2016-11-04T21:23:39Z","pushed_at":"2016-11-22T20:15:48Z","git_url":"git://github.com/SatelliteQE/automation-tools.git","ssh_url":"git@github.com:SatelliteQE/automation-tools.git","clone_url":"https://github.com/SatelliteQE/automation-tools.git","svn_url":"https://github.com/SatelliteQE/automation-tools","homepage":"","size":1085,"stargazers_count":8,"watchers_count":8,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":20,"mirror_url":null,"open_issues_count":18,"forks":20,"open_issues":18,"watchers":8,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459"},"html":{"href":"https://github.com/SatelliteQE/automation-tools/pull/459"},"issue":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/issues/459"},"comments":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/issues/459/comments"},"review_comments":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459/comments"},"review_comment":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/pulls/459/commits"},"statuses":{"href":"https://api.github.com/repos/SatelliteQE/automation-tools/statuses/cb0dec3d1455eb4455dac3e2fd356638f876d699"}}}}
{"actor":{"display_login":"jyejare"}}