On Wed, Sep 23, 2015 at 09:33:14AM +0300, Mykola Golub wrote: > Also, I am not sure we should specify <pool-name> this way, as it is > not consistent with other rbd commands. By default rbd operates on > 'rbd' pool, which can be changed by --pool option. The same reasoning for these commands: > > * rbd mirror pool add <remote-pool-spec> > > This will register a remote cluster/pool as a peer to the current, > > local pool. All existing mirrored images and all future mirrored > > images will have this peer registered as a journal client. > > > > * rbd mirror pool remove <remote-pool-spec> > > This will deregister a remote cluster/pool as a peer to the > > current, > > local pool. All existing mirrored images will have the remote > > deregistered from image journals. I am not sure we need 'pool' here. I would prefer: rbd mirror peer add <remote-peer-spec> rbd mirror peer remove <remote-peer-spec> rbd mirror peer show Where, <remote-peer-spec> should not necessary contain pool, because the default could be used or what is specified via '--pool' option. -- Mykola Golub -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
