commit 96699b71b5d45f87772134391dd2ded33ecb5789
author Ratan Gupta <ratagupt@in.ibm.com> Fri Sep 08 17:34:07 2017 +0530
committer Ratan Gupta <ratagupt@in.ibm.com> Wed Sep 13 17:29:03 2017 +0530
tree 52e4c71de434582bcff6830dd9b073ac2f84dd9f
parent a54d8f8023a42555c549b3e60c1d45e5e2414559

Don't create the IP objects explicitly

IP objects would be created automatically once network
manager detects the address change event.

Resolves openbmc/openbmc#2171
Resolves openbmc/openbmc#2258

Change-Id: I2ac26992ac992f25154f0e1bf34c1faf185c4377
Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>

ethernet_interface.cpp

diff --git a/ethernet_interface.cpp b/ethernet_interface.cpp
index 39c12cf..bcb8f6c 100644
--- a/ethernet_interface.cpp
+++ b/ethernet_interface.cpp
@@ -324,8 +324,6 @@
 
     EthernetInterfaceIntf::dHCPEnabled(value);
     writeConfigurationFile();
-    createIPAddressObjects();
-
     return value;
 }
 
@@ -373,15 +371,12 @@
 
     // write the device file for the vlan interface.
     vlanIntf->writeDeviceFile();
+    vlanIntf->writeConfigurationFile();
 
     this->vlanInterfaces.emplace(vlanInterfaceName,
                                  std::move(vlanIntf));
     // write the new vlan device entry to the configuration(network) file.
     writeConfigurationFile();
-
-    // Create the dbus object for the link local ipv6 address.
-    vlanInterfaces[vlanInterfaceName]->createIPAddressObjects();
-
 }
 
 // Need to merge the below function with the code which writes the