[libvirt] PATCH: 4/7: Convert LXC to new domain APIs
Jim Meyering
jim at meyering.net
Tue Aug 12 14:39:05 UTC 2008
"Daniel P. Berrange" <berrange at redhat.com> wrote:
> The re-architecting of the LXC controller/container process relationship
> in the previous patch removed the last obstacle to switching over to the
> generic domain XML routines. So this patch switches the driver over.
>
> First the vast majority of lxc_conf.h/c is simply deleted - this is all
> redundant when using the domain_conf.h APIs. Then, all references to lxc_vm_t
> are changed to virDomainObj, and lxc_vm_def_t switches to virDomainDef.
> Finally the LXC driver registers its capabilities data. For this I have
> chosen an OS type of 'exe', since the 'operating system' we're running
> in the container is just any plain executable process.
>
>
> lxc_conf.c | 1052 +------------------------------------------------------
> lxc_conf.h | 121 ------
> lxc_container.c | 23 -
> lxc_container.h | 2
> lxc_controller.c | 4
> lxc_controller.h | 2
> lxc_driver.c | 289 ++++++++-------
> 7 files changed, 215 insertions(+), 1278 deletions(-)
All looks fine.
ACK.
However, please note that it would have been a lot easier/quicker
to review if you'd done the mechanical/automatable changes
(i.e., the global substitutions like s/lxc_vm_t/virDomainObj/)
separately from the others.
More information about the libvir-list
mailing list