]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
mon/OSDMonitor: have `create` behavior on `new`
authorJoao Eduardo Luis <joao@suse.de>
Tue, 30 May 2017 12:34:20 +0000 (13:34 +0100)
committerJoao Eduardo Luis <joao@suse.de>
Mon, 5 Jun 2017 14:31:41 +0000 (15:31 +0100)
commit0df321b5e43e79393df962cd8b00bbb390c3fee6
treece9dae1364e23b3b4e625cab705b636e50e5f4eb
parentc9e6cac1cfae73cc4bce4a995ed2bba5dc85ae4b
mon/OSDMonitor: have `create` behavior on `new`

`osd new` will eventually replace `osd create`, and so we want to keep
as close of a behavior as possible. Some concessions needed to be made
however, specifically in terms of what the command arguments mean for
one command and the other.

For instance, and we think this is possibly the most contentious
decision, while specifying the `id` is absolutely optional, once it has
been specified we will take one of the following approaches:

 1. the osd is destroyed, and as such we will recreate the replacement
    osd.

 2. the osd exists and is not destroyed, thus we will assume it is an
    attempt at an idempotent operation; if not idempotent, fail because
    we then assume the osd was meant to be destroyed but it is not.

A commit should follow this patch series documenting the new commands.

Signed-off-by: Joao Eduardo Luis <joao@suse.de>
src/mon/AuthMonitor.cc
src/mon/ConfigKeyService.cc
src/mon/OSDMonitor.cc
src/mon/OSDMonitor.h