sp-cinder: Enable run_validation in Tempest
Runs have started failing with [1].
A recently-accepted change in upstream [2] is creating an empty dict
that gets passed to a function that does not accept an empty dict in
this context. [3]
This seems to me like a bug, but as a workaround in our CI, we can just
enable `run_validation` in Tempest [4], [5], so the dict gets populated.
[1]:
Captured traceback:
~~~~~~~~~~~~~~~~~~~
Traceback (most recent call last):
File "/opt/stack/tempest/tempest/common/utils/__init__.py", line 70, in wrapper
return f(*func_args, **func_kwargs)
File "/opt/stack/tempest/tempest/api/compute/servers/test_server_actions.py", line 868, in test_rebuild_volume_backed_server
self.get_server_ip(server, validation_resources),
File "/opt/stack/tempest/tempest/api/compute/base.py", line 519, in get_server_ip
return compute.get_server_ip(
File "/opt/stack/tempest/tempest/common/compute.py", line 76, in get_server_ip
raise lib_exc.InvalidParam(invalid_param=msg)
tempest.lib.exceptions.InvalidParam: Invalid Parameter passed: When validation.connect_method equals floating, validation_resources cannot be None
[2]: https://review.opendev.org/c/openstack/tempest/+/831018
[3]: https://review.opendev.org/c/openstack/tempest/+/831018/25..26/tempest/api/compute/servers/test_server_actions.py
[4]: https://docs.openstack.org/tempest/latest/configuration.html#ssh-connection-configuration
[5]: https://docs.openstack.org/tempest/latest/sampleconf.html
Change-Id: I3b8df5ca54f4de1270432f23c1751e92fd1b47a7
diff --git a/zuul.d/sp-cinder.yaml b/zuul.d/sp-cinder.yaml
index d4efe3d..78c08ee 100644
--- a/zuul.d/sp-cinder.yaml
+++ b/zuul.d/sp-cinder.yaml
@@ -32,6 +32,9 @@
TEMPEST_STORAGE_PROTOCOL: storpool
ENABLE_VOLUME_MULTIATTACH: true
devstack_local_conf:
+ test-config:
+ validation:
+ run_validation: true
post-config:
$CINDER_CONF:
storpool: