datastax / kaap

KAAP, Kubernetes Autoscaling for Apache Pulsar
https://datastax.github.io/kaap/
Apache License 2.0
45 stars 15 forks source link

tls global default should be null #137

Closed pgier closed 1 year ago

pgier commented 1 year ago

The TLS global default will override resource specific settings if it's set to false. Therefore, the default should be null to allow individual resource defaults and overrides to take effect.

Fixes #136

pgier commented 1 year ago

@nicoloboschi please review, I'm not sure if this is the correct way to fix this

Edit: looks like this will require more detailed analysis, I'm not sure what the behavior is supposed to be here.