On Fri, Jun 29, 2018 at 4:05 PM, John Spray <jspray@xxxxxxxxxx> wrote: > On Fri, Jun 29, 2018 at 8:48 PM Rishabh Dave <ridave@xxxxxxxxxx> wrote: >> >> Hi, >> >> I've tested this PR[1] on pulpito but it seems to fail everytime[2]. I >> suspect that LD_LIBRARY_PATH and PYTHONPATH are not updated in the >> test environment to required values before the test begins to run. >> Usually, while testing locally I run the following commands beforehand >> - >> >> export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/user1/repos/ceph/build/lib/ >> export PYTHONPATH=$PYTHONPATH:/home/user1/repos/ceph/build/lib/cython_modules/lib.2/:/home/user1/repos/ceph/src/pybind/ >> >> The reason why I suspect this is that the error in the teuthology >> log[3] is very familiar, I've seen it when I used to forget running >> above commands. Also, the following line in the log must mean that the >> PYTHONPATH doesn't hold the path to ceph_volume_client.py - >> >> ModuleNotFoundError: No module named 'ceph_volume_client' >> >> Is there a way to check, and update if required, these variables in >> environment before the test runs? > > I don't think this is a test environment issue. You shouldn't need > any special environment variables -- unlike a vstart cluster, the > teuthology tests run against installed RPMs that make python modules > available system wide. This should be the same whether you're looking > at python 2 (where I see your test is passing) or python 3. > > My suspicion would be that something is going wrong with the > packaging/installation of ceph_volume_client.py when building with > python 3 -- it should be possible to debug that by installing the > package on some handy machine, where I expect you will also find that > "import ceph_volume_client" is failing, then you can checking where > it's installing the file (perhaps somewhere outside the python path) What John says makes sense to me. I'd suggest debugging via a docker container. That should be simply done with: sudo ceph/src/script/ceph-debug-docker.sh wip-rishabh-testing-volclient-py3compat ubuntu:bionic -- Patrick Donnelly -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
