Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-2675

[Puppet] Kafka can only be deployed on master node

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: deployment
    • Labels:
      None

      Description

      There're two issues with current Kafka deployment:

      • The default role of Kafka is on master node, which is wrong.
      • The broker.id should be a unique id in a multi-node environment
        We can leverage KAFKA-1070 to automatically generate broker id here.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user evans-ye opened a pull request:

          https://github.com/apache/bigtop/pull/175

          BIGTOP-2675. [Puppet] Kafka can only be deployed on master node

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/evans-ye/bigtop BIGTOP-2675

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/bigtop/pull/175.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #175


          commit 11b4fef8434653b510cbdffb0d3a6332b1887331
          Author: Evans Ye <evansye@apache.org>
          Date: 2017-01-22T09:54:44Z

          BIGTOP-2675. [Puppet] Kafka can only be deployed on master node


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user evans-ye opened a pull request: https://github.com/apache/bigtop/pull/175 BIGTOP-2675 . [Puppet] Kafka can only be deployed on master node You can merge this pull request into a Git repository by running: $ git pull https://github.com/evans-ye/bigtop BIGTOP-2675 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/bigtop/pull/175.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #175 commit 11b4fef8434653b510cbdffb0d3a6332b1887331 Author: Evans Ye <evansye@apache.org> Date: 2017-01-22T09:54:44Z BIGTOP-2675 . [Puppet] Kafka can only be deployed on master node
          Hide
          evans_ye Evans Ye added a comment -

          Can someone has this reviewed?
          It should be a trivial patch.

          Show
          evans_ye Evans Ye added a comment - Can someone has this reviewed? It should be a trivial patch.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user rvs commented on the issue:

          https://github.com/apache/bigtop/pull/175

          LGTM! Please commit ASAP!

          Show
          githubbot ASF GitHub Bot added a comment - Github user rvs commented on the issue: https://github.com/apache/bigtop/pull/175 LGTM! Please commit ASAP!
          Hide
          evans_ye Evans Ye added a comment -

          Committed. Thanks for the review!

          Show
          evans_ye Evans Ye added a comment - Committed. Thanks for the review!

            People

            • Assignee:
              evans_ye Evans Ye
              Reporter:
              evans_ye Evans Ye
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development