Enhance DHCP beyond just OFF and IPv4/IPv6 enabled.
DHCP is not a binary option. The network interface can have DHCP
disabled, IPv4 only, IPv6 only, and IPv4/IPv6.
Tested:
Using dbus-send or busctl:
Disabled DHCP, and confirmed only link local addresses were present.
Assigned only static addresses. Both with/and without the gateway set
to 0.0.0.0
Deleted static IPv4 addresses.
Reassigned static addresses.
Enabled DHCP for ipv4 only, and witnessed a DHCP server assign a valid
address.
Assigned static IPv4 address.
Assigned static IPv6 address.
Confirmed both IPv4 and IPv6 static addresses are active.
Enabled DHCP for ipv6 only, and confirmed the static v4 address
remains. The ipv6 address is removed, waiting for a DHCP6 server.
Enabled DHCP for both ipv4 and ipv6. IPv4 address was assigned. IPv6
address is assumed to succeed, as systemd config file enables IPv6
DHCP.
Change-Id: I2e0ff80ac3a5e88bcff28adac419bf21e37be162
Signed-off-by: Johnathan Mantey <johnathanx.mantey@intel.com>
diff --git a/vlan_interface.cpp b/vlan_interface.cpp
index 6fa3b84..7762adc 100644
--- a/vlan_interface.cpp
+++ b/vlan_interface.cpp
@@ -22,7 +22,7 @@
using namespace sdbusplus::xyz::openbmc_project::Common::Error;
VlanInterface::VlanInterface(sdbusplus::bus::bus& bus,
- const std::string& objPath, bool dhcpEnabled,
+ const std::string& objPath, DHCPConf dhcpEnabled,
bool nICEnabled, uint32_t vlanID,
EthernetInterface& intf, Manager& parent) :
VlanIface(bus, objPath.c_str()),
@@ -38,22 +38,6 @@
emit_object_added();
}
-VlanInterface::VlanInterface(sdbusplus::bus::bus& bus,
- const std::string& objPath, bool dhcpEnabled,
- uint32_t vlanID, EthernetInterface& intf,
- Manager& parent) :
- VlanIface(bus, objPath.c_str()),
- DeleteIface(bus, objPath.c_str()),
- EthernetInterface(bus, objPath, dhcpEnabled, parent, false),
- parentInterface(intf)
-{
- id(vlanID);
- VlanIface::interfaceName(EthernetInterface::interfaceName());
- MacAddressIntf::mACAddress(parentInterface.mACAddress());
-
- emit_object_added();
-}
-
std::string VlanInterface::mACAddress(std::string)
{
log<level::ERR>("Tried to set MAC address on VLAN");