feat: add basic support for multi-cluster replication #494
Conversation
|
Kudos, SonarCloud Quality Gate passed!
|
There was a problem hiding this comment.
Hi @arlyon ! Thanks this is huge !!
I agree that this is the first step before a more complete replication handling by the operator. For now the documentation is sufficient.
I added some comments on the API side.
|
apologies for the delay, new version up now |
arlyon
force-pushed
the
feat/cluster-replication
branch
from
4360579 to
e1aea29
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
|
Hi @arlyon ! |
arlyon
force-pushed
the
feat/cluster-replication
branch
from
e1aea29 to
78e163c
Compare
|
Hi, I am about to push another commit after rebasing. I can't run kind on my machine (rootless podman) so this might be a little tedious but it seems like a simple enough fix. |
|
|
Hey folks, is this going to land soon? Multi-cluster replication would be huge for migrating to this operator and afaik it's the only way to change shard-count without killing the cluster! |
arlyon
force-pushed
the
feat/cluster-replication
branch
from
33ae24c to
6e8c21d
Compare
|
I fixed the lint error now. It is slow going as I can't run the tests locally since they rely on Hopefully we can get it landed soon. |
arlyon
force-pushed
the
feat/cluster-replication
branch
2 times, most recently
from
26f5f1a to
a873d99
Compare
|
Hey @alexandrevilain apologies for the back-and-forth. I got access to a machine that can run |
Greeeat! I’ll review this as soon as possible! Thanks for your work! 👌 |
|
|
Hi @arlyon ! Do you need so help for the last few steps ? :) |
arlyon
force-pushed
the
feat/cluster-replication
branch
from
73a7af8 to
4892e87
Compare
|
This adds the bare minimum changes to enable multi-cluster replication, and adds some docs to explain concretely how to achieve it.