Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-22 Thread Daniel Veillard
On Thu, Sep 18, 2008 at 03:47:53PM +0400, Evgeniy Sokolov wrote: > >> On Wed, Sep 17, 2008 at 08:28:56PM +0400, Evgeniy Sokolov wrote: >>> I attached patch without hunk which was commited by Daniel. >>> Please commit if you are agree with it. >> >> A few comments inline.. >> > ... >> >> So you're n

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-18 Thread Evgeniy Sokolov
On Wed, Sep 17, 2008 at 08:28:56PM +0400, Evgeniy Sokolov wrote: I attached patch without hunk which was commited by Daniel. Please commit if you are agree with it. A few comments inline.. So you're not artifically restricting the max length of the network name. It is limit in OpenV

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-17 Thread Daniel P. Berrange
On Wed, Sep 17, 2008 at 08:28:56PM +0400, Evgeniy Sokolov wrote: > > I attached patch without hunk which was commited by Daniel. > Please commit if you are agree with it. A few comments inline.. > Index: openvz_conf.c > === > RCS fi

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-17 Thread Evgeniy Sokolov
On Mon, Sep 08, 2008 at 03:59:55PM +0400, Evgeniy Sokolov wrote: This patch add possibility to load network configuration for container in OpenVZ driver. It support routing network and bridge devices. This types is currently supported in create/define command. Also, patch contain fix to sma

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-17 Thread Richard W.M. Jones
On Mon, Sep 08, 2008 at 03:59:55PM +0400, Evgeniy Sokolov wrote: > This patch add possibility to load network configuration for container > in OpenVZ driver. > It support routing network and bridge devices. This types is currently > supported in create/define command. > Also, patch contain fix

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-15 Thread Daniel Veillard
On Thu, Sep 11, 2008 at 11:42:21AM +0400, Evgeniy Sokolov wrote: >> On Mon, Sep 08, 2008 at 03:59:55PM +0400, Evgeniy Sokolov wrote: >>> This patch add possibility to load network configuration for >>> container in OpenVZ driver. >>> It support routing network and bridge devices. This types is >

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-11 Thread Evgeniy Sokolov
On Mon, Sep 08, 2008 at 03:59:55PM +0400, Evgeniy Sokolov wrote: This patch add possibility to load network configuration for container in OpenVZ driver. It support routing network and bridge devices. This types is currently supported in create/define command. Also, patch contain fix to small

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-09 Thread Evgeniy Sokolov
On Mon, Sep 08, 2008 at 03:59:55PM +0400, Evgeniy Sokolov wrote: This patch add possibility to load network configuration for container in OpenVZ driver. It support routing network and bridge devices. This types is currently supported in create/define command. Also, patch contain fix to small

Re: [libvirt] [PATCH] read network config in OpenVZ driver

2008-09-08 Thread Daniel Veillard
On Mon, Sep 08, 2008 at 03:59:55PM +0400, Evgeniy Sokolov wrote: > This patch add possibility to load network configuration for container > in OpenVZ driver. > It support routing network and bridge devices. This types is currently > supported in create/define command. > Also, patch contain fix

[libvirt] [PATCH] read network config in OpenVZ driver

2008-09-08 Thread Evgeniy Sokolov
This patch add possibility to load network configuration for container in OpenVZ driver. It support routing network and bridge devices. This types is currently supported in create/define command. Also, patch contain fix to small bug in openvzReadConfigParam method. Index: openvz_conf.c =