+1, this is a good improvement. In the proposal, I don't suggest you append the `local` to the cluster name:
1. It will be confusing when the original cluster name includes the `local` name 2. In some cases, it is not easy to distinguish between local and remote clusters so I suggest we should add the `local` and `remote` flags like: ``` # list remote clusters bin/pulsar-admin clusters list --remote us-east us-cent # list local clustes bin/pulsar-admin clusters list --local us-west # list all clusters bin/pulsar-admin clusters list us-east us-cent us-west ``` Thanks, Zixuan guo jiwei <techno...@apache.org> 于2023年6月20日周二 14:00写道： > > Hi community: > After configuring the geo-replication on Pulsar clusters, the `clusters > list` API will return multiple clusters, including the local Pulsar cluster > and remote clusters like > > ``` > bin/pulsar-admin clusters list > us-west > us-east > us-cent > ``` > But in this return, you can't distinguish the local and the remote cluster. > When you need to remove the geo-replication configuration, it will be hard > to decide which cluster should be removed on replicated tenants and > namespaces unless you record the cluster information. > > > ### Modification > Add `local` flag to distinguish clusters > ``` > bin/pulsar-admin clusters list > us-west(local) > us-east > us-cent > ``` > > PIP: https://github.com/apache/pulsar/pull/20614 > > > Regards > Jiwei Guo (Tboy)
