Change in vdsm[master]: net: api: remove mtu=None from Bond.objectivize calls

phoracek at redhat.com phoracek at redhat.com
Wed Feb 17 12:15:20 UTC 2016


Petr Horáček has uploaded a new change for review.

Change subject: net: api: remove mtu=None from Bond.objectivize calls
......................................................................

net: api: remove mtu=None from Bond.objectivize calls

Bond:objectivize()'s kwarg mtu has default value of None. We can
drop its explicit setting from net api.

Change-Id: I627ef356162394dce5fa4d3d23dd6f71f6f6750d
Signed-off-by: Petr Horáček <phoracek at redhat.com>
---
M lib/vdsm/network/api.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/48/53648/1

diff --git a/lib/vdsm/network/api.py b/lib/vdsm/network/api.py
index 4c48cfe..b71e80f 100644
--- a/lib/vdsm/network/api.py
+++ b/lib/vdsm/network/api.py
@@ -577,7 +577,7 @@
                     (name, bond_users))
             bond = Bond.objectivize(name, configurator, _netinfo,
                                     options=attrs.get('options'),
-                                    nics=attrs.get('nics'), mtu=None,
+                                    nics=attrs.get('nics'),
                                     destroyOnMasterRemoval='remove' in attrs)
             bond.remove()
             _netinfo.del_bonding(name)
@@ -594,7 +594,7 @@
     for name, attrs in edition:
         bond = Bond.objectivize(name, configurator, _netinfo,
                                 options=attrs.get('options'),
-                                nics=attrs.get('nics'), mtu=None,
+                                nics=attrs.get('nics'),
                                 destroyOnMasterRemoval='remove' in attrs)
         if len(bond.slaves) == 0:
             raise ConfigNetworkError(ne.ERR_BAD_PARAMS, 'Missing required nics'
@@ -604,7 +604,7 @@
     for name, attrs in addition:
         bond = Bond.objectivize(name, configurator, _netinfo,
                                 options=attrs.get('options'),
-                                nics=attrs.get('nics'), mtu=None,
+                                nics=attrs.get('nics'),
                                 destroyOnMasterRemoval='remove' in attrs)
         if len(bond.slaves) == 0:
             raise ConfigNetworkError(ne.ERR_BAD_PARAMS, 'Missing required nics'


-- 
To view, visit https://gerrit.ovirt.org/53648
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I627ef356162394dce5fa4d3d23dd6f71f6f6750d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>


More information about the vdsm-patches mailing list