Re: [vdsm] configNetwork issue

2013-06-09 Thread Dan Kenigsberg 
On Thu, Jun 06, 2013 at 05:14:17PM +0200, Sandro Bonazzola wrote:
> Il 06/06/2013 17:10, Giuseppe Vallarelli ha scritto:
> >
> > - Original Message -
> > | From: "Sandro Bonazzola" 
> > | To: "Alon Bar-Lev" 
> > | Cc: vdsm-devel@lists.fedorahosted.org
> > | Sent: Thursday, June 6, 2013 4:38:41 PM
> > | Subject: Re: [vdsm] configNetwork issue
> > | 
> > | Il 06/06/2013 16:29, Alon Bar-Lev ha scritto:
> > | > /usr/share/vdsm/addNetwork', 'ovirtmgmt', '', '', u'eth0',
> > | > 'BOOTPROTO=dhcp', 'ONBOOT=yes', 'blockingdhcp=true']
> > | 
> > | # /usr/share/vdsm/addNetwork ovirtmgmt '' '' em1 BOOTPROTO=dhcp
> > | ONBOOT=yes blockingdhcp=true
> > | WARNING:root:options BOOTPROTO is deprecated. Use bootproto instead
> > | WARNING:root:options ONBOOT is deprecated. Use onboot instead
> > | INFO:root:Adding network ovirtmgmt with vlan=, bonding=, nics=['em1'],
> > | bondingOptions=None, mtu=None, bridged=True, options={'bootproto':
> > | 'dhcp', 'blockingdhcp': 'true', 'onboot': 'yes'}
> > | Traceback (most recent call last):
> > |   File "/usr/lib64/python2.7/runpy.py", line 162, in _run_module_as_main
> > | "__main__", fname, loader, pkg_name)
> > |   File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code
> > | exec code in run_globals
> > |   File "/usr/share/vdsm/configNetwork.py", line 664, in 
> > | main()
> > |   File "/usr/share/vdsm/configNetwork.py", line 633, in main
> > | addNetwork(bridge, **kwargs)
> > |   File "/usr/share/vdsm/configNetwork.py", line 198, in addNetwork
> > | configurator, **options)
> > | TypeError: objectivizeNetwork() got multiple values for keyword argument
> > | 'bootproto'
> >
> > Hi Sandro, a very dirty and quick fix may be the following one.
> >
> > Inside configNetwork reach the addNetwork function and
> > before the call to objectivizeNetwork:
> >
> > bootproto =options.get('bootproto') 
> > 
> >   
> > if bootproto:
> > del options['bootproto']
> > netEnt = objectivizeNetwork(network if bridged else None, vlan, bonding,
> >  bondingOptions, nics, mtu, ipaddr, netmask,
> >  gateway, bootproto, _netinfo,
> >  configurator, **options)
> >
> > Giuseppe
> >
> 
> Thanks Giuseppe, that seems to fix the issue also if I don't really like
> using del as solution.
> This however is a workaround that fix it for me.

A fix by Giuseppe has been merged upstream
http://gerrit.ovirt.org/#/c/15423/ .
Thanks for reporting the breakage on master branch.

Dan.
___
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Re: [vdsm] configNetwork issue

2013-06-06 Thread Sandro Bonazzola 
Il 06/06/2013 17:10, Giuseppe Vallarelli ha scritto:
>
> - Original Message -
> | From: "Sandro Bonazzola" 
> | To: "Alon Bar-Lev" 
> | Cc: vdsm-devel@lists.fedorahosted.org
> | Sent: Thursday, June 6, 2013 4:38:41 PM
> | Subject: Re: [vdsm] configNetwork issue
> | 
> | Il 06/06/2013 16:29, Alon Bar-Lev ha scritto:
> | > /usr/share/vdsm/addNetwork', 'ovirtmgmt', '', '', u'eth0',
> | > 'BOOTPROTO=dhcp', 'ONBOOT=yes', 'blockingdhcp=true']
> | 
> | # /usr/share/vdsm/addNetwork ovirtmgmt '' '' em1 BOOTPROTO=dhcp
> | ONBOOT=yes blockingdhcp=true
> | WARNING:root:options BOOTPROTO is deprecated. Use bootproto instead
> | WARNING:root:options ONBOOT is deprecated. Use onboot instead
> | INFO:root:Adding network ovirtmgmt with vlan=, bonding=, nics=['em1'],
> | bondingOptions=None, mtu=None, bridged=True, options={'bootproto':
> | 'dhcp', 'blockingdhcp': 'true', 'onboot': 'yes'}
> | Traceback (most recent call last):
> |   File "/usr/lib64/python2.7/runpy.py", line 162, in _run_module_as_main
> | "__main__", fname, loader, pkg_name)
> |   File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code
> | exec code in run_globals
> |   File "/usr/share/vdsm/configNetwork.py", line 664, in 
> | main()
> |   File "/usr/share/vdsm/configNetwork.py", line 633, in main
> | addNetwork(bridge, **kwargs)
> |   File "/usr/share/vdsm/configNetwork.py", line 198, in addNetwork
> | configurator, **options)
> | TypeError: objectivizeNetwork() got multiple values for keyword argument
> | 'bootproto'
>
> Hi Sandro, a very dirty and quick fix may be the following one.
>
> Inside configNetwork reach the addNetwork function and
> before the call to objectivizeNetwork:
>
> bootproto =options.get('bootproto')   
>   
>   
> if bootproto:
> del options['bootproto']
> netEnt = objectivizeNetwork(network if bridged else None, vlan, bonding,
>  bondingOptions, nics, mtu, ipaddr, netmask,
>  gateway, bootproto, _netinfo,
>  configurator, **options)
>
> Giuseppe
>

