Ido Barkan has posted comments on this change.
Change subject: api: make _should_keep_bridge robust against missing running/persistent cfg ......................................................................
Patch Set 1: Code-Review-1
(2 comments)
https://gerrit.ovirt.org/#/c/38239/1/vdsm/network/api.py File vdsm/network/api.py:
Line 750: return dict( Line 751: (k, v) for k, v in conf.iteritems() Line 752: if k not in ('bonding', 'nic', 'mtu')) Line 753: Line 754: # isn't the parameter order wrong?! the order is indeed wrong Line 755: def _bridge_reconfigured(required_conf, current_conf): Line 756: # we know nothing about the running/persistent configuration because Line 757: # it has been removed e.g. by shell_helper's emergency_net_cleanup Line 758: if current_conf is None or required_conf is None:
Line 755: def _bridge_reconfigured(required_conf, current_conf): Line 756: # we know nothing about the running/persistent configuration because Line 757: # it has been removed e.g. by shell_helper's emergency_net_cleanup Line 758: if current_conf is None or required_conf is None: Line 759: return False but let's return True an throw away the bridge just in case, because we don't care what was there before anyway. Line 760: Line 761: return (_bridge_only_config(required_conf) != Line 762: _bridge_only_config(current_conf)) Line 763:
vdsm-patches@lists.fedorahosted.org