This adds a failing test, which we fix in change I76448196.
An earlier version of this change was previously merged as
change I5619728d. This was later reverted, as it was failing in the
gate. However, on inspection these failures seem to have been simply
timeouts due to load.
Changes from previous version:
- Increase the timeouts which were previously triggering, and
serialise server creation to reduce the chance of this recurring.
- Add an LVM test, which highlights the requirement to flag the
creation of ephemeral and swap disks.
- Add an Qcow2 test, essentially the same as the Flat test but ensures
coverage of the most common backends.
- Each test now uses a separate instances_path allowing for cleanup
without racing against other active tests.
- Some nits addressed.
For the time being this test does not make use of the recently improved
nova.tests.functional.libvirt.base.ServersTestBase class to ease
backports. Future changes should be made to use this class removing some
of the common setUp logic from _LibvirtEvacuateTest.
NOTE(lyarwood): Calls to _wait_for_state_change and
_wait_for_server_parameter now provide self.api as required prior to
I8c96b337f32148f8f5899c9b87af331b1fa41424 landing in Ussuri.
Co-Authored-By: Lee Yarwood <email address hidden>
Related-Bug: #1550919
Change-Id: I1062b3e74382734edbb2142a09ff0073c66af8db
(cherry picked from commit 90e0e874bde38937380d09ab27a7defbb5475cc2)
(cherry picked from commit 6ccd13f8aeeb97c2139c1abc93cb976fd57d57dd)
Reviewed: https:/ /review. opendev. org/731272 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=172eb21dee1 d93b140c2b691cb 8dfbc68b721bfe
Committed: https:/
Submitter: Zuul
Branch: stable/train
commit 172eb21dee1d93b 140c2b691cb8dfb c68b721bfe
Author: Matthew Booth <email address hidden>
Date: Tue Aug 14 16:05:11 2018 +0100
Add functional test for bug 1550919
This adds a failing test, which we fix in change I76448196.
An earlier version of this change was previously merged as
change I5619728d. This was later reverted, as it was failing in the
gate. However, on inspection these failures seem to have been simply
timeouts due to load.
Changes from previous version:
- Increase the timeouts which were previously triggering, and
serialise server creation to reduce the chance of this recurring.
- Add an LVM test, which highlights the requirement to flag the
creation of ephemeral and swap disks.
- Add an Qcow2 test, essentially the same as the Flat test but ensures
coverage of the most common backends.
- Each test now uses a separate instances_path allowing for cleanup
without racing against other active tests.
- Some nits addressed.
For the time being this test does not make use of the recently improved tests.functiona l.libvirt. base.ServersTes tBase class to ease eTest.
nova.
backports. Future changes should be made to use this class removing some
of the common setUp logic from _LibvirtEvacuat
NOTE(lyarwood): Calls to _wait_for_ state_change and for_server_ parameter now provide self.api as required prior to 2148f8f5899c9b8 7af331b1fa41424 landing in Ussuri.
_wait_
I8c96b337f3
Co-Authored-By: Lee Yarwood <email address hidden> 4edbb2142a09ff0 073c66af8db 7380d09ab27a7de fbb5475cc2) 2139c1abc93cb97 6fd57d57dd)
Related-Bug: #1550919
Change-Id: I1062b3e7438273
(cherry picked from commit 90e0e874bde3893
(cherry picked from commit 6ccd13f8aeeb97c