k8ssandra / k8ssandra-operator

The Kubernetes operator for K8ssandra
https://k8ssandra.io/
Apache License 2.0
174 stars 79 forks source link

Consider DC-level config when validating numToken updates in webhook (fixes #1222) #1358

Closed olim7t closed 4 months ago

olim7t commented 5 months ago

What this PR does: Instead of only comparing the top-level config (which can fail if fields are missing), compute and compare the numToken of each individual DC, using the inheritance and defaulting rules. This also covers cases that were not handled before, like config being nil on one side but not the other.

Which issue(s) this PR fixes: Fixes #1222

Checklist

adejanovski commented 4 months ago

@olim7t @burmanm, what's the status on this PR?