Re: [Qemu-devel] [PATCH] iotests: fix the redirection order in 083
26.04.2016 13:13, Wei Jiangang wrote: > It should redirect stdout to /dev/null first, > then redirect stderr to whatever stdout currently points at. Actually this is interesting. By doing this like it was done initially, we see any possible errors from grep or python, because errors will go to initial stdout. Now, errors are sent to /dev/null too. :) JFYI. /mjt > Signed-off-by: Wei Jiangang> --- > tests/qemu-iotests/083 | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/qemu-iotests/083 b/tests/qemu-iotests/083 > index aa99278..7d368b5 100755 > --- a/tests/qemu-iotests/083 > +++ b/tests/qemu-iotests/083 > @@ -44,7 +44,7 @@ choose_tcp_port() { > > wait_for_tcp_port() { > while ! (netstat --tcp --listening --numeric | \ > - grep "$1.*0\\.0\\.0\\.0:\\*.*LISTEN") 2>&1 >/dev/null; do > + grep "$1.*0\\.0\\.0\\.0:\\*.*LISTEN") >/dev/null 2>&1; do > sleep 0.1 > done > } > @@ -71,7 +71,7 @@ EOF > nbd_url="nbd:127.0.0.1:$port:exportname=foo" > fi > > - $PYTHON nbd-fault-injector.py $extra_args "127.0.0.1:$port" > "$TEST_DIR/nbd-fault-injector.conf" 2>&1 >/dev/null & > + $PYTHON nbd-fault-injector.py $extra_args "127.0.0.1:$port" > "$TEST_DIR/nbd-fault-injector.conf" >/dev/null 2>&1 & > wait_for_tcp_port "127\\.0\\.0\\.1:$port" > $QEMU_IO -c "read 0 512" "$nbd_url" 2>&1 | _filter_qemu_io | _filter_nbd > >
Re: [Qemu-devel] [PATCH] iotests: fix the redirection order in 083
On 26.04.2016 12:13, Wei Jiangang wrote: > It should redirect stdout to /dev/null first, > then redirect stderr to whatever stdout currently points at. > > Signed-off-by: Wei Jiangang> --- > tests/qemu-iotests/083 | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Thanks, applied to my block-next tree: https://github.com/XanClic/qemu/commits/block-next Max signature.asc Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH] iotests: fix the redirection order in 083
On 04/26/2016 04:13 AM, Wei Jiangang wrote: > It should redirect stdout to /dev/null first, > then redirect stderr to whatever stdout currently points at. > > Signed-off-by: Wei Jiangang> --- > tests/qemu-iotests/083 | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature
[Qemu-devel] [PATCH] iotests: fix the redirection order in 083
It should redirect stdout to /dev/null first, then redirect stderr to whatever stdout currently points at. Signed-off-by: Wei Jiangang--- tests/qemu-iotests/083 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/qemu-iotests/083 b/tests/qemu-iotests/083 index aa99278..7d368b5 100755 --- a/tests/qemu-iotests/083 +++ b/tests/qemu-iotests/083 @@ -44,7 +44,7 @@ choose_tcp_port() { wait_for_tcp_port() { while ! (netstat --tcp --listening --numeric | \ -grep "$1.*0\\.0\\.0\\.0:\\*.*LISTEN") 2>&1 >/dev/null; do +grep "$1.*0\\.0\\.0\\.0:\\*.*LISTEN") >/dev/null 2>&1; do sleep 0.1 done } @@ -71,7 +71,7 @@ EOF nbd_url="nbd:127.0.0.1:$port:exportname=foo" fi - $PYTHON nbd-fault-injector.py $extra_args "127.0.0.1:$port" "$TEST_DIR/nbd-fault-injector.conf" 2>&1 >/dev/null & + $PYTHON nbd-fault-injector.py $extra_args "127.0.0.1:$port" "$TEST_DIR/nbd-fault-injector.conf" >/dev/null 2>&1 & wait_for_tcp_port "127\\.0\\.0\\.1:$port" $QEMU_IO -c "read 0 512" "$nbd_url" 2>&1 | _filter_qemu_io | _filter_nbd -- 1.9.3