Re: [PATCH v2 0/5] domainRename API implementation

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

 



On 06.08.2015 12:21, Tomas Meszaros wrote:
> s is an effort to implement domain rename API. Presented patch series
> consists of the following: virDomainRename API implementation for qemu,
> implementation of the virsh command domrename and the additional support
> code.
> 
> The idea behind this endeavor is to provide convenient and safe way to rename
> a domain.
> 
> Instead of the:
> 
>     virsh dumpxml domain > domain.xml
>     (change domain name in domain.xml)
>     virsh undefine domain
>     virsh define domain.xml
> 
> user can simply type:
> 
>     virsh domrename foo bar
> 
> or call virDomainRename() API and domain "foo" will be renamed to "bar".
> 
> We currently support only renaming inactive domains without snapshots.
> Renaming procedure takes care of domain log, config, guest agent path and should
> be able to recover in case of failure.
> 
> I've been working on this functionality in collaboration with Michal Privoznik
> who is my mentor during the GSoC 2015. If you have any questions, ideas
> or criticism feel free to join the discussion.
> 
> 
> v2:
>  - removed guest agent path rename code
>  - removed rename permission
>  - added code for emitting undefined+renamed event for the old domain
> 
> 
> Tomas Meszaros (5):
>   Introduce virDomainRename API
>   virsh: Implement "domrename" command
>   domain_conf: Introducde virDomainObjListRenameAddNew() &
>     virDomainObjListRenameRemove()
>   Introduce new VIR_DOMAIN_EVENT_DEFINED_RENAMED event
>   qemu: Implement virDomainRename
> 
>  examples/object-events/event-test.c |   4 +
>  include/libvirt/libvirt-domain.h    |   4 +
>  src/access/viraccessperm.c          |   3 +-
>  src/access/viraccessperm.h          |   6 ++
>  src/conf/domain_conf.c              |  35 +++++++++
>  src/conf/domain_conf.h              |   5 ++
>  src/driver-hypervisor.h             |   5 ++
>  src/libvirt-domain.c                |  31 ++++++++
>  src/libvirt_private.syms            |   2 +
>  src/libvirt_public.syms             |   5 ++
>  src/qemu/qemu_driver.c              | 144 ++++++++++++++++++++++++++++++++++++
>  src/remote/remote_driver.c          |   1 +
>  src/remote/remote_protocol.x        |  17 ++++-
>  src/remote_protocol-structs         |   8 ++
>  tools/virsh-domain.c                |  63 +++++++++++++++-
>  tools/virsh.pod                     |   7 ++
>  16 files changed, 336 insertions(+), 4 deletions(-)
> 

I think this looks okay. Well, I have some findings in 1/5 but the rest
looks okay to me.

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]