search

scylladb / scylla-ccm

Cassandra Cluster Manager, modified for Scylla
Apache License 2.0
20 stars 64 forks source link

Add support for multi-dc cloud setup #444

Closed Gor027 closed 1 year ago

Gor027 commented 1 year ago

Currently, only one SNI proxy container is created for the whole cluster for serverless support. So, the generated configuration files config_data.yaml and config_path.yaml contain only local datacenter information. This PR adds support for multi-dc cloud setup, so when starting a multi-DC cluster with SNI proxy, an SNI proxy container will be created per DC, so that each one can only connect to nodes in their datacenter. This will allow for better testing of our drivers after adding serverless support.

fruch commented 1 year ago

@Gor027 Can you take for a ride with the python-driver, to verify it's not breaking it ?

I won't be able to get to it until Monday...

Gor027 commented 1 year ago

@Gor027 Can you take for a ride with the python-driver, to verify it's not breaking it ?

I won't be able to get to it until Monday...

I tested these changes with python-driver by triggering CI on my fork with scylla-ccm installed from my branch: https://github.com/Gor027/python-driver/pull/1

Gor027 commented 1 year ago

@fruch ping

fruch commented 1 year ago

seem like @avelanarius found the issue with CI: https://github.com/scylladb/scylla-ccm/issues/448