[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