Re: [PATCH v6 08/11] iotests: add testenv.py

[Kevin Wolf]

Am 16.01.2021 um 12:03 hat Vladimir Sementsov-Ogievskiy geschrieben:
> 15.01.2021 16:30, Vladimir Sementsov-Ogievskiy wrote:
> > 15.01.2021 16:20, Kevin Wolf wrote:
> > > Am 15.01.2021 um 14:10 hat Vladimir Sementsov-Ogievskiy geschrieben:
> > > > 15.01.2021 15:45, Kevin Wolf wrote:
> > > > > Am 15.01.2021 um 13:19 hat Vladimir Sementsov-Ogievskiy geschrieben:
> > > > > > 15.01.2021 14:18, Kevin Wolf wrote:
> > > > > > > Am 09.01.2021 um 13:26 hat Vladimir Sementsov-Ogievskiy 
> > > > > > > geschrieben:
> > > > > > > > Add TestEnv class, which will handle test environment in a new 
> > > > > > > > python
> > > > > > > > iotests running framework.
> > > > > > > > 
> > > > > > > > Difference with current ./check interface:
> > > > > > > > - -v (verbose) option dropped, as it is unused
> > > > > > > > 
> > > > > > > > - -xdiff option is dropped, until somebody complains that it is 
> > > > > > > > needed
> > > > > > > > - same for -n option
> > > > > > > > 
> > > > > > > > Signed-off-by: Vladimir Sementsov-Ogievskiy 
> > > > > > > > <vsementsov@virtuozzo.com>
> > > > > > > > ---
> > > > > > > > Â Â Â  tests/qemu-iotests/testenv.py | 328 
> > > > > > > > ++++++++++++++++++++++++++++++++++
> > > > > > > > Â Â Â  1 file changed, 328 insertions(+)
> > > > > > > > Â Â Â  create mode 100755 tests/qemu-iotests/testenv.py
> > > > > > > > 
> > > > 
> > > > [..]
> > > > 
> > > > > > > > +Â Â Â  def init_binaries(self):
> > > > > > > > +Â Â Â Â Â Â Â  """Init binary path variables:
> > > > > > > > +Â Â Â Â Â Â Â Â Â Â Â Â  PYTHON (for bash tests)
> > > > > > > > +Â Â Â Â Â Â Â Â Â Â Â Â  QEMU_PROG, QEMU_IMG_PROG, 
> > > > > > > > QEMU_IO_PROG, QEMU_NBD_PROG, QSD_PROG
> > > > > > > > +Â Â Â Â Â Â Â Â Â Â Â Â  SOCKET_SCM_HELPER
> > > > > > > > +Â Â Â Â Â Â Â  """
> > > > > > > > +Â Â Â Â Â Â Â  self.python = '/usr/bin/python3 -B'
> > > > > > > 
> > > > > > > This doesn't look right, we need to respect the Python binary set 
> > > > > > > in
> > > > > > > configure (which I think we get from common.env)
> > > > > > 
> > > > > > Oh, I missed the change. Then I should just drop this self.python.
> > > > > 
> > > > > Do we still get the value from elsewhere or do we need to manually 
> > > > > parse
> > > > > common.env?
> > > > 
> > > > Hmm.. Good question. We have either parse common.env, and still create 
> > > > self.python variable.
> > > > 
> > > > Or drop it, and include common.env directly to bash tests. For this 
> > > > we'll need to export
> > > > 
> > > > BUILD_IOTESTS, and do
> > > > Â  . $BUILD_IOTESTS/common.env
> > > > 
> > > > in common.rc..
> > > 
> > > check uses it, too, for running Python test cases.
> > > 
> > 
> > But new check (written in python) doesn't.. Should I keep bash check, which 
> > will have only one line to call check.py with help of PYTHON?
> > 
> > 
> 
> Or finally, may be just drop it? Can we just use system python for
> tests, now when we are already in a python3 world?

You can only assume the Python 3 (or more specifically 3.6+) world if
you are using the Python interpreter that was passed to configure. This
will commonly be the same thing as the first python3 in PATH, but it
doesn't have to be.

So your solution of just using the same interpreter as for check might
be okay as long as we make sure that 'make check' calls check with the
configured Python interpreter instead of relying on the shebang line.

Kevin