{"action":"created","comment":{"url":"https://api.github.com/repos/confluentinc/muckrake/pulls/comments/29472472","id":29472472,"diff_hunk":"@@ -17,28 +17,27 @@\n \n \n class KafkaService(Service):\n-    def __init__(self, service_context, zk, topics=None):\n+    def __init__(self, context, num_nodes, zk, topics=None):\n         \"\"\"\n-        :type service_context ducktape.services.service.ServiceContext\n+        :type context\n         :type zk: ZookeeperService\n         :type topics: dict\n         \"\"\"\n-        super(KafkaService, self).__init__(service_context)\n+        super(KafkaService, self).__init__(context, num_nodes)\n         self.zk = zk\n         self.topics = topics\n+        self.logs = {\n+            \"kafka_log\": {","path":"muckrake/services/kafka.py","position":17,"original_position":17,"commit_id":"8cbfdce83ff7a8034317e46cfadef9dfe2a3c181","original_commit_id":"f5c315060f19502eb0204dfa67b35d435bdf3143","user":{"login":"granders","id":1295932,"avatar_url":"https://avatars.githubusercontent.com/u/1295932?v=3","gravatar_id":"","url":"https://api.github.com/users/granders","html_url":"https://github.com/granders","followers_url":"https://api.github.com/users/granders/followers","following_url":"https://api.github.com/users/granders/following{/other_user}","gists_url":"https://api.github.com/users/granders/gists{/gist_id}","starred_url":"https://api.github.com/users/granders/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/granders/subscriptions","organizations_url":"https://api.github.com/users/granders/orgs","repos_url":"https://api.github.com/users/granders/repos","events_url":"https://api.github.com/users/granders/events{/privacy}","received_events_url":"https://api.github.com/users/granders/received_events","type":"User","site_admin":false},"body":"I don't have a better idea at the moment. Leaving as is but opened https://github.com/confluentinc/muckrake/issues/27","created_at":"2015-04-30T21:29:45Z","updated_at":"2015-04-30T21:29:45Z","html_url":"https://github.com/confluentinc/muckrake/pull/23#discussion_r29472472","pull_request_url":"https://api.github.com/repos/confluentinc/muckrake/pulls/23","_links":{"self":{"href":"https://api.github.com/repos/confluentinc/muckrake/pulls/comments/29472472"},"html":{"href":"https://github.com/confluentinc/muckrake/pull/23#discussion_r29472472"},"pull_request":{"href":"https://api.github.com/repos/confluentinc/muckrake/pulls/23"}}},"pull_request":{"url":"https://api.github.com/repos/confluentinc/muckrake/pulls/23","id":34223631,"html_url":"https://github.com/confluentinc/muckrake/pull/23","diff_url":"https://github.com/confluentinc/muckrake/pull/23.diff","patch_url":"https://github.com/confluentinc/muckrake/pull/23.patch","issue_url":"https://api.github.com/repos/confluentinc/muckrake/issues/23","number":23,"state":"open","locked":false,"title":"Service logs","user":{"login":"granders","id":1295932,"avatar_url":"https://avatars.githubusercontent.com/u/1295932?v=3","gravatar_id":"","url":"https://api.github.com/users/granders","html_url":"https://github.com/granders","followers_url":"https://api.github.com/users/granders/followers","following_url":"https://api.github.com/users/granders/following{/other_user}","gists_url":"https://api.github.com/users/granders/gists{/gist_id}","starred_url":"https://api.github.com/users/granders/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/granders/subscriptions","organizations_url":"https://api.github.com/users/granders/orgs","repos_url":"https://api.github.com/users/granders/repos","events_url":"https://api.github.com/users/granders/events{/privacy}","received_events_url":"https://api.github.com/users/granders/received_events","type":"User","site_admin":false},"body":"* Updated services and tests to reflect refactoring in ducktape service-logs branch.\r\n* Refactored PerformanceService to inherit from BackgroundThreadService.\r\n* Broke up core.py into multiple modules.\r\n* Broke up KafkaBenchmark into several small test classes.\r\n* Added very simple run_tests.py script with the intention of executing this as a jenkins job.","created_at":"2015-04-28T00:25:16Z","updated_at":"2015-04-30T21:29:45Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6c458cdf0052fa8a3ee55d05e10579d35d882cd3","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/confluentinc/muckrake/pulls/23/commits","review_comments_url":"https://api.github.com/repos/confluentinc/muckrake/pulls/23/comments","review_comment_url":"https://api.github.com/repos/confluentinc/muckrake/pulls/comments{/number}","comments_url":"https://api.github.com/repos/confluentinc/muckrake/issues/23/comments","statuses_url":"https://api.github.com/repos/confluentinc/muckrake/statuses/8cbfdce83ff7a8034317e46cfadef9dfe2a3c181","head":{"label":"confluentinc:service-logs","ref":"service-logs","sha":"8cbfdce83ff7a8034317e46cfadef9dfe2a3c181","user":{"login":"confluentinc","id":9439498,"avatar_url":"https://avatars.githubusercontent.com/u/9439498?v=3","gravatar_id":"","url":"https://api.github.com/users/confluentinc","html_url":"https://github.com/confluentinc","followers_url":"https://api.github.com/users/confluentinc/followers","following_url":"https://api.github.com/users/confluentinc/following{/other_user}","gists_url":"https://api.github.com/users/confluentinc/gists{/gist_id}","starred_url":"https://api.github.com/users/confluentinc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/confluentinc/subscriptions","organizations_url":"https://api.github.com/users/confluentinc/orgs","repos_url":"https://api.github.com/users/confluentinc/repos","events_url":"https://api.github.com/users/confluentinc/events{/privacy}","received_events_url":"https://api.github.com/users/confluentinc/received_events","type":"Organization","site_admin":false},"repo":{"id":32954806,"name":"muckrake","full_name":"confluentinc/muckrake","owner":{"login":"confluentinc","id":9439498,"avatar_url":"https://avatars.githubusercontent.com/u/9439498?v=3","gravatar_id":"","url":"https://api.github.com/users/confluentinc","html_url":"https://github.com/confluentinc","followers_url":"https://api.github.com/users/confluentinc/followers","following_url":"https://api.github.com/users/confluentinc/following{/other_user}","gists_url":"https://api.github.com/users/confluentinc/gists{/gist_id}","starred_url":"https://api.github.com/users/confluentinc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/confluentinc/subscriptions","organizations_url":"https://api.github.com/users/confluentinc/orgs","repos_url":"https://api.github.com/users/confluentinc/repos","events_url":"https://api.github.com/users/confluentinc/events{/privacy}","received_events_url":"https://api.github.com/users/confluentinc/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/confluentinc/muckrake","description":"Confluent Platform system tests","fork":false,"url":"https://api.github.com/repos/confluentinc/muckrake","forks_url":"https://api.github.com/repos/confluentinc/muckrake/forks","keys_url":"https://api.github.com/repos/confluentinc/muckrake/keys{/key_id}","collaborators_url":"https://api.github.com/repos/confluentinc/muckrake/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/confluentinc/muckrake/teams","hooks_url":"https://api.github.com/repos/confluentinc/muckrake/hooks","issue_events_url":"https://api.github.com/repos/confluentinc/muckrake/issues/events{/number}","events_url":"https://api.github.com/repos/confluentinc/muckrake/events","assignees_url":"https://api.github.com/repos/confluentinc/muckrake/assignees{/user}","branches_url":"https://api.github.com/repos/confluentinc/muckrake/branches{/branch}","tags_url":"https://api.github.com/repos/confluentinc/muckrake/tags","blobs_url":"https://api.github.com/repos/confluentinc/muckrake/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/confluentinc/muckrake/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/confluentinc/muckrake/git/refs{/sha}","trees_url":"https://api.github.com/repos/confluentinc/muckrake/git/trees{/sha}","statuses_url":"https://api.github.com/repos/confluentinc/muckrake/statuses/{sha}","languages_url":"https://api.github.com/repos/confluentinc/muckrake/languages","stargazers_url":"https://api.github.com/repos/confluentinc/muckrake/stargazers","contributors_url":"https://api.github.com/repos/confluentinc/muckrake/contributors","subscribers_url":"https://api.github.com/repos/confluentinc/muckrake/subscribers","subscription_url":"https://api.github.com/repos/confluentinc/muckrake/subscription","commits_url":"https://api.github.com/repos/confluentinc/muckrake/commits{/sha}","git_commits_url":"https://api.github.com/repos/confluentinc/muckrake/git/commits{/sha}","comments_url":"https://api.github.com/repos/confluentinc/muckrake/comments{/number}","issue_comment_url":"https://api.github.com/repos/confluentinc/muckrake/issues/comments{/number}","contents_url":"https://api.github.com/repos/confluentinc/muckrake/contents/{+path}","compare_url":"https://api.github.com/repos/confluentinc/muckrake/compare/{base}...{head}","merges_url":"https://api.github.com/repos/confluentinc/muckrake/merges","archive_url":"https://api.github.com/repos/confluentinc/muckrake/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/confluentinc/muckrake/downloads","issues_url":"https://api.github.com/repos/confluentinc/muckrake/issues{/number}","pulls_url":"https://api.github.com/repos/confluentinc/muckrake/pulls{/number}","milestones_url":"https://api.github.com/repos/confluentinc/muckrake/milestones{/number}","notifications_url":"https://api.github.com/repos/confluentinc/muckrake/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/confluentinc/muckrake/labels{/name}","releases_url":"https://api.github.com/repos/confluentinc/muckrake/releases{/id}","created_at":"2015-03-26T22:16:59Z","updated_at":"2015-04-28T11:43:50Z","pushed_at":"2015-04-30T19:14:11Z","git_url":"git://github.com/confluentinc/muckrake.git","ssh_url":"git@github.com:confluentinc/muckrake.git","clone_url":"https://github.com/confluentinc/muckrake.git","svn_url":"https://github.com/confluentinc/muckrake","homepage":null,"size":471,"stargazers_count":1,"watchers_count":1,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":23,"forks":0,"open_issues":23,"watchers":1,"default_branch":"master"}},"base":{"label":"confluentinc:master","ref":"master","sha":"c1fbc944eacd5d996ab7e8692c03fd62d19ffe8b","user":{"login":"confluentinc","id":9439498,"avatar_url":"https://avatars.githubusercontent.com/u/9439498?v=3","gravatar_id":"","url":"https://api.github.com/users/confluentinc","html_url":"https://github.com/confluentinc","followers_url":"https://api.github.com/users/confluentinc/followers","following_url":"https://api.github.com/users/confluentinc/following{/other_user}","gists_url":"https://api.github.com/users/confluentinc/gists{/gist_id}","starred_url":"https://api.github.com/users/confluentinc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/confluentinc/subscriptions","organizations_url":"https://api.github.com/users/confluentinc/orgs","repos_url":"https://api.github.com/users/confluentinc/repos","events_url":"https://api.github.com/users/confluentinc/events{/privacy}","received_events_url":"https://api.github.com/users/confluentinc/received_events","type":"Organization","site_admin":false},"repo":{"id":32954806,"name":"muckrake","full_name":"confluentinc/muckrake","owner":{"login":"confluentinc","id":9439498,"avatar_url":"https://avatars.githubusercontent.com/u/9439498?v=3","gravatar_id":"","url":"https://api.github.com/users/confluentinc","html_url":"https://github.com/confluentinc","followers_url":"https://api.github.com/users/confluentinc/followers","following_url":"https://api.github.com/users/confluentinc/following{/other_user}","gists_url":"https://api.github.com/users/confluentinc/gists{/gist_id}","starred_url":"https://api.github.com/users/confluentinc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/confluentinc/subscriptions","organizations_url":"https://api.github.com/users/confluentinc/orgs","repos_url":"https://api.github.com/users/confluentinc/repos","events_url":"https://api.github.com/users/confluentinc/events{/privacy}","received_events_url":"https://api.github.com/users/confluentinc/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/confluentinc/muckrake","description":"Confluent Platform system tests","fork":false,"url":"https://api.github.com/repos/confluentinc/muckrake","forks_url":"https://api.github.com/repos/confluentinc/muckrake/forks","keys_url":"https://api.github.com/repos/confluentinc/muckrake/keys{/key_id}","collaborators_url":"https://api.github.com/repos/confluentinc/muckrake/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/confluentinc/muckrake/teams","hooks_url":"https://api.github.com/repos/confluentinc/muckrake/hooks","issue_events_url":"https://api.github.com/repos/confluentinc/muckrake/issues/events{/number}","events_url":"https://api.github.com/repos/confluentinc/muckrake/events","assignees_url":"https://api.github.com/repos/confluentinc/muckrake/assignees{/user}","branches_url":"https://api.github.com/repos/confluentinc/muckrake/branches{/branch}","tags_url":"https://api.github.com/repos/confluentinc/muckrake/tags","blobs_url":"https://api.github.com/repos/confluentinc/muckrake/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/confluentinc/muckrake/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/confluentinc/muckrake/git/refs{/sha}","trees_url":"https://api.github.com/repos/confluentinc/muckrake/git/trees{/sha}","statuses_url":"https://api.github.com/repos/confluentinc/muckrake/statuses/{sha}","languages_url":"https://api.github.com/repos/confluentinc/muckrake/languages","stargazers_url":"https://api.github.com/repos/confluentinc/muckrake/stargazers","contributors_url":"https://api.github.com/repos/confluentinc/muckrake/contributors","subscribers_url":"https://api.github.com/repos/confluentinc/muckrake/subscribers","subscription_url":"https://api.github.com/repos/confluentinc/muckrake/subscription","commits_url":"https://api.github.com/repos/confluentinc/muckrake/commits{/sha}","git_commits_url":"https://api.github.com/repos/confluentinc/muckrake/git/commits{/sha}","comments_url":"https://api.github.com/repos/confluentinc/muckrake/comments{/number}","issue_comment_url":"https://api.github.com/repos/confluentinc/muckrake/issues/comments{/number}","contents_url":"https://api.github.com/repos/confluentinc/muckrake/contents/{+path}","compare_url":"https://api.github.com/repos/confluentinc/muckrake/compare/{base}...{head}","merges_url":"https://api.github.com/repos/confluentinc/muckrake/merges","archive_url":"https://api.github.com/repos/confluentinc/muckrake/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/confluentinc/muckrake/downloads","issues_url":"https://api.github.com/repos/confluentinc/muckrake/issues{/number}","pulls_url":"https://api.github.com/repos/confluentinc/muckrake/pulls{/number}","milestones_url":"https://api.github.com/repos/confluentinc/muckrake/milestones{/number}","notifications_url":"https://api.github.com/repos/confluentinc/muckrake/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/confluentinc/muckrake/labels{/name}","releases_url":"https://api.github.com/repos/confluentinc/muckrake/releases{/id}","created_at":"2015-03-26T22:16:59Z","updated_at":"2015-04-28T11:43:50Z","pushed_at":"2015-04-30T19:14:11Z","git_url":"git://github.com/confluentinc/muckrake.git","ssh_url":"git@github.com:confluentinc/muckrake.git","clone_url":"https://github.com/confluentinc/muckrake.git","svn_url":"https://github.com/confluentinc/muckrake","homepage":null,"size":471,"stargazers_count":1,"watchers_count":1,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":23,"forks":0,"open_issues":23,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/confluentinc/muckrake/pulls/23"},"html":{"href":"https://github.com/confluentinc/muckrake/pull/23"},"issue":{"href":"https://api.github.com/repos/confluentinc/muckrake/issues/23"},"comments":{"href":"https://api.github.com/repos/confluentinc/muckrake/issues/23/comments"},"review_comments":{"href":"https://api.github.com/repos/confluentinc/muckrake/pulls/23/comments"},"review_comment":{"href":"https://api.github.com/repos/confluentinc/muckrake/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/confluentinc/muckrake/pulls/23/commits"},"statuses":{"href":"https://api.github.com/repos/confluentinc/muckrake/statuses/8cbfdce83ff7a8034317e46cfadef9dfe2a3c181"}}}}