[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/109 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78297425 @pyr thanks, reviewed both doc and code and merged. Thanks for your contribution! --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread pyr
Github user pyr commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78251684 @bhaisaab now in apache/cloudstack-docs-admin#28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78246495 @pyr you can fork/edit this: https://github.com/apache/cloudstack-docs-admin/blob/master/source/events.rst and send PR --- If your project is set up for it, you ca

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread pyr
Github user pyr commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78245063 @bhaisaab yes, where is the repo ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does n

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78244523 @pyr like the RabbitMQ, do you want to add docs to use/configure this as well? http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.4/events.htm

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78244266 @pyr thanks for replying. LGTM. Let's wait if any of the eventbus gurus want to review, else we'll merge by tomorrow. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread pyr
Github user pyr commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78220517 @bhaisaab I made the topic configurable and provided some more explanation in the PR description. Thanks for the review. --- If your project is set up for it, you can re

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread pyr
Github user pyr commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78219876 @bhaisaab the producer has a thread which handles this and knows what broker is responsible for what partition, it maintains its own internal broker topology, so there is

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread pyr
Github user pyr commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/109#discussion_r26194042 --- Diff: plugins/event-bus/kafka/pom.xml --- @@ -0,0 +1,45 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-inst

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/109#discussion_r26193337 --- Diff: plugins/event-bus/kafka/pom.xml --- @@ -0,0 +1,45 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-11 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/109#issuecomment-78217785 @pyr looks good, should it have methods for doing/handling connection and reconnection? (I've no idea how to connect to Kafka, assuming it's over TCP) --- If your p

[GitHub] cloudstack pull request: Add Apache Kafka event-bus support - prod...

2015-03-10 Thread pyr
GitHub user pyr opened a pull request: https://github.com/apache/cloudstack/pull/109 Add Apache Kafka event-bus support - producing only. This commit produces event bus messages to a "cloudstack" topic in Apache Kafka. Configuration is expected to be found in /etc/cloudstack/