On Tue, Mar 01, 2011 at 18:57:01 +0100, Philipp Hahn wrote:
> awk splits the line on consecutive spaces, which breaks getting the name
> of a domain whose name contains spaces. Use sed instead to strip the
> "Name:" prefix from the line
> 
> Signed-off-by: Philipp Hahn <h...@univention.de>
> ---
>  tools/libvirt-guests.init.sh |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/tools/libvirt-guests.init.sh b/tools/libvirt-guests.init.sh
> index 8823d06..ea2189e 100644
> --- a/tools/libvirt-guests.init.sh
> +++ b/tools/libvirt-guests.init.sh
> @@ -105,7 +105,7 @@ guest_name() {
>      uuid=$2
>  
>      name=$(run_virsh_c $uri dominfo $uuid 2>/dev/null | \
> -           awk '/^Name:/{print $2}')
> +           sed -ne 's/^Name: *//p')
>      [ -n "$name" ] || name=$uuid
>  
>      echo "$name"

Ah yes, domain names with spaces... I guess we should enhance virsh so that it
can start domains based on UUID so the we can get rid of the last domain name
usage when calling virsh in this script.

ACK

Jirka

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to