{"action":"opened","number":230,"pull_request":{"url":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/230","id":235264600,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM1MjY0NjAw","html_url":"https://github.com/Yolean/kubernetes-kafka/pull/230","diff_url":"https://github.com/Yolean/kubernetes-kafka/pull/230.diff","patch_url":"https://github.com/Yolean/kubernetes-kafka/pull/230.patch","issue_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/230","number":230,"state":"open","locked":false,"title":"Allows a maximum of 1 connection to zookeeper per (pod) IP","user":{"login":"solsson","id":144945,"node_id":"MDQ6VXNlcjE0NDk0NQ==","avatar_url":"https://avatars2.githubusercontent.com/u/144945?v=4","gravatar_id":"","url":"https://api.github.com/users/solsson","html_url":"https://github.com/solsson","followers_url":"https://api.github.com/users/solsson/followers","following_url":"https://api.github.com/users/solsson/following{/other_user}","gists_url":"https://api.github.com/users/solsson/gists{/gist_id}","starred_url":"https://api.github.com/users/solsson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/solsson/subscriptions","organizations_url":"https://api.github.com/users/solsson/orgs","repos_url":"https://api.github.com/users/solsson/repos","events_url":"https://api.github.com/users/solsson/events{/privacy}","received_events_url":"https://api.github.com/users/solsson/received_events","type":"User","site_admin":false},"body":"Part of the 5.0 destabilization effort :)\r\n\r\nI like this kind of restrictions as sanity check. It discourages use of the Kafka pods for administration stuff (bad practice wrt resource limits etc) and use of legacy clients that connect directly to zookeeper. Zookeeper [docs](https://zookeeper.apache.org/doc/r3.4.13/zookeeperAdmin.html) also mentions \"file descriptor exhaustion\".\r\n\r\nSee the commit comment for the rationale behind the number 1. Sure, we'll increase it if anyone sees practical issues. For now it's on par with the [philosophy behind memory limits in 5.0.0](https://github.com/Yolean/kubernetes-kafka/blob/v5.0.0/kafka/50kafka.yml#L71).","created_at":"2018-12-02T15:10:35Z","updated_at":"2018-12-02T15:10:35Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/230/commits","review_comments_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/230/comments","review_comment_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/230/comments","statuses_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/statuses/00701ce01157cd8f8cfc8d0572d32eff8360c397","head":{"label":"Yolean:zookeeper-max-connections","ref":"zookeeper-max-connections","sha":"00701ce01157cd8f8cfc8d0572d32eff8360c397","user":{"login":"Yolean","id":11188184,"node_id":"MDEyOk9yZ2FuaXphdGlvbjExMTg4MTg0","avatar_url":"https://avatars1.githubusercontent.com/u/11188184?v=4","gravatar_id":"","url":"https://api.github.com/users/Yolean","html_url":"https://github.com/Yolean","followers_url":"https://api.github.com/users/Yolean/followers","following_url":"https://api.github.com/users/Yolean/following{/other_user}","gists_url":"https://api.github.com/users/Yolean/gists{/gist_id}","starred_url":"https://api.github.com/users/Yolean/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yolean/subscriptions","organizations_url":"https://api.github.com/users/Yolean/orgs","repos_url":"https://api.github.com/users/Yolean/repos","events_url":"https://api.github.com/users/Yolean/events{/privacy}","received_events_url":"https://api.github.com/users/Yolean/received_events","type":"Organization","site_admin":false},"repo":{"id":67395891,"node_id":"MDEwOlJlcG9zaXRvcnk2NzM5NTg5MQ==","name":"kubernetes-kafka","full_name":"Yolean/kubernetes-kafka","private":false,"owner":{"login":"Yolean","id":11188184,"node_id":"MDEyOk9yZ2FuaXphdGlvbjExMTg4MTg0","avatar_url":"https://avatars1.githubusercontent.com/u/11188184?v=4","gravatar_id":"","url":"https://api.github.com/users/Yolean","html_url":"https://github.com/Yolean","followers_url":"https://api.github.com/users/Yolean/followers","following_url":"https://api.github.com/users/Yolean/following{/other_user}","gists_url":"https://api.github.com/users/Yolean/gists{/gist_id}","starred_url":"https://api.github.com/users/Yolean/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yolean/subscriptions","organizations_url":"https://api.github.com/users/Yolean/orgs","repos_url":"https://api.github.com/users/Yolean/repos","events_url":"https://api.github.com/users/Yolean/events{/privacy}","received_events_url":"https://api.github.com/users/Yolean/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/Yolean/kubernetes-kafka","description":"Kafka cluster as Kubernetes StatefulSet, plain manifests and config","fork":false,"url":"https://api.github.com/repos/Yolean/kubernetes-kafka","forks_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/forks","keys_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/teams","hooks_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/hooks","issue_events_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/events{/number}","events_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/events","assignees_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/assignees{/user}","branches_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/branches{/branch}","tags_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/tags","blobs_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/refs{/sha}","trees_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/statuses/{sha}","languages_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/languages","stargazers_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/stargazers","contributors_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/contributors","subscribers_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/subscribers","subscription_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/subscription","commits_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/commits{/sha}","git_commits_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/commits{/sha}","comments_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/comments{/number}","issue_comment_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/comments{/number}","contents_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/contents/{+path}","compare_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/merges","archive_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/downloads","issues_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues{/number}","pulls_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls{/number}","milestones_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/milestones{/number}","notifications_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/labels{/name}","releases_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/releases{/id}","deployments_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/deployments","created_at":"2016-09-05T06:58:56Z","updated_at":"2018-12-02T15:03:38Z","pushed_at":"2018-12-02T15:03:35Z","git_url":"git://github.com/Yolean/kubernetes-kafka.git","ssh_url":"git@github.com:Yolean/kubernetes-kafka.git","clone_url":"https://github.com/Yolean/kubernetes-kafka.git","svn_url":"https://github.com/Yolean/kubernetes-kafka","homepage":"","size":546,"stargazers_count":808,"watchers_count":808,"language":"Shell","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":373,"mirror_url":null,"archived":false,"open_issues_count":55,"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":373,"open_issues":55,"watchers":808,"default_branch":"master"}},"base":{"label":"Yolean:master","ref":"master","sha":"cd0b3f0c5f141c6c96722c4f7db15409c95a0cd1","user":{"login":"Yolean","id":11188184,"node_id":"MDEyOk9yZ2FuaXphdGlvbjExMTg4MTg0","avatar_url":"https://avatars1.githubusercontent.com/u/11188184?v=4","gravatar_id":"","url":"https://api.github.com/users/Yolean","html_url":"https://github.com/Yolean","followers_url":"https://api.github.com/users/Yolean/followers","following_url":"https://api.github.com/users/Yolean/following{/other_user}","gists_url":"https://api.github.com/users/Yolean/gists{/gist_id}","starred_url":"https://api.github.com/users/Yolean/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yolean/subscriptions","organizations_url":"https://api.github.com/users/Yolean/orgs","repos_url":"https://api.github.com/users/Yolean/repos","events_url":"https://api.github.com/users/Yolean/events{/privacy}","received_events_url":"https://api.github.com/users/Yolean/received_events","type":"Organization","site_admin":false},"repo":{"id":67395891,"node_id":"MDEwOlJlcG9zaXRvcnk2NzM5NTg5MQ==","name":"kubernetes-kafka","full_name":"Yolean/kubernetes-kafka","private":false,"owner":{"login":"Yolean","id":11188184,"node_id":"MDEyOk9yZ2FuaXphdGlvbjExMTg4MTg0","avatar_url":"https://avatars1.githubusercontent.com/u/11188184?v=4","gravatar_id":"","url":"https://api.github.com/users/Yolean","html_url":"https://github.com/Yolean","followers_url":"https://api.github.com/users/Yolean/followers","following_url":"https://api.github.com/users/Yolean/following{/other_user}","gists_url":"https://api.github.com/users/Yolean/gists{/gist_id}","starred_url":"https://api.github.com/users/Yolean/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yolean/subscriptions","organizations_url":"https://api.github.com/users/Yolean/orgs","repos_url":"https://api.github.com/users/Yolean/repos","events_url":"https://api.github.com/users/Yolean/events{/privacy}","received_events_url":"https://api.github.com/users/Yolean/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/Yolean/kubernetes-kafka","description":"Kafka cluster as Kubernetes StatefulSet, plain manifests and config","fork":false,"url":"https://api.github.com/repos/Yolean/kubernetes-kafka","forks_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/forks","keys_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/teams","hooks_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/hooks","issue_events_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/events{/number}","events_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/events","assignees_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/assignees{/user}","branches_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/branches{/branch}","tags_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/tags","blobs_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/refs{/sha}","trees_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/statuses/{sha}","languages_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/languages","stargazers_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/stargazers","contributors_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/contributors","subscribers_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/subscribers","subscription_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/subscription","commits_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/commits{/sha}","git_commits_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/git/commits{/sha}","comments_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/comments{/number}","issue_comment_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/comments{/number}","contents_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/contents/{+path}","compare_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/merges","archive_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/downloads","issues_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues{/number}","pulls_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls{/number}","milestones_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/milestones{/number}","notifications_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/labels{/name}","releases_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/releases{/id}","deployments_url":"https://api.github.com/repos/Yolean/kubernetes-kafka/deployments","created_at":"2016-09-05T06:58:56Z","updated_at":"2018-12-02T15:03:38Z","pushed_at":"2018-12-02T15:03:35Z","git_url":"git://github.com/Yolean/kubernetes-kafka.git","ssh_url":"git@github.com:Yolean/kubernetes-kafka.git","clone_url":"https://github.com/Yolean/kubernetes-kafka.git","svn_url":"https://github.com/Yolean/kubernetes-kafka","homepage":"","size":546,"stargazers_count":808,"watchers_count":808,"language":"Shell","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":373,"mirror_url":null,"archived":false,"open_issues_count":55,"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":373,"open_issues":55,"watchers":808,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/230"},"html":{"href":"https://github.com/Yolean/kubernetes-kafka/pull/230"},"issue":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/230"},"comments":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/issues/230/comments"},"review_comments":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/230/comments"},"review_comment":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/pulls/230/commits"},"statuses":{"href":"https://api.github.com/repos/Yolean/kubernetes-kafka/statuses/00701ce01157cd8f8cfc8d0572d32eff8360c397"}},"author_association":"CONTRIBUTOR","merged":false,"mergeable":null,"rebaseable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"maintainer_can_modify":false,"commits":1,"additions":1,"deletions":0,"changed_files":1}}
{"actor":{"display_login":"solsson"}}