Tempest installation fails to upgrade to new directory structure

Bug #1437813 reported by Yair Fried
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Medium
Yair Fried

Bug Description

This is due to relative paths being used in move() call

(rally)[yfried ~] # rally-manage -vd tempest install --source /home/yfried/workspace/git/tempest
2015-03-29 14:23:26.252 7717 RALLYDEBUG rally.cmd.cliutils [-] INFO logs from urllib3 and requests module are hide.
2015-03-29 14:23:26.286 7717 RALLYDEBUG rally.verification.tempest.tempest [-] Upgrading Tempest directory tree: Moving Tempest base dir /home/yfried/.rally/tempest/base into subdirectory tempest_base-vjrckjretawpwcrv
2015-03-29 14:23:26.287 7717 ERROR rally.cmd.cliutils [-] [Errno 2] No such file or directory: 'run_tempest.sh'
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils Traceback (most recent call last):
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/cmd/cliutils.py", line 433, in run
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils ret = fn(*fn_args, **fn_kwargs)
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "<string>", line 2, in install
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/cmd/envutils.py", line 66, in default_from_global
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils return f(*args, **kwargs)
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/cmd/manage.py", line 51, in install
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils api.Verification.install_tempest(deployment, source)
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/api.py", line 261, in install_tempest
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils verifier.install()
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/verification/tempest/tempest.py", line 242, in install
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils if not os.path.exists(self.base_repo):
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/verification/tempest/tempest.py", line 141, in base_repo
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils directory)
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/opt/rally/lib/python2.7/site-packages/rally/verification/tempest/tempest.py", line 111, in _move_contents_to_subdir
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils shutil.move(filename, os.path.join(base, subdir, filename))
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/usr/lib64/python2.7/shutil.py", line 301, in move
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils copy2(src, real_dst)
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/usr/lib64/python2.7/shutil.py", line 130, in copy2
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils copyfile(src, dst)
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils File "/usr/lib64/python2.7/shutil.py", line 82, in copyfile
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils with open(src, 'rb') as fsrc:
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils IOError: [Errno 2] No such file or directory: 'run_tempest.sh'
2015-03-29 14:23:26.287 7717 TRACE rally.cmd.cliutils

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to rally (master)

Fix proposed to branch: master
Review: https://review.openstack.org/168690

Changed in rally:
status: New → In Progress
Changed in rally:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to rally (master)

Reviewed: https://review.openstack.org/168690
Committed: https://git.openstack.org/cgit/stackforge/rally/commit/?id=d780231da61b2aaba062dffd72947c667e1f5082
Submitter: Jenkins
Branch: master

commit d780231da61b2aaba062dffd72947c667e1f5082
Author: Yair Fried <email address hidden>
Date: Sun Mar 29 14:31:42 2015 +0300

    Fixes Tempest path update to use abspath

    Fixes unittests accordingly.
    Also adds debug to file transfer.

    Closes-bug: #1437813
    Change-Id: I37670127d530c7add4cddef4b60f305e54ed2ca3

Changed in rally:
status: In Progress → Fix Committed
Changed in rally:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.