Closed necaris closed 1 year ago
cc also @marcosmoyano
I think this is something that should be reflected in the docs since having two clusters with the same name on two different regions is possible. The region then becomes the differential factor.
Hello, I've tested this issue with:
cluster_one = coiled.Cluster(name="test-region", backend_options={"region": "us-west-1"})
# wait for cluster to spin up
cluster_two = coiled.Cluster(name="test-region", backend_options={"region": "us-east-2"})
And got the message: "Using existing cluster: test-region". After speaking with Rami, we might want to do one of the two:
I'm just updating this issue and perhaps spark some discussion as to what would be better to do here 😄
fixed
/cc @quasiben
When connecting to an existing cluster with
coiled.Cluster(name="foo")
, it errors with:when connecting to the existing cluster is exactly what's wanted. It turns out that passing the
region=
argument is needed; this should be in the docs (or ideally, be unnecessary) :smile: