yahoo-eng-team team mailing list archive

[Matt Riedemann <mriedem@xxxxxxxxxx>]

Public bug reported:

I noticed this while running unit tests on stable/mitaka, but I think it
applies to other branches including master.

When you run tox -r -e py27 you'll have a nova/instances directory
created, I think from the state_path variable used in the libvirt
driver. nova/keys is also created and left around, but wasn't an issue
for me.

On stable/mitaka, because of the files existing on disk in
nova/instances, the test_rescue() test in the libvirt driver would find
a file that exists (which normally wouldn't) and tries to chown it,
which hangs the test process on sudo prompting for a password.

Ideally the tests would cleanup any resources they create, but as a
quick fix we could simply just rm -rf nova/instances from tox.ini when
we run tests, just like we do with removing *.pyc files.

** Affects: nova
     Importance: Low
         Status: Confirmed


** Tags: low-hanging-fruit testing

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1664669

Title:
  Leftover nova/instances from test runs can make tests fail locally

Status in OpenStack Compute (nova):
  Confirmed

Bug description:
  I noticed this while running unit tests on stable/mitaka, but I think
  it applies to other branches including master.

  When you run tox -r -e py27 you'll have a nova/instances directory
  created, I think from the state_path variable used in the libvirt
  driver. nova/keys is also created and left around, but wasn't an issue
  for me.

  On stable/mitaka, because of the files existing on disk in
  nova/instances, the test_rescue() test in the libvirt driver would
  find a file that exists (which normally wouldn't) and tries to chown
  it, which hangs the test process on sudo prompting for a password.

  Ideally the tests would cleanup any resources they create, but as a
  quick fix we could simply just rm -rf nova/instances from tox.ini when
  we run tests, just like we do with removing *.pyc files.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1664669/+subscriptions