{"action":"created","comment":{"url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/comments/84262401","pull_request_review_id":5049794,"id":84262401,"diff_hunk":"@@ -26,212 +26,396 @@\n ch.setFormatter(formatter)\n logger.addHandler(ch)\n \n-# set blank values so that they can be used later to clean environment\n-\n-image_id = \"\"\n-image_rootfs_path = \"\"\n+SCANNERS = {\n+        \"registry.centos.org/pipeline-images/pipeline-scanner\": [\n+            \"image_scan_results.json\"],\n+        \"registry.centos.org/pipeline-images/scanner-rpm-verify\": [\n+            \"RPMVerfiy.json\"]}","path":"beanstalk_worker/worker_start_scan.py","position":12,"original_position":12,"commit_id":"3d2ffd688985f0e280331e911f5e3a6c81ab874b","original_commit_id":"3d2ffd688985f0e280331e911f5e3a6c81ab874b","user":{"login":"navidshaikh","id":2510184,"avatar_url":"https://avatars.githubusercontent.com/u/2510184?v=3","gravatar_id":"","url":"https://api.github.com/users/navidshaikh","html_url":"https://github.com/navidshaikh","followers_url":"https://api.github.com/users/navidshaikh/followers","following_url":"https://api.github.com/users/navidshaikh/following{/other_user}","gists_url":"https://api.github.com/users/navidshaikh/gists{/gist_id}","starred_url":"https://api.github.com/users/navidshaikh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/navidshaikh/subscriptions","organizations_url":"https://api.github.com/users/navidshaikh/orgs","repos_url":"https://api.github.com/users/navidshaikh/repos","events_url":"https://api.github.com/users/navidshaikh/events{/privacy}","received_events_url":"https://api.github.com/users/navidshaikh/received_events","type":"User","site_admin":false},"body":"Add a lookup hash per scanner and their output filenames.","created_at":"2016-10-20T11:28:30Z","updated_at":"2016-10-20T11:28:30Z","html_url":"https://github.com/CentOS/container-pipeline-service/pull/57#discussion_r84262401","pull_request_url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57","_links":{"self":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/comments/84262401"},"html":{"href":"https://github.com/CentOS/container-pipeline-service/pull/57#discussion_r84262401"},"pull_request":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57"}}},"pull_request":{"url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57","id":84075557,"html_url":"https://github.com/CentOS/container-pipeline-service/pull/57","diff_url":"https://github.com/CentOS/container-pipeline-service/pull/57.diff","patch_url":"https://github.com/CentOS/container-pipeline-service/pull/57.patch","issue_url":"https://api.github.com/repos/CentOS/container-pipeline-service/issues/57","number":57,"state":"open","locked":false,"title":"[WIP] Fix #48 #56 : Consolidates the results from multiple scanners","user":{"login":"navidshaikh","id":2510184,"avatar_url":"https://avatars.githubusercontent.com/u/2510184?v=3","gravatar_id":"","url":"https://api.github.com/users/navidshaikh","html_url":"https://github.com/navidshaikh","followers_url":"https://api.github.com/users/navidshaikh/followers","following_url":"https://api.github.com/users/navidshaikh/following{/other_user}","gists_url":"https://api.github.com/users/navidshaikh/gists{/gist_id}","starred_url":"https://api.github.com/users/navidshaikh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/navidshaikh/subscriptions","organizations_url":"https://api.github.com/users/navidshaikh/orgs","repos_url":"https://api.github.com/users/navidshaikh/repos","events_url":"https://api.github.com/users/navidshaikh/events{/privacy}","received_events_url":"https://api.github.com/users/navidshaikh/received_events","type":"User","site_admin":false},"body":"  Fixes #56: Fixes verifying the results from pipeline scanner.\r\n  Now checks for actual result field from scanner and sets the\r\n  value for \"msg\" field accordingly.\r\n\r\n  Fixes #48: This commit also consolidates results from pipeline\r\n  scanner and scanner-rpm-verify and sends over tube in one go.\r\n\r\n  The data structure of the info sent over the message bus has changed.\r\n  The updates data structure for the output:\r\n\r\n  ```\r\n  d = {\r\n  \"image_under_test\": <full_image_name>,\r\n  \"action\": \"completed_scheduled_scan\",\r\n  \"namespace\": <job_data.namespace>,\r\n  \"msg\": {\r\n    \"scanner-rpm-verify\": \"<scanner_specific_message>\",\r\n    \"pipeine-scanner\": \"<scanner_specific_message>\"\r\n  },\r\n  \"logs\": {\r\n    \"scanner-rpm-verify\": {<scanner_json_data>},\r\n    \"pipeline-scanner\": {<scanner_json_data>}\r\n  }\r\n}\r\n  ```","created_at":"2016-09-06T07:52:31Z","updated_at":"2016-10-20T11:28:30Z","closed_at":null,"merged_at":null,"merge_commit_sha":"442e535b234314281529042c8aa44240cd3155cd","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57/commits","review_comments_url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57/comments","review_comment_url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/comments{/number}","comments_url":"https://api.github.com/repos/CentOS/container-pipeline-service/issues/57/comments","statuses_url":"https://api.github.com/repos/CentOS/container-pipeline-service/statuses/3d2ffd688985f0e280331e911f5e3a6c81ab874b","head":{"label":"navidshaikh:fix-48","ref":"fix-48","sha":"3d2ffd688985f0e280331e911f5e3a6c81ab874b","user":{"login":"navidshaikh","id":2510184,"avatar_url":"https://avatars.githubusercontent.com/u/2510184?v=3","gravatar_id":"","url":"https://api.github.com/users/navidshaikh","html_url":"https://github.com/navidshaikh","followers_url":"https://api.github.com/users/navidshaikh/followers","following_url":"https://api.github.com/users/navidshaikh/following{/other_user}","gists_url":"https://api.github.com/users/navidshaikh/gists{/gist_id}","starred_url":"https://api.github.com/users/navidshaikh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/navidshaikh/subscriptions","organizations_url":"https://api.github.com/users/navidshaikh/orgs","repos_url":"https://api.github.com/users/navidshaikh/repos","events_url":"https://api.github.com/users/navidshaikh/events{/privacy}","received_events_url":"https://api.github.com/users/navidshaikh/received_events","type":"User","site_admin":false},"repo":{"id":63863929,"name":"container-pipeline-service","full_name":"navidshaikh/container-pipeline-service","owner":{"login":"navidshaikh","id":2510184,"avatar_url":"https://avatars.githubusercontent.com/u/2510184?v=3","gravatar_id":"","url":"https://api.github.com/users/navidshaikh","html_url":"https://github.com/navidshaikh","followers_url":"https://api.github.com/users/navidshaikh/followers","following_url":"https://api.github.com/users/navidshaikh/following{/other_user}","gists_url":"https://api.github.com/users/navidshaikh/gists{/gist_id}","starred_url":"https://api.github.com/users/navidshaikh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/navidshaikh/subscriptions","organizations_url":"https://api.github.com/users/navidshaikh/orgs","repos_url":"https://api.github.com/users/navidshaikh/repos","events_url":"https://api.github.com/users/navidshaikh/events{/privacy}","received_events_url":"https://api.github.com/users/navidshaikh/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/navidshaikh/container-pipeline-service","description":"Code, infrastructure and deployment backend for the CentOS Container Pipeline","fork":true,"url":"https://api.github.com/repos/navidshaikh/container-pipeline-service","forks_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/forks","keys_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/keys{/key_id}","collaborators_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/teams","hooks_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/hooks","issue_events_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/issues/events{/number}","events_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/events","assignees_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/assignees{/user}","branches_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/branches{/branch}","tags_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/tags","blobs_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/git/refs{/sha}","trees_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/git/trees{/sha}","statuses_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/statuses/{sha}","languages_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/languages","stargazers_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/stargazers","contributors_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/contributors","subscribers_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/subscribers","subscription_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/subscription","commits_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/commits{/sha}","git_commits_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/git/commits{/sha}","comments_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/comments{/number}","issue_comment_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/issues/comments{/number}","contents_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/contents/{+path}","compare_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/compare/{base}...{head}","merges_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/merges","archive_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/downloads","issues_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/issues{/number}","pulls_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/pulls{/number}","milestones_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/milestones{/number}","notifications_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/labels{/name}","releases_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/releases{/id}","deployments_url":"https://api.github.com/repos/navidshaikh/container-pipeline-service/deployments","created_at":"2016-07-21T11:31:16Z","updated_at":"2016-07-21T11:31:17Z","pushed_at":"2016-10-20T11:26:19Z","git_url":"git://github.com/navidshaikh/container-pipeline-service.git","ssh_url":"git@github.com:navidshaikh/container-pipeline-service.git","clone_url":"https://github.com/navidshaikh/container-pipeline-service.git","svn_url":"https://github.com/navidshaikh/container-pipeline-service","homepage":null,"size":440,"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":"CentOS:master","ref":"master","sha":"9df4cdb1fe53d8245173938df503193bf4cd75d6","user":{"login":"CentOS","id":79192,"avatar_url":"https://avatars.githubusercontent.com/u/79192?v=3","gravatar_id":"","url":"https://api.github.com/users/CentOS","html_url":"https://github.com/CentOS","followers_url":"https://api.github.com/users/CentOS/followers","following_url":"https://api.github.com/users/CentOS/following{/other_user}","gists_url":"https://api.github.com/users/CentOS/gists{/gist_id}","starred_url":"https://api.github.com/users/CentOS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CentOS/subscriptions","organizations_url":"https://api.github.com/users/CentOS/orgs","repos_url":"https://api.github.com/users/CentOS/repos","events_url":"https://api.github.com/users/CentOS/events{/privacy}","received_events_url":"https://api.github.com/users/CentOS/received_events","type":"Organization","site_admin":false},"repo":{"id":63242015,"name":"container-pipeline-service","full_name":"CentOS/container-pipeline-service","owner":{"login":"CentOS","id":79192,"avatar_url":"https://avatars.githubusercontent.com/u/79192?v=3","gravatar_id":"","url":"https://api.github.com/users/CentOS","html_url":"https://github.com/CentOS","followers_url":"https://api.github.com/users/CentOS/followers","following_url":"https://api.github.com/users/CentOS/following{/other_user}","gists_url":"https://api.github.com/users/CentOS/gists{/gist_id}","starred_url":"https://api.github.com/users/CentOS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CentOS/subscriptions","organizations_url":"https://api.github.com/users/CentOS/orgs","repos_url":"https://api.github.com/users/CentOS/repos","events_url":"https://api.github.com/users/CentOS/events{/privacy}","received_events_url":"https://api.github.com/users/CentOS/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/CentOS/container-pipeline-service","description":"Code, infrastructure and deployment backend for the CentOS Container Pipeline","fork":false,"url":"https://api.github.com/repos/CentOS/container-pipeline-service","forks_url":"https://api.github.com/repos/CentOS/container-pipeline-service/forks","keys_url":"https://api.github.com/repos/CentOS/container-pipeline-service/keys{/key_id}","collaborators_url":"https://api.github.com/repos/CentOS/container-pipeline-service/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/CentOS/container-pipeline-service/teams","hooks_url":"https://api.github.com/repos/CentOS/container-pipeline-service/hooks","issue_events_url":"https://api.github.com/repos/CentOS/container-pipeline-service/issues/events{/number}","events_url":"https://api.github.com/repos/CentOS/container-pipeline-service/events","assignees_url":"https://api.github.com/repos/CentOS/container-pipeline-service/assignees{/user}","branches_url":"https://api.github.com/repos/CentOS/container-pipeline-service/branches{/branch}","tags_url":"https://api.github.com/repos/CentOS/container-pipeline-service/tags","blobs_url":"https://api.github.com/repos/CentOS/container-pipeline-service/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/CentOS/container-pipeline-service/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/CentOS/container-pipeline-service/git/refs{/sha}","trees_url":"https://api.github.com/repos/CentOS/container-pipeline-service/git/trees{/sha}","statuses_url":"https://api.github.com/repos/CentOS/container-pipeline-service/statuses/{sha}","languages_url":"https://api.github.com/repos/CentOS/container-pipeline-service/languages","stargazers_url":"https://api.github.com/repos/CentOS/container-pipeline-service/stargazers","contributors_url":"https://api.github.com/repos/CentOS/container-pipeline-service/contributors","subscribers_url":"https://api.github.com/repos/CentOS/container-pipeline-service/subscribers","subscription_url":"https://api.github.com/repos/CentOS/container-pipeline-service/subscription","commits_url":"https://api.github.com/repos/CentOS/container-pipeline-service/commits{/sha}","git_commits_url":"https://api.github.com/repos/CentOS/container-pipeline-service/git/commits{/sha}","comments_url":"https://api.github.com/repos/CentOS/container-pipeline-service/comments{/number}","issue_comment_url":"https://api.github.com/repos/CentOS/container-pipeline-service/issues/comments{/number}","contents_url":"https://api.github.com/repos/CentOS/container-pipeline-service/contents/{+path}","compare_url":"https://api.github.com/repos/CentOS/container-pipeline-service/compare/{base}...{head}","merges_url":"https://api.github.com/repos/CentOS/container-pipeline-service/merges","archive_url":"https://api.github.com/repos/CentOS/container-pipeline-service/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/CentOS/container-pipeline-service/downloads","issues_url":"https://api.github.com/repos/CentOS/container-pipeline-service/issues{/number}","pulls_url":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls{/number}","milestones_url":"https://api.github.com/repos/CentOS/container-pipeline-service/milestones{/number}","notifications_url":"https://api.github.com/repos/CentOS/container-pipeline-service/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/CentOS/container-pipeline-service/labels{/name}","releases_url":"https://api.github.com/repos/CentOS/container-pipeline-service/releases{/id}","deployments_url":"https://api.github.com/repos/CentOS/container-pipeline-service/deployments","created_at":"2016-07-13T11:55:19Z","updated_at":"2016-10-19T13:00:47Z","pushed_at":"2016-10-20T11:26:19Z","git_url":"git://github.com/CentOS/container-pipeline-service.git","ssh_url":"git@github.com:CentOS/container-pipeline-service.git","clone_url":"https://github.com/CentOS/container-pipeline-service.git","svn_url":"https://github.com/CentOS/container-pipeline-service","homepage":null,"size":479,"stargazers_count":7,"watchers_count":7,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":8,"mirror_url":null,"open_issues_count":22,"forks":8,"open_issues":22,"watchers":7,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57"},"html":{"href":"https://github.com/CentOS/container-pipeline-service/pull/57"},"issue":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/issues/57"},"comments":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/issues/57/comments"},"review_comments":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57/comments"},"review_comment":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/pulls/57/commits"},"statuses":{"href":"https://api.github.com/repos/CentOS/container-pipeline-service/statuses/3d2ffd688985f0e280331e911f5e3a6c81ab874b"}}}}
{"actor":{"display_login":"navidshaikh"}}