- using the deactivate/destroy feature to destroy osd
- test reactivate option when the osd goes deactive
- add check_osd_status to check osd status when osd goes up
Signed-off-by: Vicente Cheng <freeze.bilsted@gmail.com>
- modify the behavior to handle dmcrypt/mpath osd
- add some functions to get the information of dmcrypt osd
- fixed the logging format Use `%(filename)s` instead of `%(filename)`
Signed-off-by: Vicente Cheng <freeze.bilsted@gmail.com>
- Add some unit test to coverage all destroy/deactivate feature.
- Do some minor modifications on the ceph-diskw
Signed-off-by: Vicente Cheng <freeze.bilsted@gmail.com>
- add `--reactivate` option (activate do no-op without reactivate with deactive flag)
- for consistency, make both deactivate and destroy take the device/partition name
- add `--deactivate-by-id` to deactivate and destroy for ease of use
Signed-off-by: Vicente Cheng <freeze.bilsted@gmail.com>
- Using `--mark-out` option to mark osd out when
deactivate this osd instead of always mark osd out.
Signed-off-by: Vicente Cheng <freeze.bilsted@gmail.com>
Implement deactivate option on ceph-disk.
- stop ceph-osd service if needed (If osd still in osd map, make it out first)
- remove 'ready', 'active', and INIT-specific files
- remove gpt partition type and change partition name (prevent triggered by udev)
- create deactive flag
- umount device and remove mount point
Signed-off-by: Vicente Cheng <freeze.bilsted@gmail.com>
--user is the equivalent of --id (mentioned in docs and ceph -h) and
used by at least a couple of qa scripts and teuthology. Add it to the
list of valid rbd options - it's handled inside global_init() before
Shell:execute() is called.
OTOH allowing -i is bogus - global_init() wouldn't process it for
clients (CEPH_ENTITY_TYPE_CLIENT), which rbd tool is one of. Drop it.
Signed-off-by: Ilya Dryomov <idryomov@redhat.com>
Now that the tests at the top level directory are run via TESTS and not
check-local:, the test-suite.log file is created and must bit
gitignored.
Signed-off-by: Loic Dachary <loic@dachary.org>
Previously, if snapname wasn't specified we would pass NULL to
krbd_map(), which was a cue for it to use "-" as a snapshot name. With
the new rbd CLI, "" is passed in; same goes for map options.
Change krbd_map() accordingly and update its other user.
Signed-off-by: Ilya Dryomov <idryomov@redhat.com>
osd: FileStore: support multiple ondisk finish and apply finishers
Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: Samuel Just <sjust@redhat.com>
The shell used by the cli tests is not always bash. Not using the
here-word is also more readable in this specific case.
Signed-off-by: Loic Dachary <loic@dachary.org>
Avoid dynamically detecting the application name and instead hardcode
the rbd CLI name into the help output.
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
e4ca468 moved src/test/run-cli-tests from check-local to check_SCRIPTS
but did not add it to the TESTS variable.
Signed-off-by: Loic Dachary <loic@dachary.org>
