search

Netflix / dynomite-manager

A sidecar to manage Dynomite clusters
https://github.com/Netflix/dynomite
Apache License 2.0
91 stars 59 forks source link

add deprecation warning for DM_CASSANDRA_CLUSTER_SEEDS #63

Closed akbarahmed closed 7 years ago

ipapapa commented 7 years ago

We probably want to wrap some small tests around it, but accepting it to solve the issue on Gitter.

akbarahmed commented 7 years ago

@ipapapa The test is in DynomiteManagerConfigurationTest.java in the testGetCassandraSeeds() method. The test checks that DM conf gets the env var, but the test does not check for a specific env var.

ipapapa commented 7 years ago

@akbarahmed I am sorry for missing that test.