{"action":"created","comment":{"url":"https://api.github.com/repos/swiftstack/1space/pulls/comments/224907345","pull_request_review_id":164399211,"id":224907345,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIyNDkwNzM0NQ==","diff_hunk":"@@ -162,3 +162,14 @@ def handle(self, row, swift_client):\n                 except UnexpectedResponse as e:\n                     if '409 Conflict' in e.message:\n                         pass\n+\n+\n+class SyncContainerFactory(object):","path":"s3_sync/sync_container.py","position":6,"original_position":6,"commit_id":"b7db05c3434356e8974179660dbac9ccaffe7277","original_commit_id":"b7db05c3434356e8974179660dbac9ccaffe7277","user":{"login":"tipabu","id":4432952,"node_id":"MDQ6VXNlcjQ0MzI5NTI=","avatar_url":"https://avatars3.githubusercontent.com/u/4432952?v=4","gravatar_id":"","url":"https://api.github.com/users/tipabu","html_url":"https://github.com/tipabu","followers_url":"https://api.github.com/users/tipabu/followers","following_url":"https://api.github.com/users/tipabu/following{/other_user}","gists_url":"https://api.github.com/users/tipabu/gists{/gist_id}","starred_url":"https://api.github.com/users/tipabu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tipabu/subscriptions","organizations_url":"https://api.github.com/users/tipabu/orgs","repos_url":"https://api.github.com/users/tipabu/repos","events_url":"https://api.github.com/users/tipabu/events{/privacy}","received_events_url":"https://api.github.com/users/tipabu/received_events","type":"User","site_admin":false},"body":"Should we define a friendlier `__str__` for this? Looks like we used to log something like\r\n```\r\nFailed to process account/container with SyncContainer\r\n```\r\nbut currently it'd be something like\r\n```\r\nFailed to process account/container with <s3_sync.sync_container.SyncContainerFactory object at 0x108861b10>\r\n```\r\n?","created_at":"2018-10-12T20:33:07Z","updated_at":"2018-10-12T20:34:28Z","html_url":"https://github.com/swiftstack/1space/pull/195#discussion_r224907345","pull_request_url":"https://api.github.com/repos/swiftstack/1space/pulls/195","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/swiftstack/1space/pulls/comments/224907345"},"html":{"href":"https://github.com/swiftstack/1space/pull/195#discussion_r224907345"},"pull_request":{"href":"https://api.github.com/repos/swiftstack/1space/pulls/195"}}},"pull_request":{"url":"https://api.github.com/repos/swiftstack/1space/pulls/195","id":222556457,"node_id":"MDExOlB1bGxSZXF1ZXN0MjIyNTU2NDU3","html_url":"https://github.com/swiftstack/1space/pull/195","diff_url":"https://github.com/swiftstack/1space/pull/195.diff","patch_url":"https://github.com/swiftstack/1space/pull/195.patch","issue_url":"https://api.github.com/repos/swiftstack/1space/issues/195","number":195,"state":"open","locked":false,"title":"Add a SyncContainerFactory class.","user":{"login":"timuralp","id":703870,"node_id":"MDQ6VXNlcjcwMzg3MA==","avatar_url":"https://avatars2.githubusercontent.com/u/703870?v=4","gravatar_id":"","url":"https://api.github.com/users/timuralp","html_url":"https://github.com/timuralp","followers_url":"https://api.github.com/users/timuralp/followers","following_url":"https://api.github.com/users/timuralp/following{/other_user}","gists_url":"https://api.github.com/users/timuralp/gists{/gist_id}","starred_url":"https://api.github.com/users/timuralp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timuralp/subscriptions","organizations_url":"https://api.github.com/users/timuralp/orgs","repos_url":"https://api.github.com/users/timuralp/repos","events_url":"https://api.github.com/users/timuralp/events{/privacy}","received_events_url":"https://api.github.com/users/timuralp/received_events","type":"User","site_admin":false},"body":"Allows us to work against a new version of the container-crawler\r\nlibrary, using a factory pattern.","created_at":"2018-10-12T18:56:05Z","updated_at":"2018-10-12T20:34:28Z","closed_at":null,"merged_at":null,"merge_commit_sha":"8d3c558ddbc919a91c7d1e2fd2ae404db365e0aa","assignee":null,"assignees":[],"requested_reviewers":[{"login":"thiagodasilva","id":1494299,"node_id":"MDQ6VXNlcjE0OTQyOTk=","avatar_url":"https://avatars0.githubusercontent.com/u/1494299?v=4","gravatar_id":"","url":"https://api.github.com/users/thiagodasilva","html_url":"https://github.com/thiagodasilva","followers_url":"https://api.github.com/users/thiagodasilva/followers","following_url":"https://api.github.com/users/thiagodasilva/following{/other_user}","gists_url":"https://api.github.com/users/thiagodasilva/gists{/gist_id}","starred_url":"https://api.github.com/users/thiagodasilva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thiagodasilva/subscriptions","organizations_url":"https://api.github.com/users/thiagodasilva/orgs","repos_url":"https://api.github.com/users/thiagodasilva/repos","events_url":"https://api.github.com/users/thiagodasilva/events{/privacy}","received_events_url":"https://api.github.com/users/thiagodasilva/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/swiftstack/1space/pulls/195/commits","review_comments_url":"https://api.github.com/repos/swiftstack/1space/pulls/195/comments","review_comment_url":"https://api.github.com/repos/swiftstack/1space/pulls/comments{/number}","comments_url":"https://api.github.com/repos/swiftstack/1space/issues/195/comments","statuses_url":"https://api.github.com/repos/swiftstack/1space/statuses/b7db05c3434356e8974179660dbac9ccaffe7277","head":{"label":"swiftstack:improvement/factory-sync-container","ref":"improvement/factory-sync-container","sha":"b7db05c3434356e8974179660dbac9ccaffe7277","user":{"login":"swiftstack","id":1240687,"node_id":"MDEyOk9yZ2FuaXphdGlvbjEyNDA2ODc=","avatar_url":"https://avatars1.githubusercontent.com/u/1240687?v=4","gravatar_id":"","url":"https://api.github.com/users/swiftstack","html_url":"https://github.com/swiftstack","followers_url":"https://api.github.com/users/swiftstack/followers","following_url":"https://api.github.com/users/swiftstack/following{/other_user}","gists_url":"https://api.github.com/users/swiftstack/gists{/gist_id}","starred_url":"https://api.github.com/users/swiftstack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swiftstack/subscriptions","organizations_url":"https://api.github.com/users/swiftstack/orgs","repos_url":"https://api.github.com/users/swiftstack/repos","events_url":"https://api.github.com/users/swiftstack/events{/privacy}","received_events_url":"https://api.github.com/users/swiftstack/received_events","type":"Organization","site_admin":false},"repo":{"id":58976963,"node_id":"MDEwOlJlcG9zaXRvcnk1ODk3Njk2Mw==","name":"1space","full_name":"swiftstack/1space","private":false,"owner":{"login":"swiftstack","id":1240687,"node_id":"MDEyOk9yZ2FuaXphdGlvbjEyNDA2ODc=","avatar_url":"https://avatars1.githubusercontent.com/u/1240687?v=4","gravatar_id":"","url":"https://api.github.com/users/swiftstack","html_url":"https://github.com/swiftstack","followers_url":"https://api.github.com/users/swiftstack/followers","following_url":"https://api.github.com/users/swiftstack/following{/other_user}","gists_url":"https://api.github.com/users/swiftstack/gists{/gist_id}","starred_url":"https://api.github.com/users/swiftstack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swiftstack/subscriptions","organizations_url":"https://api.github.com/users/swiftstack/orgs","repos_url":"https://api.github.com/users/swiftstack/repos","events_url":"https://api.github.com/users/swiftstack/events{/privacy}","received_events_url":"https://api.github.com/users/swiftstack/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/swiftstack/1space","description":"1space enables a single object namespace between object storage locations. 1space enables data lifecycling, migration, and data protection. 1space integrates with OpenStack Swift. The 1space connector is a container that runs in the cloud.","fork":false,"url":"https://api.github.com/repos/swiftstack/1space","forks_url":"https://api.github.com/repos/swiftstack/1space/forks","keys_url":"https://api.github.com/repos/swiftstack/1space/keys{/key_id}","collaborators_url":"https://api.github.com/repos/swiftstack/1space/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/swiftstack/1space/teams","hooks_url":"https://api.github.com/repos/swiftstack/1space/hooks","issue_events_url":"https://api.github.com/repos/swiftstack/1space/issues/events{/number}","events_url":"https://api.github.com/repos/swiftstack/1space/events","assignees_url":"https://api.github.com/repos/swiftstack/1space/assignees{/user}","branches_url":"https://api.github.com/repos/swiftstack/1space/branches{/branch}","tags_url":"https://api.github.com/repos/swiftstack/1space/tags","blobs_url":"https://api.github.com/repos/swiftstack/1space/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/swiftstack/1space/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/swiftstack/1space/git/refs{/sha}","trees_url":"https://api.github.com/repos/swiftstack/1space/git/trees{/sha}","statuses_url":"https://api.github.com/repos/swiftstack/1space/statuses/{sha}","languages_url":"https://api.github.com/repos/swiftstack/1space/languages","stargazers_url":"https://api.github.com/repos/swiftstack/1space/stargazers","contributors_url":"https://api.github.com/repos/swiftstack/1space/contributors","subscribers_url":"https://api.github.com/repos/swiftstack/1space/subscribers","subscription_url":"https://api.github.com/repos/swiftstack/1space/subscription","commits_url":"https://api.github.com/repos/swiftstack/1space/commits{/sha}","git_commits_url":"https://api.github.com/repos/swiftstack/1space/git/commits{/sha}","comments_url":"https://api.github.com/repos/swiftstack/1space/comments{/number}","issue_comment_url":"https://api.github.com/repos/swiftstack/1space/issues/comments{/number}","contents_url":"https://api.github.com/repos/swiftstack/1space/contents/{+path}","compare_url":"https://api.github.com/repos/swiftstack/1space/compare/{base}...{head}","merges_url":"https://api.github.com/repos/swiftstack/1space/merges","archive_url":"https://api.github.com/repos/swiftstack/1space/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/swiftstack/1space/downloads","issues_url":"https://api.github.com/repos/swiftstack/1space/issues{/number}","pulls_url":"https://api.github.com/repos/swiftstack/1space/pulls{/number}","milestones_url":"https://api.github.com/repos/swiftstack/1space/milestones{/number}","notifications_url":"https://api.github.com/repos/swiftstack/1space/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/swiftstack/1space/labels{/name}","releases_url":"https://api.github.com/repos/swiftstack/1space/releases{/id}","deployments_url":"https://api.github.com/repos/swiftstack/1space/deployments","created_at":"2016-05-17T00:20:29Z","updated_at":"2018-10-11T19:07:11Z","pushed_at":"2018-10-12T19:33:29Z","git_url":"git://github.com/swiftstack/1space.git","ssh_url":"git@github.com:swiftstack/1space.git","clone_url":"https://github.com/swiftstack/1space.git","svn_url":"https://github.com/swiftstack/1space","homepage":"","size":1204,"stargazers_count":13,"watchers_count":13,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"archived":false,"open_issues_count":17,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":6,"open_issues":17,"watchers":13,"default_branch":"master"}},"base":{"label":"swiftstack:master","ref":"master","sha":"224c2349bb9b8e4e56aa458b9a696a0ca55239a9","user":{"login":"swiftstack","id":1240687,"node_id":"MDEyOk9yZ2FuaXphdGlvbjEyNDA2ODc=","avatar_url":"https://avatars1.githubusercontent.com/u/1240687?v=4","gravatar_id":"","url":"https://api.github.com/users/swiftstack","html_url":"https://github.com/swiftstack","followers_url":"https://api.github.com/users/swiftstack/followers","following_url":"https://api.github.com/users/swiftstack/following{/other_user}","gists_url":"https://api.github.com/users/swiftstack/gists{/gist_id}","starred_url":"https://api.github.com/users/swiftstack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swiftstack/subscriptions","organizations_url":"https://api.github.com/users/swiftstack/orgs","repos_url":"https://api.github.com/users/swiftstack/repos","events_url":"https://api.github.com/users/swiftstack/events{/privacy}","received_events_url":"https://api.github.com/users/swiftstack/received_events","type":"Organization","site_admin":false},"repo":{"id":58976963,"node_id":"MDEwOlJlcG9zaXRvcnk1ODk3Njk2Mw==","name":"1space","full_name":"swiftstack/1space","private":false,"owner":{"login":"swiftstack","id":1240687,"node_id":"MDEyOk9yZ2FuaXphdGlvbjEyNDA2ODc=","avatar_url":"https://avatars1.githubusercontent.com/u/1240687?v=4","gravatar_id":"","url":"https://api.github.com/users/swiftstack","html_url":"https://github.com/swiftstack","followers_url":"https://api.github.com/users/swiftstack/followers","following_url":"https://api.github.com/users/swiftstack/following{/other_user}","gists_url":"https://api.github.com/users/swiftstack/gists{/gist_id}","starred_url":"https://api.github.com/users/swiftstack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swiftstack/subscriptions","organizations_url":"https://api.github.com/users/swiftstack/orgs","repos_url":"https://api.github.com/users/swiftstack/repos","events_url":"https://api.github.com/users/swiftstack/events{/privacy}","received_events_url":"https://api.github.com/users/swiftstack/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/swiftstack/1space","description":"1space enables a single object namespace between object storage locations. 1space enables data lifecycling, migration, and data protection. 1space integrates with OpenStack Swift. The 1space connector is a container that runs in the cloud.","fork":false,"url":"https://api.github.com/repos/swiftstack/1space","forks_url":"https://api.github.com/repos/swiftstack/1space/forks","keys_url":"https://api.github.com/repos/swiftstack/1space/keys{/key_id}","collaborators_url":"https://api.github.com/repos/swiftstack/1space/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/swiftstack/1space/teams","hooks_url":"https://api.github.com/repos/swiftstack/1space/hooks","issue_events_url":"https://api.github.com/repos/swiftstack/1space/issues/events{/number}","events_url":"https://api.github.com/repos/swiftstack/1space/events","assignees_url":"https://api.github.com/repos/swiftstack/1space/assignees{/user}","branches_url":"https://api.github.com/repos/swiftstack/1space/branches{/branch}","tags_url":"https://api.github.com/repos/swiftstack/1space/tags","blobs_url":"https://api.github.com/repos/swiftstack/1space/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/swiftstack/1space/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/swiftstack/1space/git/refs{/sha}","trees_url":"https://api.github.com/repos/swiftstack/1space/git/trees{/sha}","statuses_url":"https://api.github.com/repos/swiftstack/1space/statuses/{sha}","languages_url":"https://api.github.com/repos/swiftstack/1space/languages","stargazers_url":"https://api.github.com/repos/swiftstack/1space/stargazers","contributors_url":"https://api.github.com/repos/swiftstack/1space/contributors","subscribers_url":"https://api.github.com/repos/swiftstack/1space/subscribers","subscription_url":"https://api.github.com/repos/swiftstack/1space/subscription","commits_url":"https://api.github.com/repos/swiftstack/1space/commits{/sha}","git_commits_url":"https://api.github.com/repos/swiftstack/1space/git/commits{/sha}","comments_url":"https://api.github.com/repos/swiftstack/1space/comments{/number}","issue_comment_url":"https://api.github.com/repos/swiftstack/1space/issues/comments{/number}","contents_url":"https://api.github.com/repos/swiftstack/1space/contents/{+path}","compare_url":"https://api.github.com/repos/swiftstack/1space/compare/{base}...{head}","merges_url":"https://api.github.com/repos/swiftstack/1space/merges","archive_url":"https://api.github.com/repos/swiftstack/1space/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/swiftstack/1space/downloads","issues_url":"https://api.github.com/repos/swiftstack/1space/issues{/number}","pulls_url":"https://api.github.com/repos/swiftstack/1space/pulls{/number}","milestones_url":"https://api.github.com/repos/swiftstack/1space/milestones{/number}","notifications_url":"https://api.github.com/repos/swiftstack/1space/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/swiftstack/1space/labels{/name}","releases_url":"https://api.github.com/repos/swiftstack/1space/releases{/id}","deployments_url":"https://api.github.com/repos/swiftstack/1space/deployments","created_at":"2016-05-17T00:20:29Z","updated_at":"2018-10-11T19:07:11Z","pushed_at":"2018-10-12T19:33:29Z","git_url":"git://github.com/swiftstack/1space.git","ssh_url":"git@github.com:swiftstack/1space.git","clone_url":"https://github.com/swiftstack/1space.git","svn_url":"https://github.com/swiftstack/1space","homepage":"","size":1204,"stargazers_count":13,"watchers_count":13,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"archived":false,"open_issues_count":17,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":6,"open_issues":17,"watchers":13,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/swiftstack/1space/pulls/195"},"html":{"href":"https://github.com/swiftstack/1space/pull/195"},"issue":{"href":"https://api.github.com/repos/swiftstack/1space/issues/195"},"comments":{"href":"https://api.github.com/repos/swiftstack/1space/issues/195/comments"},"review_comments":{"href":"https://api.github.com/repos/swiftstack/1space/pulls/195/comments"},"review_comment":{"href":"https://api.github.com/repos/swiftstack/1space/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/swiftstack/1space/pulls/195/commits"},"statuses":{"href":"https://api.github.com/repos/swiftstack/1space/statuses/b7db05c3434356e8974179660dbac9ccaffe7277"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"tipabu"}}