[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 1/4] libvirt-guest.init: handle domain name with spaces



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 <hahn 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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]