diff options
| author | Martin Schwenke <martin@meltin.net> | 2019-06-28 15:44:59 +1000 |
|---|---|---|
| committer | Amitay Isaacs <amitay@samba.org> | 2019-07-05 05:03:22 +0000 |
| commit | f3feb4df3a90bd40998fa60887a56961954f6423 (patch) | |
| tree | fc1e60d3fb3534fb8468243116429eb86a3869e1 /ctdb/tools/onnode | |
| parent | 90de5e0594b9180226b9a13293afe31f18576b3d (diff) | |
| download | samba-f3feb4df3a90bd40998fa60887a56961954f6423.tar.gz | |
ctdb-tools: Drop no-op stdout-filter from non-parallel case
This filter no longer does anything useful in this context. By
default it adds a pipeline with trailing cat process. In many
contexts, stdout of the process being run is still open so the cat
process will stay around and will stop onnode from exiting.
The filters should all go away because they are simply an example of
code that is trying to be too clever while causing unfortunate corner
cases.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Diffstat (limited to 'ctdb/tools/onnode')
| -rwxr-xr-x | ctdb/tools/onnode | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ctdb/tools/onnode b/ctdb/tools/onnode index 76cd2217a0d..863b8b393c5 100755 --- a/ctdb/tools/onnode +++ b/ctdb/tools/onnode @@ -352,7 +352,7 @@ for n in $nodes ; do echo >&2 ; echo ">> NODE: $n <<" >&2 fi { - $ssh_cmd "$n" "$command" | stdout_filter + $ssh_cmd "$n" "$command" } || retcode=$? fi done |