scylladb / scylla-manager

The Scylla Manager
https://manager.docs.scylladb.com/stable/
Other
48 stars 33 forks source link

testing: enable sstable uuid identifiers in Scylla config #3883

Closed karol-kokoszka closed 2 weeks ago

karol-kokoszka commented 2 weeks ago

This PR enables UUID in SSTables names feature in Scylla Cluster against which all tests are being executed.


Please make sure that:

Michal-Leszczynski commented 2 weeks ago

It won't hurt to specify it manually, but I believe that sstable UUID identifiers are enabled by default.

karol-kokoszka commented 2 weeks ago

Actually you're right :) There is no need to set it explicitly. 5.4.1 (the oldest we use to execute tests against) keeps it as a default.

karol-kokoszka commented 2 weeks ago

OK, 2023.1 needs explicit config value if we want to support UUID identifiers.

But we execute the tests against newer version that keeps this UUID enabled by default.

So, we cover both options (sstables with/without UUID).

This PR definitely makes no sense.