[libvirt] [PATCH] tools: avoid text spilling into variables
Michal Privoznik
mprivozn at redhat.com
Fri Feb 23 12:43:47 UTC 2018
On 01/16/2018 04:05 PM, Dariusz Gadomski wrote:
> From: Christian Ehrhardt <christian.ehrhardt at canonical.com>
>
> While libvirt-guests.sh is running cases can let guest_is_on fail which
> causes check_guests_shutdown to print output.
> That output shall not spill into the users of function
> check_guests_shutdown which is therefore now returning values in a
> variable like guest_is_on already did.
>
> Original-Author: Christian Ehrhardt <christian.ehrhardt at canonical.com>
> Modified-By: Jorge Niedbalski <niedbalski at ubuntu.com>
> ---
> tools/libvirt-guests.sh.in | 15 +++++++++------
> 1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/tools/libvirt-guests.sh.in b/tools/libvirt-guests.sh.in
> index 8a158cca4..91a2f3283 100644
> --- a/tools/libvirt-guests.sh.in
> +++ b/tools/libvirt-guests.sh.in
> @@ -329,12 +329,13 @@ guest_count()
> # check_guests_shutdown URI GUESTS
> # check if shutdown is complete on guests in "GUESTS" and returns only
> # guests that are still shutting down
> +# Result is returned in "guests_shutting_down"
> check_guests_shutdown()
> {
> uri=$1
> guests=$2
>
> - guests_up=
> + guests_shutting_down=
> for guest in $guests; do
> if ! guest_is_on "$uri" "$guest" >/dev/null 2>&1; then
> eval_gettext "Failed to determine state of guest: \$guest. Not tracking it anymore."
> @@ -342,10 +343,9 @@ check_guests_shutdown()
> continue
> fi
> if "$guest_running"; then
> - guests_up="$guests_up $guest"
> + guests_shutting_down="$guests_shutting_down $guest"
> fi
> done
> - echo "$guests_up"
> }
>
> # print_guests_shutdown URI BEFORE AFTER
> @@ -392,8 +392,10 @@ shutdown_guests_parallel()
> guest=$1
> shift
> guests=$*
> - shutdown_guest_async "$uri" "$guest"
> - on_shutdown="$on_shutdown $guest"
> + if [ -z "$(echo $on_shutdown | grep $guest)" -a -n "$(guest_name "$uri" "$guest")" ]; then
We prefer if test cond1 && test cond2 over -a.
I've fixed that, ACKed and pushed.
Michal
More information about the libvir-list
mailing list