From: Ondrej Lichtner <olichtne(a)redhat.com>
getattr and setattr are commonly used so we should be consistent with
these names.
Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com>
---
lnst/Devices/RemoteDevice.py | 4 ++--
lnst/Slave/NetTestSlave.py | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/lnst/Devices/RemoteDevice.py b/lnst/Devices/RemoteDevice.py
index 8a3f0cf..00d937b 100644
--- a/lnst/Devices/RemoteDevice.py
+++ b/lnst/Devices/RemoteDevice.py
@@ -119,7 +119,7 @@ class RemoteDevice(object):
if self._cached:
return self._cache[name]
- return self._machine.rpc_call("dev_attr", self.ifindex, name,
+ return self._machine.rpc_call("dev_getattr", self.ifindex, name,
netns=self.netns)
def __setattr__(self, name, value):
@@ -132,7 +132,7 @@ class RemoteDevice(object):
if self._cached:
raise DeviceReadOnly("Can't set attributes when in ReadOnly
cache mode.")
- return self._machine.rpc_call("dev_set_attr", self.ifindex, name,
value,
+ return self._machine.rpc_call("dev_setattr", self.ifindex, name,
value,
netns=self.netns)
except AttributeError:
return super(RemoteDevice, self).__setattr__(name, value)
diff --git a/lnst/Slave/NetTestSlave.py b/lnst/Slave/NetTestSlave.py
index 795ec1d..1bb8f23 100644
--- a/lnst/Slave/NetTestSlave.py
+++ b/lnst/Slave/NetTestSlave.py
@@ -174,11 +174,11 @@ class SlaveMethods:
return method(*args, **kwargs)
- def dev_attr(self, ifindex, name):
+ def dev_getattr(self, ifindex, name):
dev = self._if_manager.get_device(ifindex)
return getattr(dev, name)
- def dev_set_attr(self, ifindex, name, value):
+ def dev_setattr(self, ifindex, name, value):
dev = self._if_manager.get_device(ifindex)
return setattr(dev, name, value)
--
2.19.0