[libvirt] [PATCH] virsh: add aliases 'boot', 'stop', and 'restart'

[Viktor Mihajlovski]

On 11/05/2012 08:59 PM, Eric Blake wrote:
>
> This patch documents both spellings.  An alternative would be to
> leave the alternate spellings as hidden aliases (virsh has support
> for that), but still mention them in virsh.pod (see how we did an
> alias for nodedev-dettach, for reference).
In my opinion start, stop, restart is better than the mix before and 
semantically more accurate than boot, shutdown, reboot. The latter uses 
terms that apply to the operating system inside the domain rather than 
the domain (virtual machine) itself.
So I would even go so far to mark the latter group as legacy and 
recommend the usage of the former.
>
>   =item B<reboot> I<domain> [I<--mode acpi|agent>]
> +=item B<restart> I<domain> [I<--mode acpi|agent>]
>
>   Reboot a domain.  This acts just as if the domain had the B<reboot>
>   command run from the console.  The command returns as soon as it has
Now it would be a good time to correct the documentation. In reality the 
restart/reboot is a combination of stop/shutdown followed by an in-place 
reset of the domain (at least for QEMU/KVM). Which can behave the same 
as a reboot but doesn't have to. This is one area of confusion for users 
I have observed. Restart seems to raise a lower expectation level ;-).
For that reason I would also vote to exchange the order of appearance 
(restart first).
> @@ -1523,6 +1524,7 @@ be hot-plugged the next time the domain is booted.  As such, it must only be
>   used with the I<--config> flag, and not with the I<--live> flag.
>
>   =item B<shutdown> I<domain> [I<--mode acpi|agent>]
> +=item B<stop> I<domain> [I<--mode acpi|agent>]
>
Same suggestion: first line stop, second shutdown.


-- 

Mit freundlichen Grüßen/Kind Regards
    Viktor Mihajlovski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294