Project

General

Profile

Actions

Bug #806

closed

init-ceph should use do_cmd consistently, or not at all

Added by Colin McCabe about 13 years ago. Updated about 13 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
% Done:

0%

Source:
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

init-ceph sometimes uses do_cmd, and sometimes not.

Example:

 [ -n "$pid_file" ] && mkdir -p `dirname $pid_file`
...
 [ -n "$log_dir" ] && do_cmd "mkdir -p $log_dir" 

The second one uses do_cmd, but the first one doesn't. do_cmd seems to involve some logic to inject 'ssh' if it feels that it needs to.

It's probably best if we don't go running ssh inside an init script. It's weird and non-traditional, and most sysadmins will be confused. If we do decide to do it, though, we should at least be consistent and use do_cmd everywhere. That will also mean getting rid of things like [ -e $pidfile ], which implicitly assume that we're running locally. (Another reason NOT to use do_cmd.)

Actions

Also available in: Atom PDF