yahoo / CMAK

CMAK is a tool for managing Apache Kafka clusters
Apache License 2.0
11.73k stars 2.5k forks source link

JMX with authentication #106

Open bwdezend opened 8 years ago

bwdezend commented 8 years ago

Is there a way to configure kaka-manager to poll JMX with a username and password? I don't see any options or settings for it, and I have JMX authentication enabled on our kafka brokers.

zeph commented 8 years ago

hah, I got the same problem ... @bwdezend have u found a workaround?

zeph commented 8 years ago

k, there is no code for it ...let me see how hard is to expose user/pass on the UI

  case class BrokerIdentity(id: Int, host: String, port: Int, jmxPort: Int)
bwdezend commented 8 years ago

Thanks for working on this zeph; I don't have a workaround for this. I am already consuming metrics into graphite via jmxtrans, but it would be handy to be able to see stats in the kafka-manager web interface as well.

zeph commented 8 years ago

@bwdezend please test my patch (-> #113 ) and see if u have the same issues I do

zeph commented 8 years ago

k, it won't work without installing Scala on Kafka side too...

bwdezend commented 8 years ago

I will setup a test kafka instance shortly and see if I can re-create your issues.

nictrix commented 8 years ago

Hi, was wondering if progress has been made on this? Has anyone tested patch #113?

zeph commented 8 years ago

@nictrix my code seems to break the CI, there are already comments to it from @patelh I'll have time to merge them and to retest in the next couple of weeks (to answer your question: i had no feedback from anybody else on the patch)

lynch0227 commented 8 years ago

Hi, does this patch already merge into the master branch?

zeph commented 8 years ago

@lynch0227 not yet, my fault, I'm looking into the proposed changes to my patch this week

zeph commented 8 years ago

@lynch0227 fixed but having issue at the travis-ci

zeph commented 8 years ago

with the requested fixes from @patelh now it is passing the travis-ci

@lynch0227, @nictrix , @bwdezend good to be merged?

zeph commented 8 years ago

@patelh just merged (and contributed to) my patch, I guess we shall resolve/close this issue, no?