VDSM unit test 'test_getDevicePartedInfo' fails with Python 2.7.6 with the following traceback:

  File "/home/Data/src/vdsm/src/vdsm/tests/testValidation.py", line 99, in wrapper
    return f(*args, **kwargs)
  File "/home/Data/src/vdsm/src/vdsm/tests/parted_utils_tests.py", line 57, in setUp
    rc, out, err = utils.execCmd(cmd)
  File "/home/Data/src/vdsm/src/vdsm/lib/vdsm/utils.py", line 537, in execCmd
    deathSignal=deathSignal)
  File "/usr/lib/python2.7/site-packages/cpopen/__init__.py", line 48, in __init__
    stderr=PIPE)
  File "/usr/lib/python2.7/subprocess.py", line 732, in __init__
    errread, errwrite)
TypeError: _execute_child() takes exactly 17 arguments (18 given)

This is due to python-cpopen replacing subprocess.Popen's internal method _execute_child which gained a new parameter 'to_close' in Python 2.7.6 (released on November 10, 2013), see http://hg.python.org/cpython/rev/43749cb6bdbd

Adding the 'to_close' parameter after 'shell' in cpopen's __init__.py fixes the test with Python 2.7.6. Hopefully cpopen's implementation doesn't suffer from the bug that inspired adding the parameter.

I guess that with cpopen not being updated often it is sufficient to release a new version sometime, adding the new parameter and requiring Python 2.7.6.

Should I file a bug at https://github.com/ficoos/cpopen too?