From: Ondrej Lichtner olichtne@redhat.com
The if_data method now returns a complete dictionary describing the Device object.
This includes a nonbackwards compatible change of renaming "devname" to just "name".
Signed-off-by: Ondrej Lichtner olichtne@redhat.com --- lnst/Controller/Machine.py | 2 +- lnst/Slave/InterfaceManager.py | 10 +++++++++- 2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/lnst/Controller/Machine.py b/lnst/Controller/Machine.py index 5d8db7d..08df4d8 100644 --- a/lnst/Controller/Machine.py +++ b/lnst/Controller/Machine.py @@ -695,7 +695,7 @@ class Interface(object):
def update(self, if_data): self.set_hwaddr(if_data["hwaddr"]) - self.set_devname(if_data["devname"]) + self.set_devname(if_data["name"]) self._mtu = if_data["mtu"] self._driver = if_data["driver"]
diff --git a/lnst/Slave/InterfaceManager.py b/lnst/Slave/InterfaceManager.py index 30b9117..104fc62 100644 --- a/lnst/Slave/InterfaceManager.py +++ b/lnst/Slave/InterfaceManager.py @@ -604,8 +604,16 @@ class Device(object): return None
def get_if_data(self): - if_data = {"devname": self._name, + if_data = {"if_index": self._if_index, "hwaddr": self._hwaddr, + "name": self._name, + "ip_addrs": self._ip_addrs, + "ifi_type": self._ifi_type, + "state": self._state, + "master": self._master, + "slaves": self._slaves, + "netns": self._netns, + "peer": self._peer, "mtu": self._mtu, "driver": self._driver} return if_data