Change in vdsm[master]: Adding python3 run for nosetests
nsoffer at redhat.com
nsoffer at redhat.com
Thu Nov 19 21:32:41 UTC 2015
Nir Soffer has posted comments on this change.
Change subject: Adding python3 run for nosetests
......................................................................
Patch Set 10:
(3 comments)
https://gerrit.ovirt.org/#/c/48051/10/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 630: """
Line 631:
Line 632: # Moving import here so that import utils over python3 will work. Once
Line 633: # python3-cpopen will be available this import should return to top
Line 634: from cpopen import CPopen
We don't need cpopen on python 3.
Why not use the compat trick to get Popen from either cpopen or subprocess?
Line 635:
Line 636: command = cmdutils.wrap_command(command, with_ioclass=ioclass,
Line 637: ioclassdata=ioclassdata, with_nice=nice,
Line 638: with_setsid=setsid, with_sudo=sudo,
https://gerrit.ovirt.org/#/c/48051/10/tests/Makefile.am
File tests/Makefile.am:
Line 29
Line 30
Line 31
Line 32
Line 33
And add here _py2?
I guess we are going to live with both py3 and py2 for long time, so lets make it more clear.
Line 25: devices \
Line 26: integration \
Line 27: $(NULL)
Line 28:
Line 29: test_modules_python3 = \
Maybe _py3?
Line 30: apiData.py \
Line 31: cmdutilsTests.py \
Line 32: concurrentTests.py \
Line 33: $(NULL)
--
To view, visit https://gerrit.ovirt.org/48051
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I83355cce2af9125e6f017017905056956cd17081
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list