Re: [Rdo-list] Tempest IceHouse broken?

-----Original Message----- From: rdo-list-bounces(a)redhat.com [mailto:rdo-list-bounces@redhat.com] On Behalf Of Ihar Hrachyshka Sent: Thursday, October 02, 2014 3:38 PM To: rdo-list(a)redhat.com Subject: Re: [Rdo-list] Tempest IceHouse broken? -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 It's not about 'which'... The problem is that subprocess.check_output was introduced in Python 2.7 only, and not available in Python 2.6. We should not use the method for the code that is expected to be portable between those python versions.

On 02/10/14 14:29, David Kranz wrote: > I believe some recent improvements introduced an external dependency > on 'which' so that needs to be installed. This is taken care of if you > install via the tempest rpm. Tempest has never had clear documentation > about what external (non-python) packages are needed. In the upstream > gate jobs this is handled by devstack.

> > -David > > On 10/02/2014 08:01 AM, Kaul, Yaniv wrote: >> >> On CentOS 6.5 (Python 2.6!) >> >> >> >> git init /var/jenkins/workspace/Tempest # timeout=10 >> >> Fetching upstream changes from >> https://github.com/redhat-openstack/tempest.git >> >>> git --version # timeout=10 >> >>> git fetch --tags --progress >> https://github.com/redhat-openstack/tempest.git >> +refs/heads/*:refs/remotes/origin/* >> >>> git config remote.origin.url >> https://github.com/redhat-openstack/tempest.git # timeout=10 >> >>> git config remote.origin.fetch >>> +refs/heads/*:refs/remotes/origin/* # >> timeout=10 >> >>> git config remote.origin.url >> https://github.com/redhat-openstack/tempest.git # timeout=10 >> >> Fetching upstream changes from >> https://github.com/redhat-openstack/tempest.git >> >>> git fetch --tags --progress >> https://github.com/redhat-openstack/tempest.git >> +refs/heads/*:refs/remotes/origin/* >> >>> git rev-parse refs/remotes/origin/icehouse^{commit} # >>> timeout=10 >> >>> git rev-parse refs/remotes/origin/origin/icehouse^{commit} # >>> timeout=10 >> >> Checking out Revision fe12b3d0e2fb713efe923a44393880b0d4d9b0ba >> (refs/remotes/origin/icehouse) >> >> >> >> ? >> >> >> >> TEMPEST_CONF=/var/jenkins/workspace/Tempest/etc/tempest.conf >> >> + tools/config_tempest.py --create identity.uri >> http://10.103.234.141:5000/v2.0/ identity.admin_username admin >> identity.admin_password secret identity.admin_tenant_name admin >> >> /usr/lib64/python2.6/site-packages/Crypto/Util/number.py:57: >> PowmInsecureWarning: Not using mpz_powm_sec. You should rebuild >> using libgmp >= 5 to avoid timing attack vulnerability. >> >> _warn("Not using mpz_powm_sec. You should rebuild using libgmp >> >= 5 to avoid timing attack vulnerability.", >> PowmInsecureWarning) >> >> Traceback (most recent call last): >> >> File "tools/config_tempest.py", line 654, in <module> >> >> main() >> >> File "tools/config_tempest.py", line 141, in main >> >> configure_cli(conf) >> >> File "tools/config_tempest.py", line 528, in configure_cli >> >> cli_dir = get_program_dir("nova") >> >> File "tools/config_tempest.py", line 595, in get_program_dir >> >> path = subprocess.check_output(["which", program], >> stderr=devnull) >> >> AttributeError: 'module' object has no attribute 'check_output' >> >> >> >> >> >> >> >> Looks similar to https://bugs.launchpad.net/nova/+bug/1029014 ? >> >> Seems to be added in: >> >> commit 76ca1350749ea5e3c877813c6e7268122b5214ce >> >> Author: Martina Kollarova <mkollaro(a)redhat.com&gt; >> >> Date: Wed Sep 3 18:53:05 2014 +0200 >> >> >> >> Refactored cli_dir setting >> >> >> >> >> >> And possibly because of: >> >> +def get_program_dir(program): >> >> + """Get directory path of the external program. >> >> + >> >> + :param program: name of program, e.g. 'ls' or 'cat' >> >> + :returns: None if it wasn't found, '/path/to/it/' if found >> >> + """ >> >> + devnull = open(os.devnull, 'w') >> >> + try: >> >> + path = subprocess.check_output(["which", program], >> stderr=devnull) >> >> + return os.path.dirname(path.strip()) >> >> + except subprocess.CalledProcessError: >> >> + return None >> >> + >> >> >> >> >> >> Y. >> >> >> >> _______________________________________________ Rdo-list mailing list >> Rdo-list(a)redhat.com https://www.redhat.com/mailman/listinfo/rdo-list > > > > _______________________________________________ Rdo-list mailing > list Rdo-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/rdo-list > _______________________________________________ Rdo-list mailing list Rdo-list(a)redhat.com https://www.redhat.com/mailman/listinfo/rdo-list