Need chunk size configuration for volume creation

[Vladislav Belogrudov]

Some storage systems only allow to create volumes of sizes multiple of a chunk size. Such size can be bigger than 1G, e.g. for PowerFlex systems it's 8GB. When user asks for 5GB an 8GB volume is created and returned. To correctly test such situations we need to configure extra_size:
https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_clone.py#L47

Currently it is hardcoded to '1', but it would be much better to have storage specific CONF.volume.chunk_size or similar

[Martin Kopec (mkopec)]

Can't volume_size option be used for that?

https://opendev.org/openstack/tempest/src/commit/70ca1e9188d8e6875ff5086f8e16c08a0c359909/tempest/config.py#L1000

[Martin Kopec (mkopec)]

Moving to incomplete while the previous question is answered.

[Launchpad Janitor (janitor)]

[Expired for tempest because there has been no activity for 60 days.]

[Luigi Toscano (ltoscano)]

Looking at the bug: no, because volume_size set the initial size. The same create_volume also creates a second volume with 'size=src_size + 1', and then _verify_volume_clone is not going to work as expected.

This is connected to https://bugs.launchpad.net/cinder/+bug/1915015, which maybe should have been used to track both the cinder and the tempest issue.

[Luigi Toscano (ltoscano)]

Reopening and confirming

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tempest/+/797966

[Martin Kopec (mkopec)]

Thank Luigi for clarifying that ... I pushed a review, let me know what you think about it.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/tempest/+/797966
Committed: https://opendev.org/openstack/tempest/commit/37c4bd3023110f2a00e55439e2086d393b44a974
Submitter: "Zuul (22348)"
Branch: master

commit 37c4bd3023110f2a00e55439e2086d393b44a974
Author: Martin Kopec <email address hidden>
Date: Thu Jun 24 15:46:00 2021 +0000

    Add volume_size_extend opt for volume tests

    Volume tests had hardcoded value (1) in case they were creating
    a second volume with a different size than the first one
    (CONF.volume.volume_size). This is a problem for systems which
    have a chunk size other than 1. The patch is adding a new opt
    CONF.volume.volume_size_extend which allows customization of an
    extended volume size.

    Closes-Bug: 1917299
    Change-Id: Ic8ae486224cd2a470f4f9bbad62d4d6715cc63ac

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/tempest train-last release.