[vdsm] addNetwork to use existing nic conf

Antoni Segura Puimedon asegurap at redhat.com
Wed Nov 27 14:54:06 UTC 2013



----- Original Message -----
> From: "Yedidyah Bar David" <didi at redhat.com>
> To: "Dan Kenigsberg" <danken at redhat.com>
> Cc: "Antoni Segura Puimedon" <asegurap at redhat.com>, "vdsm-devel" <vdsm-devel at fedorahosted.org>
> Sent: Wednesday, November 27, 2013 3:47:20 PM
> Subject: Re: [vdsm] addNetwork to use existing nic conf
> 
> ----- Original Message -----
> > From: "Dan Kenigsberg" <danken at redhat.com>
> > To: "Antoni Segura Puimedon" <asegurap at redhat.com>
> > Cc: "Yedidyah Bar David" <didi at redhat.com>, "vdsm-devel"
> > <vdsm-devel at fedorahosted.org>
> > Sent: Wednesday, November 27, 2013 2:51:37 PM
> > Subject: Re: [vdsm] addNetwork to use existing nic conf
> > 
> > On Wed, Nov 27, 2013 at 07:28:23AM -0500, Antoni Segura Puimedon wrote:
> > > 
> > > 
> > > ----- Original Message -----
> > > > From: "Dan Kenigsberg" <danken at redhat.com>
> > > > To: "Antoni Segura Puimedon" <asegurap at redhat.com>
> > > > Cc: "Yedidyah Bar David" <didi at redhat.com>, "vdsm-devel"
> > > > <vdsm-devel at fedorahosted.org>
> > > > Sent: Wednesday, November 27, 2013 1:22:47 PM
> > > > Subject: Re: [vdsm] addNetwork to use existing nic conf
> > > > 
> > > > On Wed, Nov 27, 2013 at 06:55:26AM -0500, Antoni Segura Puimedon wrote:
> > > > > 
> > > > > 
> > > > > ----- Original Message -----
> > > > > > From: "Yedidyah Bar David" <didi at redhat.com>
> > > > > > To: "Antoni Segura Puimedon" <asegurap at redhat.com>
> > > > > > Cc: "vdsm-devel" <vdsm-devel at fedorahosted.org>, "Alon Bar-Lev"
> > > > > > <alonbl at redhat.com>, "Sandro Bonazzola"
> > > > > > <sbonazzo at redhat.com>
> > > > > > Sent: Wednesday, November 27, 2013 11:18:39 AM
> > > > > > Subject: Re: addNetwork to use existing nic conf
> > > > > > 
> > > > > > ----- Original Message -----
> > > > > > > From: "Antoni Segura Puimedon" <asegurap at redhat.com>
> > > > > > > To: "Yedidyah Bar David" <didi at redhat.com>
> > > > > > > Cc: "vdsm-devel" <vdsm-devel at fedorahosted.org>, "Alon Bar-Lev"
> > > > > > > <alonbl at redhat.com>, "Sandro Bonazzola"
> > > > > > > <sbonazzo at redhat.com>
> > > > > > > Sent: Wednesday, November 27, 2013 11:52:51 AM
> > > > > > > Subject: Re: addNetwork to use existing nic conf
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > ----- Original Message -----
> > > > > > > > From: "Yedidyah Bar David" <didi at redhat.com>
> > > > > > > > To: "vdsm-devel" <vdsm-devel at fedorahosted.org>
> > > > > > > > Cc: "Alon Bar-Lev" <alonbl at redhat.com>, "Sandro Bonazzola"
> > > > > > > > <sbonazzo at redhat.com>, "Antoni Segura Puimedon"
> > > > > > > > <asegurap at redhat.com>
> > > > > > > > Sent: Wednesday, November 27, 2013 10:38:54 AM
> > > > > > > > Subject: addNetwork to use existing nic conf
> > > > > > > > 
> > > > > > > > Hi all,
> > > > > > > > 
> > > > > > > > While looking at [1], and reading the source of bridge.py of
> > > > > > > > ovirt-hosted-engine-setup
> > > > > > > > and of ovirt-host-deploy, I think the correct thing will be for
> > > > > > > > addNetwork
> > > > > > > > to
> > > > > > > > not
> > > > > > > > require either bootproto=dhcp or ipaddr/netmask/etc., but to
> > > > > > > > take
> > > > > > > > this
> > > > > > > > from
> > > > > > > > the existing
> > > > > > > > conf of the nics to be put inside the bridge.
> > > > > > > 
> > > > > > > You mean the addNetwork in vdsm/vdsm/configNetwork.py?
> > > > > > 
> > > > > > We currently use vdsClient - 'vdsClient addNetwork'.
> > > > > > 
> > > > > > > If that is the case I would very much prefer not to make such
> > > > > > > change,
> > > > > > > since
> > > > > > > that
> > > > > > > would make addNetwork even farther from referential transparency.
> > > > > > > 
> > > > > > > What I propose is that the calling code will use the new
> > > > > > > getBootProto
> > > > > > > introduced in:
> > > > > > > http://gerrit.ovirt.org/18484 (already merged).
> > > > > > 
> > > > > > Does vdsClient support that?
> > > > > 
> > > > > Well, vdsClient getVdsCaps returns the cfg information for all the
> > > > > devices,
> > > > > part of that will be the bootproto if it is defined on the ifcfg file
> > > > > of
> > > > > the
> > > > > device.
> > > > 
> > > > We'd very much like to drop the 'cfg' element reported per nic (it
> > > > breaks Vdsm's "network" abrstraction and would be anoying to maintain
> > > > with iproute2 when we move to it. "BOOTPROTO" is there, and would have
> > > > to stay there (for backward compatibility with Engine), but please
> > > > avoid
> > > > using any other data.
> > > 
> > > Well, for installation time, there must be something (not necessarily
> > > part
> > > of
> > > vdsm proper that is able to get the ip configuration of the device we
> > > connect
> > > through.
> > > 
> > > Since it could be ifcfg info (el6), NM (newer distros),
> > > /etc/network/interfaces
> > > (debian derived) I would really make a separate solution to vdsm.
> > 
> > No doubt about that:
> >     Bug 987813 - [RFE] report BOOTPROTO and BONDING_OPTS independent of
> >     netdevice.cfg
> > 
> 
> Sorry, I did not understand. Should I or should I not rely on getVdsCaps
> to get the ipaddr/netmask/gateway/whatever? Just getting bootproto from it
> does not help much if I have to read ifcfg files anyway.

getVdsCaps as it stands now is only working on el6 and Fedora for getting the
bootProto in the cfg field.

> --
> Didi
> 


More information about the vdsm-devel mailing list