Skip to content

Removed add clusters and modify cluster#42

Open
vvutharkar wants to merge 3 commits into
yahoo:masterfrom
vvutharkar:master
Open

Removed add clusters and modify cluster#42
vvutharkar wants to merge 3 commits into
yahoo:masterfrom
vvutharkar:master

Conversation

@vvutharkar

Copy link
Copy Markdown

This pull request resolves #37 .

  • Add cluster operation removed
  • Modify cluster operation removed (since it allows to specify a list of ZK hosts, which allows a user to change and point to an entirely different Kafka cluster that he/she not supposed to manage)
  • Left enable/disable operation as it is harmless
  • Introduced config param to prime the Kafka Manager
    managed-kafka-clusters=["kmgr1,0.8.2-beta,zk1:2181,zk2:2181", "kmgr2,0.8.1.1,zk3:2181,zk4:2181"]

@yahoocla

Copy link
Copy Markdown

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄

@vvutharkar vvutharkar closed this Mar 28, 2015
@vvutharkar vvutharkar reopened this Mar 28, 2015
@yahoocla

Copy link
Copy Markdown

CLA is valid!

@sheeley

sheeley commented Jun 8, 2015

Copy link
Copy Markdown

What's the status here? Can I help with this somehow, @vvutharkar?

and also bumping .deb version to 1.2
@patelh

patelh commented Aug 26, 2015

Copy link
Copy Markdown
Collaborator

Hi guys, you can now control UI by changing this option, more info in the README:

application.features=["KMClusterManagerFeature","KMTopicManagerFeature","KMPreferredReplicaElectionFeature","KMReassignPartitionsFeature"]

For example, to remove the add, modify, delete buttons, remove "KMClusterManagerFeature" from the above list.

@patelh

patelh commented Aug 26, 2015

Copy link
Copy Markdown
Collaborator

Still need to implement the bootstrap config feature.

@sheeley

sheeley commented Aug 27, 2015

Copy link
Copy Markdown

@patelh looks like you've been doing some work! :) any idea when this stuff will wind up merged in?

@patelh

patelh commented Aug 27, 2015

Copy link
Copy Markdown
Collaborator

@sheeley everything is already in master except for bootstrap config feature.

@prune998

prune998 commented Sep 5, 2019

Copy link
Copy Markdown

any solution to bootstrap kafka manager ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Priming kafka-manager with 'zkhosts' of a known set of kafka clusters

6 participants