You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/06/08 20:08:59 UTC
[GitHub] [pulsar] rdhabalia opened a new pull request, #15987: Support mechanism to provide external zookeeper-server list to build global/configuration zookeeper
rdhabalia opened a new pull request, #15987:
URL: https://github.com/apache/pulsar/pull/15987
### Motivation
This PR requires for : https://github.com/apache/pulsar-helm-chart/pull/269
Right now, [chart dynamically](https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/zookeeper-statefulset.yaml#L140) creates zk cluster with zk pods initialized in the same namespace. However, for global/configuration zookeeper, user requires to build zk clusters with pods deployed in different namespaces. Therefore, user needs a mechanism to pass an external list of zk-servers to the chart and build zk-cluster with pods across different namespaces.
### Modification
- Chart should be considering zk-value's configuration for external zookeeper and generate zk-configuration file with appropriate zk-server list and unique id of that zookeeper.
This PR allows user to generate zk-config with external zk-server list and also updates appropriate zk-id file.
### Result
Fixes: https://github.com/apache/pulsar-helm-chart/issues/268
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] github-actions[bot] commented on pull request #15987: Support mechanism to provide external zookeeper-server list to build global/configuration zookeeper
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15987:
URL: https://github.com/apache/pulsar/pull/15987#issuecomment-1150355506
@rdhabalia:Thanks for your contribution. For this PR, do we need to update docs?
(The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md#documentation), which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] rdhabalia merged pull request #15987: Support mechanism to provide external zookeeper-server list to build global/configuration zookeeper
Posted by GitBox <gi...@apache.org>.
rdhabalia merged PR #15987:
URL: https://github.com/apache/pulsar/pull/15987
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] momo-jun commented on pull request #15987: Support mechanism to provide external zookeeper-server list to build global/configuration zookeeper
Posted by GitBox <gi...@apache.org>.
momo-jun commented on PR #15987:
URL: https://github.com/apache/pulsar/pull/15987#issuecomment-1216199435
@rdhabalia just wanted to check in - are there any doc updates required for this improvement?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org