{"action":"created","comment":{"url":"https://api.github.com/repos/edx/journals/pulls/comments/222687295","pull_request_review_id":161633584,"id":222687295,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIyMjY4NzI5NQ==","diff_hunk":"@@ -66,18 +68,28 @@ def _handle_create(self, options):\n                 journal.organization.site.siteconfiguration.discovery_journal_api_client,\n                 journal_meta_data.get_discovery_data()\n             )\n-        except HttpClientError as err:\n+        except (HttpClientError, HttpServerError) as err:\n             # TODO - roll back journal updates if this fails\n-            raise CommandError('Error publishing to discovery-service: {}'.format(err.content))\n+            err_str = 'Error publishing to discovery-service, discovery data={discovery_data} err={err}'.format(\n+                discovery_data=journal_meta_data.get_discovery_data(),\n+                err=err.content\n+            )\n+            logger.error(err_str)\n+            raise CommandError(err_str)\n \n         try:\n             self._update_ecommerce(\n                 journal.organization.site.siteconfiguration.ecommerce_journal_api_client,\n                 journal_meta_data.get_ecommerce_data()\n             )\n-        except HttpClientError as err:\n+        except (HttpClientError, HttpServerError) as err:\n             # TODO - roll back discovery updates if this fails\n-            raise CommandError('Error publishing to ecommerce-service: {}'.format(err.content))\n+            err_str = 'Error publishing to ecommerce-service, ecomm data={ecomm_data} err={err}'.format(\n+                ecomm_data=journal_meta_data.get_ecommerce_data(),","path":"journals/apps/journals/management/commands/publish_journals.py","position":45,"original_position":45,"commit_id":"b7cf2af0cfbcbdeb233101c79c90d485ee8f2697","original_commit_id":"b7cf2af0cfbcbdeb233101c79c90d485ee8f2697","user":{"login":"tuchfarber","id":6179874,"node_id":"MDQ6VXNlcjYxNzk4NzQ=","avatar_url":"https://avatars3.githubusercontent.com/u/6179874?v=4","gravatar_id":"","url":"https://api.github.com/users/tuchfarber","html_url":"https://github.com/tuchfarber","followers_url":"https://api.github.com/users/tuchfarber/followers","following_url":"https://api.github.com/users/tuchfarber/following{/other_user}","gists_url":"https://api.github.com/users/tuchfarber/gists{/gist_id}","starred_url":"https://api.github.com/users/tuchfarber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tuchfarber/subscriptions","organizations_url":"https://api.github.com/users/tuchfarber/orgs","repos_url":"https://api.github.com/users/tuchfarber/repos","events_url":"https://api.github.com/users/tuchfarber/events{/privacy}","received_events_url":"https://api.github.com/users/tuchfarber/received_events","type":"User","site_admin":false},"body":"Same as above with `get_ecommerce_data()`.","created_at":"2018-10-04T14:17:16Z","updated_at":"2018-10-04T14:18:26Z","html_url":"https://github.com/edx/journals/pull/154#discussion_r222687295","pull_request_url":"https://api.github.com/repos/edx/journals/pulls/154","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/edx/journals/pulls/comments/222687295"},"html":{"href":"https://github.com/edx/journals/pull/154#discussion_r222687295"},"pull_request":{"href":"https://api.github.com/repos/edx/journals/pulls/154"}}},"pull_request":{"url":"https://api.github.com/repos/edx/journals/pulls/154","id":220388638,"node_id":"MDExOlB1bGxSZXF1ZXN0MjIwMzg4NjM4","html_url":"https://github.com/edx/journals/pull/154","diff_url":"https://github.com/edx/journals/pull/154.diff","patch_url":"https://github.com/edx/journals/pull/154.patch","issue_url":"https://api.github.com/repos/edx/journals/issues/154","number":154,"state":"open","locked":false,"title":"Journal publishing fixes","user":{"login":"bill-filler","id":27503539,"node_id":"MDQ6VXNlcjI3NTAzNTM5","avatar_url":"https://avatars3.githubusercontent.com/u/27503539?v=4","gravatar_id":"","url":"https://api.github.com/users/bill-filler","html_url":"https://github.com/bill-filler","followers_url":"https://api.github.com/users/bill-filler/followers","following_url":"https://api.github.com/users/bill-filler/following{/other_user}","gists_url":"https://api.github.com/users/bill-filler/gists{/gist_id}","starred_url":"https://api.github.com/users/bill-filler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bill-filler/subscriptions","organizations_url":"https://api.github.com/users/bill-filler/orgs","repos_url":"https://api.github.com/users/bill-filler/repos","events_url":"https://api.github.com/users/bill-filler/events{/privacy}","received_events_url":"https://api.github.com/users/bill-filler/received_events","type":"User","site_admin":false},"body":"Journal publishing fixes\r\n\r\nLog errors when pushing to discovery and ecommerce during publishing.\r\nAlso pass updated Organization dict to discovery needed for api change.","created_at":"2018-10-04T14:00:21Z","updated_at":"2018-10-04T14:18:26Z","closed_at":null,"merged_at":null,"merge_commit_sha":"cec658e49b2c7496c52f12775403fc07df733db3","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/edx/journals/pulls/154/commits","review_comments_url":"https://api.github.com/repos/edx/journals/pulls/154/comments","review_comment_url":"https://api.github.com/repos/edx/journals/pulls/comments{/number}","comments_url":"https://api.github.com/repos/edx/journals/issues/154/comments","statuses_url":"https://api.github.com/repos/edx/journals/statuses/b7cf2af0cfbcbdeb233101c79c90d485ee8f2697","head":{"label":"edx:bfiller/fix-publish-error","ref":"bfiller/fix-publish-error","sha":"b7cf2af0cfbcbdeb233101c79c90d485ee8f2697","user":{"login":"edx","id":3179841,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxNzk4NDE=","avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"repo":{"id":126215869,"node_id":"MDEwOlJlcG9zaXRvcnkxMjYyMTU4Njk=","name":"journals","full_name":"edx/journals","private":false,"owner":{"login":"edx","id":3179841,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxNzk4NDE=","avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/edx/journals","description":"Service to create and serve journals for open edX.","fork":false,"url":"https://api.github.com/repos/edx/journals","forks_url":"https://api.github.com/repos/edx/journals/forks","keys_url":"https://api.github.com/repos/edx/journals/keys{/key_id}","collaborators_url":"https://api.github.com/repos/edx/journals/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/edx/journals/teams","hooks_url":"https://api.github.com/repos/edx/journals/hooks","issue_events_url":"https://api.github.com/repos/edx/journals/issues/events{/number}","events_url":"https://api.github.com/repos/edx/journals/events","assignees_url":"https://api.github.com/repos/edx/journals/assignees{/user}","branches_url":"https://api.github.com/repos/edx/journals/branches{/branch}","tags_url":"https://api.github.com/repos/edx/journals/tags","blobs_url":"https://api.github.com/repos/edx/journals/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/edx/journals/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/edx/journals/git/refs{/sha}","trees_url":"https://api.github.com/repos/edx/journals/git/trees{/sha}","statuses_url":"https://api.github.com/repos/edx/journals/statuses/{sha}","languages_url":"https://api.github.com/repos/edx/journals/languages","stargazers_url":"https://api.github.com/repos/edx/journals/stargazers","contributors_url":"https://api.github.com/repos/edx/journals/contributors","subscribers_url":"https://api.github.com/repos/edx/journals/subscribers","subscription_url":"https://api.github.com/repos/edx/journals/subscription","commits_url":"https://api.github.com/repos/edx/journals/commits{/sha}","git_commits_url":"https://api.github.com/repos/edx/journals/git/commits{/sha}","comments_url":"https://api.github.com/repos/edx/journals/comments{/number}","issue_comment_url":"https://api.github.com/repos/edx/journals/issues/comments{/number}","contents_url":"https://api.github.com/repos/edx/journals/contents/{+path}","compare_url":"https://api.github.com/repos/edx/journals/compare/{base}...{head}","merges_url":"https://api.github.com/repos/edx/journals/merges","archive_url":"https://api.github.com/repos/edx/journals/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/edx/journals/downloads","issues_url":"https://api.github.com/repos/edx/journals/issues{/number}","pulls_url":"https://api.github.com/repos/edx/journals/pulls{/number}","milestones_url":"https://api.github.com/repos/edx/journals/milestones{/number}","notifications_url":"https://api.github.com/repos/edx/journals/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/edx/journals/labels{/name}","releases_url":"https://api.github.com/repos/edx/journals/releases{/id}","deployments_url":"https://api.github.com/repos/edx/journals/deployments","created_at":"2018-03-21T17:23:51Z","updated_at":"2018-10-04T10:45:41Z","pushed_at":"2018-10-04T14:00:45Z","git_url":"git://github.com/edx/journals.git","ssh_url":"git@github.com:edx/journals.git","clone_url":"https://github.com/edx/journals.git","svn_url":"https://github.com/edx/journals","homepage":"","size":4518,"stargazers_count":3,"watchers_count":3,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"archived":false,"open_issues_count":4,"license":{"key":"agpl-3.0","name":"GNU Affero General Public License v3.0","spdx_id":"AGPL-3.0","url":"https://api.github.com/licenses/agpl-3.0","node_id":"MDc6TGljZW5zZTE="},"forks":1,"open_issues":4,"watchers":3,"default_branch":"master"}},"base":{"label":"edx:master","ref":"master","sha":"f16f2d60067dc0591b0392b30d383d5dda278245","user":{"login":"edx","id":3179841,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxNzk4NDE=","avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"repo":{"id":126215869,"node_id":"MDEwOlJlcG9zaXRvcnkxMjYyMTU4Njk=","name":"journals","full_name":"edx/journals","private":false,"owner":{"login":"edx","id":3179841,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxNzk4NDE=","avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/edx/journals","description":"Service to create and serve journals for open edX.","fork":false,"url":"https://api.github.com/repos/edx/journals","forks_url":"https://api.github.com/repos/edx/journals/forks","keys_url":"https://api.github.com/repos/edx/journals/keys{/key_id}","collaborators_url":"https://api.github.com/repos/edx/journals/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/edx/journals/teams","hooks_url":"https://api.github.com/repos/edx/journals/hooks","issue_events_url":"https://api.github.com/repos/edx/journals/issues/events{/number}","events_url":"https://api.github.com/repos/edx/journals/events","assignees_url":"https://api.github.com/repos/edx/journals/assignees{/user}","branches_url":"https://api.github.com/repos/edx/journals/branches{/branch}","tags_url":"https://api.github.com/repos/edx/journals/tags","blobs_url":"https://api.github.com/repos/edx/journals/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/edx/journals/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/edx/journals/git/refs{/sha}","trees_url":"https://api.github.com/repos/edx/journals/git/trees{/sha}","statuses_url":"https://api.github.com/repos/edx/journals/statuses/{sha}","languages_url":"https://api.github.com/repos/edx/journals/languages","stargazers_url":"https://api.github.com/repos/edx/journals/stargazers","contributors_url":"https://api.github.com/repos/edx/journals/contributors","subscribers_url":"https://api.github.com/repos/edx/journals/subscribers","subscription_url":"https://api.github.com/repos/edx/journals/subscription","commits_url":"https://api.github.com/repos/edx/journals/commits{/sha}","git_commits_url":"https://api.github.com/repos/edx/journals/git/commits{/sha}","comments_url":"https://api.github.com/repos/edx/journals/comments{/number}","issue_comment_url":"https://api.github.com/repos/edx/journals/issues/comments{/number}","contents_url":"https://api.github.com/repos/edx/journals/contents/{+path}","compare_url":"https://api.github.com/repos/edx/journals/compare/{base}...{head}","merges_url":"https://api.github.com/repos/edx/journals/merges","archive_url":"https://api.github.com/repos/edx/journals/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/edx/journals/downloads","issues_url":"https://api.github.com/repos/edx/journals/issues{/number}","pulls_url":"https://api.github.com/repos/edx/journals/pulls{/number}","milestones_url":"https://api.github.com/repos/edx/journals/milestones{/number}","notifications_url":"https://api.github.com/repos/edx/journals/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/edx/journals/labels{/name}","releases_url":"https://api.github.com/repos/edx/journals/releases{/id}","deployments_url":"https://api.github.com/repos/edx/journals/deployments","created_at":"2018-03-21T17:23:51Z","updated_at":"2018-10-04T10:45:41Z","pushed_at":"2018-10-04T14:00:45Z","git_url":"git://github.com/edx/journals.git","ssh_url":"git@github.com:edx/journals.git","clone_url":"https://github.com/edx/journals.git","svn_url":"https://github.com/edx/journals","homepage":"","size":4518,"stargazers_count":3,"watchers_count":3,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"archived":false,"open_issues_count":4,"license":{"key":"agpl-3.0","name":"GNU Affero General Public License v3.0","spdx_id":"AGPL-3.0","url":"https://api.github.com/licenses/agpl-3.0","node_id":"MDc6TGljZW5zZTE="},"forks":1,"open_issues":4,"watchers":3,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/edx/journals/pulls/154"},"html":{"href":"https://github.com/edx/journals/pull/154"},"issue":{"href":"https://api.github.com/repos/edx/journals/issues/154"},"comments":{"href":"https://api.github.com/repos/edx/journals/issues/154/comments"},"review_comments":{"href":"https://api.github.com/repos/edx/journals/pulls/154/comments"},"review_comment":{"href":"https://api.github.com/repos/edx/journals/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/edx/journals/pulls/154/commits"},"statuses":{"href":"https://api.github.com/repos/edx/journals/statuses/b7cf2af0cfbcbdeb233101c79c90d485ee8f2697"}},"author_association":"MEMBER"}}
{"actor":{"display_login":"tuchfarber"}}