search

confluentinc / cp-helm-charts

The Confluent Platform Helm charts enable you to deploy Confluent Platform services on Kubernetes for development, test, and proof of concept environments.
https://cnfl.io/getting-started-kafka-kubernetes
Apache License 2.0
788 stars 846 forks source link

[Feature Request] Integrate control-center in cp-helm-chart #210

Open Amit-Thawait opened 5 years ago

Amit-Thawait commented 5 years ago

Current helm chart of Confluent Kafka doesn't have control center bundled in it. It would be nice to have control center chart integrated in the cp-helm-chart itself as it is there in the cp-demo docker-compose https://github.com/confluentinc/cp-demo

qshao-pivotal commented 5 years ago

This helm chart will only deal with OSS components at least for now as I believe. @gAmUssA @mikkin for more info.

Amit-Thawait commented 5 years ago

@qshao-pivotal Yes sure..I can understand it.

However, the client for whom we are working will be purchasing an enterprise license.

Can you please help me with the below question: Does confluent provides a different helm chart for enterprise users? or How do enterprise users of Kafka deploy kafka cluster in kubernetes?

gAmUssA commented 5 years ago

@Amit-Thawait I will add this chart but I don't have ETA for you. If you under deadline, could you take a stab on the PR? I can help to review and validate.

gAmUssA commented 5 years ago

@Amit-Thawait FYI, we are working on Confluent operator to support all of the confluent platform components on k8s. It will be available in Q12019

Amit-Thawait commented 5 years ago

Thanks for your reply @gAmUssA

For the time being, what I did was ..I created a deployment and a service for control-center and added the necessary env variables in the deployment by referring cp-demo example and guess what.. it worked.

I used cp-enterprise-control-center image instead of cp-control-center since cp-control-center seems to be outdated (last updated 2 years ago).

I can give it a shot to add cp-control-center in a week or two.

I will reference the PR here.

gAmUssA commented 5 years ago

Thanks for update @Amit-Thawait Let me know if you need help/review.

davidsung commented 5 years ago

@gAmUssA I have made a PR https://github.com/confluentinc/cp-helm-charts/pull/271, could you help reviewing it. Thanks!

Thanks @Amit-Thawait for the pointer.