GitHub PullRequestReviewCommentEvent at 2014-09-09 22:07:13
Repository: socorro
Actor: peterbe
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/mozilla/socorro/pulls/comments/17331569","id":17331569,"diff_hunk":"@@ -103,6 +103,11 @@ def parseBuildJsonFile(url, nightly=False):\n         # due to a quirk in our build system\n         except ValueError:\n             pass\n+        # bug 1065071 - there are some JSON files that we\n+        # don't currently ingest.\n+        except KeyError:\n+            print 'warning, unsupported JSON file: %s' % url","path":"socorro/cron/jobs/ftpscraper.py","position":7,"original_position":7,"commit_id":"836d257f81870f470350ba093273013c2de44585","original_commit_id":"050cc6ab9b1e376e98f7f13bc0adb70fb75474df","user":{"login":"peterbe","id":26739,"avatar_url":"https://avatars.githubusercontent.com/u/26739?v=2","gravatar_id":"37d081c393f95a14e2704af38ecc4c8d","url":"https://api.github.com/users/peterbe","html_url":"https://github.com/peterbe","followers_url":"https://api.github.com/users/peterbe/followers","following_url":"https://api.github.com/users/peterbe/following{/other_user}","gists_url":"https://api.github.com/users/peterbe/gists{/gist_id}","starred_url":"https://api.github.com/users/peterbe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peterbe/subscriptions","organizations_url":"https://api.github.com/users/peterbe/orgs","repos_url":"https://api.github.com/users/peterbe/repos","events_url":"https://api.github.com/users/peterbe/events{/privacy}","received_events_url":"https://api.github.com/users/peterbe/received_events","type":"User","site_admin":false},"body":"...in case it's there but an empty string. ","created_at":"2014-09-09T22:07:13Z","updated_at":"2014-09-09T22:07:13Z","html_url":"https://github.com/mozilla/socorro/pull/2346#discussion_r17331569","pull_request_url":"https://api.github.com/repos/mozilla/socorro/pulls/2346","_links":{"self":{"href":"https://api.github.com/repos/mozilla/socorro/pulls/comments/17331569"},"html":{"href":"https://github.com/mozilla/socorro/pull/2346#discussion_r17331569"},"pull_request":{"href":"https://api.github.com/repos/mozilla/socorro/pulls/2346"}}},"pull_request":{"url":"https://api.github.com/repos/mozilla/socorro/pulls/2346","id":20986484,"html_url":"https://github.com/mozilla/socorro/pull/2346","diff_url":"https://github.com/mozilla/socorro/pull/2346.diff","patch_url":"https://github.com/mozilla/socorro/pull/2346.patch","issue_url":"https://api.github.com/repos/mozilla/socorro/issues/2346","number":2346,"state":"open","locked":false,"title":"fix bug 1065071 - ignore unrecognizable JSON files","user":{"login":"rhelmer","id":61412,"avatar_url":"https://avatars.githubusercontent.com/u/61412?v=2","gravatar_id":"aedfcfd22f63c0f0a6b3ce6b9024df9e","url":"https://api.github.com/users/rhelmer","html_url":"https://github.com/rhelmer","followers_url":"https://api.github.com/users/rhelmer/followers","following_url":"https://api.github.com/users/rhelmer/following{/other_user}","gists_url":"https://api.github.com/users/rhelmer/gists{/gist_id}","starred_url":"https://api.github.com/users/rhelmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhelmer/subscriptions","organizations_url":"https://api.github.com/users/rhelmer/orgs","repos_url":"https://api.github.com/users/rhelmer/repos","events_url":"https://api.github.com/users/rhelmer/events{/privacy}","received_events_url":"https://api.github.com/users/rhelmer/received_events","type":"User","site_admin":false},"body":"","created_at":"2014-09-09T21:55:52Z","updated_at":"2014-09-09T22:07:13Z","closed_at":null,"merged_at":null,"merge_commit_sha":"bced04c35349c463fd3495809ebdd7096e9ac67f","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/mozilla/socorro/pulls/2346/commits","review_comments_url":"https://api.github.com/repos/mozilla/socorro/pulls/2346/comments","review_comment_url":"https://api.github.com/repos/mozilla/socorro/pulls/comments/{number}","comments_url":"https://api.github.com/repos/mozilla/socorro/issues/2346/comments","statuses_url":"https://api.github.com/repos/mozilla/socorro/statuses/836d257f81870f470350ba093273013c2de44585","head":{"label":"rhelmer:bug1065071-ignore-unrecognizable-JSON-files","ref":"bug1065071-ignore-unrecognizable-JSON-files","sha":"836d257f81870f470350ba093273013c2de44585","user":{"login":"rhelmer","id":61412,"avatar_url":"https://avatars.githubusercontent.com/u/61412?v=2","gravatar_id":"aedfcfd22f63c0f0a6b3ce6b9024df9e","url":"https://api.github.com/users/rhelmer","html_url":"https://github.com/rhelmer","followers_url":"https://api.github.com/users/rhelmer/followers","following_url":"https://api.github.com/users/rhelmer/following{/other_user}","gists_url":"https://api.github.com/users/rhelmer/gists{/gist_id}","starred_url":"https://api.github.com/users/rhelmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhelmer/subscriptions","organizations_url":"https://api.github.com/users/rhelmer/orgs","repos_url":"https://api.github.com/users/rhelmer/repos","events_url":"https://api.github.com/users/rhelmer/events{/privacy}","received_events_url":"https://api.github.com/users/rhelmer/received_events","type":"User","site_admin":false},"repo":{"id":4216256,"name":"socorro","full_name":"rhelmer/socorro","owner":{"login":"rhelmer","id":61412,"avatar_url":"https://avatars.githubusercontent.com/u/61412?v=2","gravatar_id":"aedfcfd22f63c0f0a6b3ce6b9024df9e","url":"https://api.github.com/users/rhelmer","html_url":"https://github.com/rhelmer","followers_url":"https://api.github.com/users/rhelmer/followers","following_url":"https://api.github.com/users/rhelmer/following{/other_user}","gists_url":"https://api.github.com/users/rhelmer/gists{/gist_id}","starred_url":"https://api.github.com/users/rhelmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhelmer/subscriptions","organizations_url":"https://api.github.com/users/rhelmer/orgs","repos_url":"https://api.github.com/users/rhelmer/repos","events_url":"https://api.github.com/users/rhelmer/events{/privacy}","received_events_url":"https://api.github.com/users/rhelmer/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/rhelmer/socorro","description":"Socorro is a server to accept and process Breakpad crash reports. ","fork":true,"url":"https://api.github.com/repos/rhelmer/socorro","forks_url":"https://api.github.com/repos/rhelmer/socorro/forks","keys_url":"https://api.github.com/repos/rhelmer/socorro/keys{/key_id}","collaborators_url":"https://api.github.com/repos/rhelmer/socorro/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/rhelmer/socorro/teams","hooks_url":"https://api.github.com/repos/rhelmer/socorro/hooks","issue_events_url":"https://api.github.com/repos/rhelmer/socorro/issues/events{/number}","events_url":"https://api.github.com/repos/rhelmer/socorro/events","assignees_url":"https://api.github.com/repos/rhelmer/socorro/assignees{/user}","branches_url":"https://api.github.com/repos/rhelmer/socorro/branches{/branch}","tags_url":"https://api.github.com/repos/rhelmer/socorro/tags","blobs_url":"https://api.github.com/repos/rhelmer/socorro/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/rhelmer/socorro/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/rhelmer/socorro/git/refs{/sha}","trees_url":"https://api.github.com/repos/rhelmer/socorro/git/trees{/sha}","statuses_url":"https://api.github.com/repos/rhelmer/socorro/statuses/{sha}","languages_url":"https://api.github.com/repos/rhelmer/socorro/languages","stargazers_url":"https://api.github.com/repos/rhelmer/socorro/stargazers","contributors_url":"https://api.github.com/repos/rhelmer/socorro/contributors","subscribers_url":"https://api.github.com/repos/rhelmer/socorro/subscribers","subscription_url":"https://api.github.com/repos/rhelmer/socorro/subscription","commits_url":"https://api.github.com/repos/rhelmer/socorro/commits{/sha}","git_commits_url":"https://api.github.com/repos/rhelmer/socorro/git/commits{/sha}","comments_url":"https://api.github.com/repos/rhelmer/socorro/comments{/number}","issue_comment_url":"https://api.github.com/repos/rhelmer/socorro/issues/comments/{number}","contents_url":"https://api.github.com/repos/rhelmer/socorro/contents/{+path}","compare_url":"https://api.github.com/repos/rhelmer/socorro/compare/{base}...{head}","merges_url":"https://api.github.com/repos/rhelmer/socorro/merges","archive_url":"https://api.github.com/repos/rhelmer/socorro/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/rhelmer/socorro/downloads","issues_url":"https://api.github.com/repos/rhelmer/socorro/issues{/number}","pulls_url":"https://api.github.com/repos/rhelmer/socorro/pulls{/number}","milestones_url":"https://api.github.com/repos/rhelmer/socorro/milestones{/number}","notifications_url":"https://api.github.com/repos/rhelmer/socorro/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/rhelmer/socorro/labels{/name}","releases_url":"https://api.github.com/repos/rhelmer/socorro/releases{/id}","created_at":"2012-05-03T16:46:07Z","updated_at":"2014-07-08T17:49:17Z","pushed_at":"2014-09-09T22:06:45Z","git_url":"git://github.com/rhelmer/socorro.git","ssh_url":"git@github.com:rhelmer/socorro.git","clone_url":"https://github.com/rhelmer/socorro.git","svn_url":"https://github.com/rhelmer/socorro","homepage":"https://wiki.mozilla.org/Socorro","size":237250,"stargazers_count":1,"watchers_count":1,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":1,"default_branch":"master"}},"base":{"label":"mozilla:master","ref":"master","sha":"fe30e984ff28e80c9abb7a51ce73951b913e8485","user":{"login":"mozilla","id":131524,"avatar_url":"https://avatars.githubusercontent.com/u/131524?v=2","gravatar_id":"9ba739c8f3288e256b13553e4d257b5e","url":"https://api.github.com/users/mozilla","html_url":"https://github.com/mozilla","followers_url":"https://api.github.com/users/mozilla/followers","following_url":"https://api.github.com/users/mozilla/following{/other_user}","gists_url":"https://api.github.com/users/mozilla/gists{/gist_id}","starred_url":"https://api.github.com/users/mozilla/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mozilla/subscriptions","organizations_url":"https://api.github.com/users/mozilla/orgs","repos_url":"https://api.github.com/users/mozilla/repos","events_url":"https://api.github.com/users/mozilla/events{/privacy}","received_events_url":"https://api.github.com/users/mozilla/received_events","type":"Organization","site_admin":false},"repo":{"id":2263742,"name":"socorro","full_name":"mozilla/socorro","owner":{"login":"mozilla","id":131524,"avatar_url":"https://avatars.githubusercontent.com/u/131524?v=2","gravatar_id":"9ba739c8f3288e256b13553e4d257b5e","url":"https://api.github.com/users/mozilla","html_url":"https://github.com/mozilla","followers_url":"https://api.github.com/users/mozilla/followers","following_url":"https://api.github.com/users/mozilla/following{/other_user}","gists_url":"https://api.github.com/users/mozilla/gists{/gist_id}","starred_url":"https://api.github.com/users/mozilla/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mozilla/subscriptions","organizations_url":"https://api.github.com/users/mozilla/orgs","repos_url":"https://api.github.com/users/mozilla/repos","events_url":"https://api.github.com/users/mozilla/events{/privacy}","received_events_url":"https://api.github.com/users/mozilla/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mozilla/socorro","description":"Socorro is a server to accept and process Breakpad crash reports. ","fork":false,"url":"https://api.github.com/repos/mozilla/socorro","forks_url":"https://api.github.com/repos/mozilla/socorro/forks","keys_url":"https://api.github.com/repos/mozilla/socorro/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mozilla/socorro/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mozilla/socorro/teams","hooks_url":"https://api.github.com/repos/mozilla/socorro/hooks","issue_events_url":"https://api.github.com/repos/mozilla/socorro/issues/events{/number}","events_url":"https://api.github.com/repos/mozilla/socorro/events","assignees_url":"https://api.github.com/repos/mozilla/socorro/assignees{/user}","branches_url":"https://api.github.com/repos/mozilla/socorro/branches{/branch}","tags_url":"https://api.github.com/repos/mozilla/socorro/tags","blobs_url":"https://api.github.com/repos/mozilla/socorro/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mozilla/socorro/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mozilla/socorro/git/refs{/sha}","trees_url":"https://api.github.com/repos/mozilla/socorro/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mozilla/socorro/statuses/{sha}","languages_url":"https://api.github.com/repos/mozilla/socorro/languages","stargazers_url":"https://api.github.com/repos/mozilla/socorro/stargazers","contributors_url":"https://api.github.com/repos/mozilla/socorro/contributors","subscribers_url":"https://api.github.com/repos/mozilla/socorro/subscribers","subscription_url":"https://api.github.com/repos/mozilla/socorro/subscription","commits_url":"https://api.github.com/repos/mozilla/socorro/commits{/sha}","git_commits_url":"https://api.github.com/repos/mozilla/socorro/git/commits{/sha}","comments_url":"https://api.github.com/repos/mozilla/socorro/comments{/number}","issue_comment_url":"https://api.github.com/repos/mozilla/socorro/issues/comments/{number}","contents_url":"https://api.github.com/repos/mozilla/socorro/contents/{+path}","compare_url":"https://api.github.com/repos/mozilla/socorro/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mozilla/socorro/merges","archive_url":"https://api.github.com/repos/mozilla/socorro/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mozilla/socorro/downloads","issues_url":"https://api.github.com/repos/mozilla/socorro/issues{/number}","pulls_url":"https://api.github.com/repos/mozilla/socorro/pulls{/number}","milestones_url":"https://api.github.com/repos/mozilla/socorro/milestones{/number}","notifications_url":"https://api.github.com/repos/mozilla/socorro/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mozilla/socorro/labels{/name}","releases_url":"https://api.github.com/repos/mozilla/socorro/releases{/id}","created_at":"2011-08-24T19:21:49Z","updated_at":"2014-09-09T01:39:52Z","pushed_at":"2014-09-09T20:26:39Z","git_url":"git://github.com/mozilla/socorro.git","ssh_url":"git@github.com:mozilla/socorro.git","clone_url":"https://github.com/mozilla/socorro.git","svn_url":"https://github.com/mozilla/socorro","homepage":"https://wiki.mozilla.org/Socorro","size":189139,"stargazers_count":187,"watchers_count":187,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"forks_count":96,"mirror_url":null,"open_issues_count":6,"forks":96,"open_issues":6,"watchers":187,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/mozilla/socorro/pulls/2346"},"html":{"href":"https://github.com/mozilla/socorro/pull/2346"},"issue":{"href":"https://api.github.com/repos/mozilla/socorro/issues/2346"},"comments":{"href":"https://api.github.com/repos/mozilla/socorro/issues/2346/comments"},"review_comments":{"href":"https://api.github.com/repos/mozilla/socorro/pulls/2346/comments"},"review_comment":{"href":"https://api.github.com/repos/mozilla/socorro/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/mozilla/socorro/pulls/2346/commits"},"statuses":{"href":"https://api.github.com/repos/mozilla/socorro/statuses/836d257f81870f470350ba093273013c2de44585"}}}}