Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed add clusters and modify cluster #42

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vvutharkar
Copy link

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

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

CLA is valid!

@sheeley
Copy link

sheeley commented Jun 8, 2015

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

and also bumping .deb version to 1.2
@patelh
Copy link
Collaborator

patelh commented Aug 26, 2015

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
Copy link
Collaborator

patelh commented Aug 26, 2015

Still need to implement the bootstrap config feature.

@sheeley
Copy link

sheeley commented Aug 27, 2015

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

@patelh
Copy link
Collaborator

patelh commented Aug 27, 2015

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

@prune998
Copy link

prune998 commented Sep 5, 2019

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