Re: [PATCH] virsh: Report name from edited object

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

 



On 14.06.2012 14:57, Eric Blake wrote:
> On 06/14/2012 06:50 AM, Michal Privoznik wrote:
>> There is a little easter egg in virsh: one can easily clone
>> an object (domain, network, ...). Just 'virsh edit' change the name
>> and remove <uuid>. And then, in the end when reporting success
>> the new name was printed out.
>> However, with recent edit rewrite we lost the final part and are
>> still printing the original name out.
>> ---
>> Or even better - if name changes, we can choose better message,
>> something like "New domain %s defined". But I am not sure
>> we want to expose this easter egg to users.
> 
> Yeah, I ran into some of that fun when implementing snapshot-edit.
> 
> I think your patch goes far enough; printing the final name is good
> enough, without having to try and produce a smarter output message.
> 
> ACK.
> 

Thanks, pushed.

Michal

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


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]