summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Smith <dansmith@redhat.com>2023-05-01 09:19:31 -0700
committerDan Smith <dansmith@redhat.com>2023-05-01 13:49:39 -0700
commited1797c125e4c02c8c5257259d9e63ebf96ac814 (patch)
tree135f99b1cdaaa010165466c097069a732dfeb12a
parentcb008e2167826ad0950b4fd694f344c639413c91 (diff)
downloadtempest-ed1797c125e4c02c8c5257259d9e63ebf96ac814.tar.gz
Actually make volume extend tests validate
This fixes the previous patch to *actually* wait for the server to be SSHABLE before continuing the test. Because the volumes tests don't have full networking and we didn't provide validation resources to create_server() we weren't *actually* doing the wait for ssh. Change-Id: I783bbed4bea6137de3afa9a7ecd1a9cd37bb17e8
-rw-r--r--tempest/api/volume/base.py2
-rw-r--r--tempest/api/volume/test_volumes_extend.py5
2 files changed, 6 insertions, 1 deletions
diff --git a/tempest/api/volume/base.py b/tempest/api/volume/base.py
index 9ba9949d7..ae1dc8ade 100644
--- a/tempest/api/volume/base.py
+++ b/tempest/api/volume/base.py
@@ -51,6 +51,8 @@ class BaseVolumeTest(api_version_utils.BaseMicroversionTest,
def setup_credentials(cls):
cls.set_network_resources(
network=cls.create_default_network,
+ router=cls.create_default_network,
+ dhcp=cls.create_default_network,
subnet=cls.create_default_network)
super(BaseVolumeTest, cls).setup_credentials()
diff --git a/tempest/api/volume/test_volumes_extend.py b/tempest/api/volume/test_volumes_extend.py
index 906697999..51405b881 100644
--- a/tempest/api/volume/test_volumes_extend.py
+++ b/tempest/api/volume/test_volumes_extend.py
@@ -114,7 +114,10 @@ class BaseVolumesExtendAttachedTest(base.BaseVolumeTest):
if the action on the server fails.
"""
# Create a test server. Will be automatically cleaned up on teardown.
- server = self.create_server(wait_until='SSHABLE')
+ validation_resources = self.get_test_validation_resources(
+ self.os_primary)
+ server = self.create_server(wait_until='SSHABLE', validatable=True,
+ validation_resources=validation_resources)
# Attach the volume to the server and wait for the volume status to be
# "in-use".
self.attach_volume(server['id'], volume['id'])