Hello 🙂 In Pinot's Helm chart, is there any reason to have

.Values.{controller|broker}.external.enabled

set to

true

by default ? Maybe having it to

false

would be a safer alternative for first time users not knowing the chart well enough yet

Hello! Chart defaults are intended for quick starts. Gence a public service makes it easier to access it after you’ve just deployed it. I agree non exposed is safer.

Yeah I get that some chart defaults are more "quick test" oriented while others can be more prod oriented I guess the question now is, what should we do for Pinot 😄 Maybe have separate values files ?

Helm doesn’t really have a notion of “profiles”. So the best we can do is document recommended production ready helm values.

My helm knowledge is (very) limited but that sounds like a step in a good direction