k8ssandra / management-api-for-apache-cassandra

RESTful / Secure Management Sidecar for Apache Cassandra
Apache License 2.0
69 stars 51 forks source link

Remove `cassandra-topology.properties` from the configuration folder #463

Closed adejanovski closed 3 months ago

adejanovski commented 3 months ago

In order to avoid some weird and annoying bugs with the GossipPropertyFileSnitch still considering the cassandra-topology.properties file in some cases, which could cause unsafe instant topology changes, we should remove the file altogether at runtime.

### Definition of Done
- [ ] The `cassandra-topology.properties` is deleted from disk before Cassandra starts up
emerkle826 commented 3 months ago

@bradfordcp @adejanovski Do we need to do this for DSE as well, or just OSS Cassandra images?

adejanovski commented 3 months ago

I'd say both. We don't want this file anywhere.