[PATCH stable-2.15 17/17] Disable pylint too-many-nested-blocks in _RunCmdPipe

230 views
Skip to first unread message

Brian Foley

unread,
Dec 6, 2016, 12:30:28 PM12/6/16
to ganeti...@googlegroups.com
There doesn't appear to be any easy way of reducing the complexity
of this function without moving it elsewhere or completely reorganising
the function, so disable this warning for the time being.

Signed-off-by: Brian Foley <bpf...@google.com>
---
lib/utils/process.py | 1 +
1 file changed, 1 insertion(+)

diff --git a/lib/utils/process.py b/lib/utils/process.py
index bb5aed2..05553fc 100644
--- a/lib/utils/process.py
+++ b/lib/utils/process.py
@@ -530,6 +530,7 @@ def _RunCmdPipe(cmd, env, via_shell, cwd, interactive, timeout, noclose_fds,
@return: (out, err, status)

"""
+ # pylint: disable=R0101
poller = select.poll()

if interactive:
--
2.8.0.rc3.226.g39d4020

Iustin Pop

unread,
Dec 6, 2016, 4:37:00 PM12/6/16
to Brian Foley, ganeti...@googlegroups.com
On 2016-12-06 17:30:26, Ganeti Development List wrote:
> There doesn't appear to be any easy way of reducing the complexity
> of this function without moving it elsewhere or completely reorganising
> the function, so disable this warning for the time being.

LGTM.
Reply all
Reply to author
Forward
0 new messages