jetstack / navigator

Managed Database-as-a-Service (DBaaS) on Kubernetes
Apache License 2.0
271 stars 31 forks source link

Changing CassandraCluster persistence settings should not be allowed #233

Closed wallrj closed 6 years ago

wallrj commented 6 years ago

Right now, if you change the persistence settings for a running cluster, the navigator controller will attempt to update the statefulset with the new volumeclaimtemplate settings.

I'm not sure whether this is supported and even if so, Navigator doesn't yet have a way to control the rollout of the updated statefulset.

We should prevent the user from changing the persistence setting for now.

See:

/kind bug

wallrj commented 6 years ago

This shouldn't happen after #256 but we can add validation to make the persistence setting immutable. Similar to what @kragniz did for Rack and DC