Thanks Giuseppe, that seems to fix the issue also if I don't really like
using del as solution.
This however is a workaround that fix it for me.



> | 
> | --
> | Sandro Bonazzola
> | Better technology. Faster innovation. Powered by community collaboration.
> | See how it works at redhat.com
> | 
> | ___
> | vdsm-devel mailing list
> | vdsm-devel@lists.fedorahosted.org
> | https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
> | 


-- 
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com

___
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Re: [vdsm] configNetwork issue

2013-06-06 Thread Giuseppe Vallarelli 


- Original Message -
| From: "Sandro Bonazzola" 
| To: "Alon Bar-Lev" 
| Cc: vdsm-devel@lists.fedorahosted.org
| Sent: Thursday, June 6, 2013 4:38:41 PM
| Subject: Re: [vdsm] configNetwork issue
| 
| Il 06/06/2013 16:29, Alon Bar-Lev ha scritto:
| > /usr/share/vdsm/addNetwork', 'ovirtmgmt', '', '', u'eth0',
| > 'BOOTPROTO=dhcp', 'ONBOOT=yes', 'blockingdhcp=true']
| 
| # /usr/share/vdsm/addNetwork ovirtmgmt '' '' em1 BOOTPROTO=dhcp
| ONBOOT=yes blockingdhcp=true
| WARNING:root:options BOOTPROTO is deprecated. Use bootproto instead
| WARNING:root:options ONBOOT is deprecated. Use onboot instead
| INFO:root:Adding network ovirtmgmt with vlan=, bonding=, nics=['em1'],
| bondingOptions=None, mtu=None, bridged=True, options={'bootproto':
| 'dhcp', 'blockingdhcp': 'true', 'onboot': 'yes'}
| Traceback (most recent call last):
|   File "/usr/lib64/python2.7/runpy.py", line 162, in _run_module_as_main
| "__main__", fname, loader, pkg_name)
|   File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code
| exec code in run_globals
|   File "/usr/share/vdsm/configNetwork.py", line 664, in 
| main()
|   File "/usr/share/vdsm/configNetwork.py", line 633, in main
| addNetwork(bridge, **kwargs)
|   File "/usr/share/vdsm/configNetwork.py", line 198, in addNetwork
| configurator, **options)
| TypeError: objectivizeNetwork() got multiple values for keyword argument
| 'bootproto'

Hi Sandro, a very dirty and quick fix may be the following one.

Inside configNetwork reach the addNetwork function and
before the call to objectivizeNetwork:

bootproto = options.get('bootproto')

   
if bootproto:
del options['bootproto']
netEnt = objectivizeNetwork(network if bridged else None, vlan, bonding,
 bondingOptions, nics, mtu, ipaddr, netmask,
 gateway, bootproto, _netinfo,
 configurator, **options)

Giuseppe


| 
| --
| Sandro Bonazzola
| Better technology. Faster innovation. Powered by community collaboration.
| See how it works at redhat.com
| 
| ___
| vdsm-devel mailing list
| vdsm-devel@lists.fedorahosted.org
| https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
| 
___
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Re: [vdsm] configNetwork issue

2013-06-06 Thread Sandro Bonazzola 
Il 06/06/2013 16:29, Alon Bar-Lev ha scritto:
> /usr/share/vdsm/addNetwork', 'ovirtmgmt', '', '', u'eth0', 'BOOTPROTO=dhcp', 
> 'ONBOOT=yes', 'blockingdhcp=true']

# /usr/share/vdsm/addNetwork ovirtmgmt '' '' em1 BOOTPROTO=dhcp
ONBOOT=yes blockingdhcp=true
WARNING:root:options BOOTPROTO is deprecated. Use bootproto instead
WARNING:root:options ONBOOT is deprecated. Use onboot instead
INFO:root:Adding network ovirtmgmt with vlan=, bonding=, nics=['em1'],
bondingOptions=None, mtu=None, bridged=True, options={'bootproto':
'dhcp', 'blockingdhcp': 'true', 'onboot': 'yes'}
Traceback (most recent call last):
  File "/usr/lib64/python2.7/runpy.py", line 162, in _run_module_as_main
"__main__", fname, loader, pkg_name)
  File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
  File "/usr/share/vdsm/configNetwork.py", line 664, in 
main()
  File "/usr/share/vdsm/configNetwork.py", line 633, in main
addNetwork(bridge, **kwargs)
  File "/usr/share/vdsm/configNetwork.py", line 198, in addNetwork
configurator, **options)
TypeError: objectivizeNetwork() got multiple values for keyword argument
'bootproto'

-- 
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com

___
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Re: [vdsm] configNetwork issue

2013-06-06 Thread Alon Bar-Lev 


- Original Message -
> From: "Sandro Bonazzola" 
> To: vdsm-devel@lists.fedorahosted.org
> Sent: Thursday, June 6, 2013 5:26:14 PM
> Subject: [vdsm] configNetwork issue
> 
> Hi,
> I'm trying to c reate a bridge adding as config bootproto ='dhcp'
> but I've the following error:
> 
> >>> import sys
> >>> sys.path.append('/usr/share/vdsm/')
> >>> import configNetwork
> >>> configNetwork.addNetwork(network='engine', nics=['em1'],
> >>> bootproto='dhcp')
> 
> TypeError: objectivizeNetwork() got multiple values for keyword argument
> 'bootproto'
> 
> seems like objectivizeNetwork already take boo tproto from **opts and fails.

Hi!

I think you should follow the ovirt-host-deploy bridge.py, and use addNetwork 
utility:

['/usr/share/vdsm/addNetwork', 'ovirtmgmt', '', '', u'eth0', 'BOOTPROTO=dhcp', 
'ONBOOT=yes', 'blockingdhcp=true']

___
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel