diff options
author | Dan Prince <dprince@redhat.com> | 2015-05-14 15:18:26 -0400 |
---|---|---|
committer | Dan Prince <dprince@redhat.com> | 2015-06-03 21:44:08 -0400 |
commit | ca30a67a6d56e8f72d23003ffde477e4546a209d (patch) | |
tree | ef131b73c29e1370a466d1ba259ebe966d5658f9 /os_net_config/impl_ifcfg.py | |
parent | 6f2f62b73b165f1fc431cff0656d70adfe97e4a0 (diff) |
Don't set OVSDHCPINTERFACES if bridge is static
Updates the ifcfg provider so that we don't set
the OVSDHCPINTERFACES config setting if
the interface is private. This doesn't seem to cause
any functional issues but it is not required
and can be confusing.
Change-Id: I9d51905df87969ae1b221680e8f0b5befc7b6bfe
Diffstat (limited to 'os_net_config/impl_ifcfg.py')
-rw-r--r-- | os_net_config/impl_ifcfg.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/os_net_config/impl_ifcfg.py b/os_net_config/impl_ifcfg.py index 097552a..2411951 100644 --- a/os_net_config/impl_ifcfg.py +++ b/os_net_config/impl_ifcfg.py @@ -95,7 +95,8 @@ class IfcfgNetConfig(os_net_config.NetConfig): if base_opt.members: members = [member.name for member in base_opt.members] self.member_names[base_opt.name] = members - data += ("OVSDHCPINTERFACES=\"%s\"\n" % " ".join(members)) + if base_opt.use_dhcp: + data += ("OVSDHCPINTERFACES=\"%s\"\n" % " ".join(members)) if base_opt.primary_interface_name: mac = utils.interface_mac(base_opt.primary_interface_name) ovs_extra.append("set bridge %s other-config:hwaddr=%s" % |