From f64e1a09ae5059ebcd916e09053fdf286216ffd4 Mon Sep 17 00:00:00 2001 From: Konrad Gube Date: Thu, 21 Sep 2023 11:38:47 +0200 Subject: [PATCH] Re-propose extend volume completion action Previously-approved: 2023.1, 2023.2 For previous discussion see the change at: https://review.opendev.org/c/openstack/cinder-specs/+/866718 Blueprint: extend-volume-completion-action Change-Id: Iacb0781deac38ff3a280588c40373139323c0853 --- .../extend-volume-completion-action.rst | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) rename specs/{2023.2 => 2024.1}/extend-volume-completion-action.rst (96%) diff --git a/specs/2023.2/extend-volume-completion-action.rst b/specs/2024.1/extend-volume-completion-action.rst similarity index 96% rename from specs/2023.2/extend-volume-completion-action.rst rename to specs/2024.1/extend-volume-completion-action.rst index 2b1091b4..0be51033 100644 --- a/specs/2023.2/extend-volume-completion-action.rst +++ b/specs/2024.1/extend-volume-completion-action.rst @@ -421,7 +421,8 @@ Work Items * Add OpenStack SDK support. * Add Nova support. * Update drivers to use the feature. -* Add integration tests. +* Adapt the ``devstack-plugin-nfs-tempest`` CI-jobs to also test online volume + extend. Dependencies ============ @@ -435,9 +436,12 @@ Testing responses. * Unit tests for ``VolumeManager.extend_volume`` will test all the code paths described in `VolumeManager.extend_volume`_. -* Integration tests will test the new behavior of the ``os-extend`` and - ``os-extend_volume_completion`` volume actions, as well as the interaction - between Cinder and Nova. +* The new volume action cannot be independently tested by Tempest, because it + requires the volume to be in a state that cannot be reproduced externally. + It is, however, covered by the existing tests for online volume extend when + they are run with one of the volume drivers that use this feature. + The ``devstack-plugin-nfs-tempest`` jobs that run as part of the Cinder and + Nova CI gates will be configured to enable online volume extend tests. Documentation Impact ==================== @@ -454,4 +458,4 @@ References .. [1] https://review.opendev.org/c/openstack/cinder/+/739079 .. [2] https://review.opendev.org/c/openstack/nova-specs/+/855490/6 .. [3] https://review.opendev.org/c/openstack/cinder-specs/+/864020 -.. [4] https://review.opendev.org/c/openstack/nova-specs/+/877233 +.. [4] https://review.opendev.org/c/openstack/nova-specs/+/895648