]> xenbits.xensource.com Git - libvirt.git/commitdiff
tools: do not report unknown guests in print_guests_shutdown
authorChristian Ehrhardt <christian.ehrhardt@canonical.com>
Thu, 19 Apr 2018 06:41:03 +0000 (08:41 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 24 Apr 2018 08:46:41 +0000 (10:46 +0200)
If another event in background while running libvirt-guests.sh
completely undefines a guest it will no more be available for proper
reporting of its shutdown.

This appears in the log as:
  Failed to determint state of guest: <UUID>. Not tracking it anymore
  Shutdown of guest  complete

The first message already reports that we are giving up on the guest
(per UUID which is all we have left at that point). To avoid the message
with an empty guest_name in such a case lets check what guest_name
returned and only print a report on valid content.

Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Andrea Bolognani <abologna@redhat.com>
Reviewed-by: Dariusz Gadomski <dariusz.gadomski@canonical.com>
tools/libvirt-guests.sh.in

index 28fe5060174a1fb7df4a48232b3ec1808228894e..4bc6e866f0ad9ce0b95bfb1ee2d87542e3a2a1ba 100644 (file)
@@ -363,8 +363,10 @@ print_guests_shutdown()
         esac
 
         name=$(guest_name "$uri" "$guest")
-        eval_gettext "Shutdown of guest \$name complete."
-        echo
+        if [ -n "$name" ]; then
+            eval_gettext "Shutdown of guest \$name complete."
+            echo
+        fi
     done
 }