search

vlab-research / fly

The Fly Survey platform
Other
2 stars 3 forks source link

Remove deprecated helm/incubator kafka chart for dev env #17

Closed nandanrao closed 3 years ago

nandanrao commented 3 years ago

In our dev/test environments we don't expect a Kafka operator, rather we use a standalone Kafka helm chart which currently points to the old helm/incubator kafka chart.

We can either:

A) Include the Kafka operator in the Helm chart and use it for dev. This is not a good idea, as we usually want one operator per cluster and multiple apps per cluster.

B) Switch to the Bitnami Kafka chart for dev/test environments.

C) Add a Kafka operator to the bootstrap scripts for dev/test clusters and change the dev/test environments to use CRDs.

nandanrao commented 3 years ago

Went with Bitnami chart.

Fixed in: https://github.com/vlab-research/fly/commit/c3a81650e22e65f0b1b3bf0e84d70d530df00ab0