A piece of old code in bdev.py uses a for loop over a single variable
because we can 'break' out of the loop or exit on the 'else' path. This
is not a nice usage of the for loop, it should be converted to a
standard if...elif...else structure.
In the meantime we silence a warning from pylint (it is actually
invalid, IMHO) and add a TODO.
---
lib/bdev.py | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/lib/bdev.py b/lib/bdev.py
index 5253ddb..24b242a 100644
--- a/lib/bdev.py
+++ b/lib/bdev.py
@@ -1573,6 +1573,8 @@ class DRBD8(BaseDRBD):
the attach if can return success.
"""
+ # TODO: Rewrite to not use a for loop just because there is 'break'
+ # pylint: disable-msg=W0631
net_data = (self._lhost, self._lport, self._rhost, self._rport)
for minor in (self._aminor,):
info = self._GetDevInfo(self._GetShowData(minor))
--
1.6.5.7
