NetApp ONTAP cannot resize a volume LUN with sub-clone

Bug #1924643 reported by Felipe Rodrigues
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Felipe Rodrigues

Bug Description

Steps to reproduce:
1. Deploy a queens release
2. Configure ONTAP 9.3
3. Create a volume smaller than 64G (e.g. openstack volume create --size 4 test)
4. Try to extend the volume to larger than 64G (e.g. openstack volume set test --size 65)

Expected result:
Volume is extended to the specified size (e.g. volume 'test' would be 65G)

Actual result:
Volume is in error_extending state, and c-vol logs an error: http://paste.openstack.org/show/804536/

Notes:

1) The bug only is reached with releases rocky or older, given that the patch [1] solves the issue of resizing with ONTAP version lower than 9.5. This patch guarantees that the resize using LUN sub-clone is never reached. All volume LUNs are created to be possible to extend without sub-clone.

2) The bug comes from the fact that the sub-clone operation must not use the `reserve-space`. The API does not accept `reserve-space` together with `block-ranges`.

[1] https://review.opendev.org/c/openstack/cinder/+/729782

Changed in cinder:
assignee: nobody → Felipe Rodrigues (felipefutty)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
importance: Undecided → Low
tags: added: netapp ontap
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/798208
Committed: https://opendev.org/openstack/cinder/commit/dd0b1076df936b9cb1111567f6b4c3b747744b72
Submitter: "Zuul (22348)"
Branch: master

commit dd0b1076df936b9cb1111567f6b4c3b747744b72
Author: Felipe Rodrigues <email address hidden>
Date: Fri Apr 16 10:05:40 2021 -0300

    NetApp ONTAP: Fix sub-clone zapi call

    The ONTAP documentation states that the `clone-create` ZAPI call
    fails with `block-ranges` and `space-reserve` parameters sent
    together. The sub-clone uses the `block-ranges` and is failing
    because of that restriction.

    This patch fixes the `clone-create` operation by using exactly one
    of `block-ranges` or `space-reserve`.

    Change-Id: I05d83d73de69c57d885e0c417e8a376f7cfb1e4f
    Closes-Bug: #1924643

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/xena)

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/cinder/+/821890

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/wallaby)

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/cinder/+/821892

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/victoria)

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/cinder/+/821893

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/cinder/+/821894

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/c/openstack/cinder/+/821905

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/ussuri)

Change abandoned by "Fábio Oliveira <email address hidden>" on branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/cinder/+/821894
Reason: wrong cherry pick

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/wallaby)

Change abandoned by "Fábio Oliveira <email address hidden>" on branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/cinder/+/821892
Reason: wrong cherry pick

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/train)

Change abandoned by "Fábio Oliveira <email address hidden>" on branch: stable/train
Review: https://review.opendev.org/c/openstack/cinder/+/821905
Reason: wrong cherry pick

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/victoria)

Change abandoned by "Fábio Oliveira <email address hidden>" on branch: stable/victoria
Review: https://review.opendev.org/c/openstack/cinder/+/821893
Reason: wrong cherry pick

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/821890
Committed: https://opendev.org/openstack/cinder/commit/344f3e8b098ff52c137ece8a28b3b40a5893e30e
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 344f3e8b098ff52c137ece8a28b3b40a5893e30e
Author: Felipe Rodrigues <email address hidden>
Date: Fri Apr 16 10:05:40 2021 -0300

    NetApp ONTAP: Fix sub-clone zapi call

    The ONTAP documentation states that the `clone-create` ZAPI call
    fails with `block-ranges` and `space-reserve` parameters sent
    together. The sub-clone uses the `block-ranges` and is failing
    because of that restriction.

    This patch fixes the `clone-create` operation by using exactly one
    of `block-ranges` or `space-reserve`.

    Change-Id: I05d83d73de69c57d885e0c417e8a376f7cfb1e4f
    Closes-Bug: #1924643
    (cherry picked from commit dd0b1076df936b9cb1111567f6b4c3b747744b72)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/821892
Committed: https://opendev.org/openstack/cinder/commit/fffe9b57b984ba402fbe9f0c578b419926491b5b
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit fffe9b57b984ba402fbe9f0c578b419926491b5b
Author: Felipe Rodrigues <email address hidden>
Date: Fri Apr 16 10:05:40 2021 -0300

    NetApp ONTAP: Fix sub-clone zapi call

    The ONTAP documentation states that the `clone-create` ZAPI call
    fails with `block-ranges` and `space-reserve` parameters sent
    together. The sub-clone uses the `block-ranges` and is failing
    because of that restriction.

    This patch fixes the `clone-create` operation by using exactly one
    of `block-ranges` or `space-reserve`.

    Change-Id: I05d83d73de69c57d885e0c417e8a376f7cfb1e4f
    Closes-Bug: #1924643
    (cherry picked from commit dd0b1076df936b9cb1111567f6b4c3b747744b72)
    (cherry picked from commit 344f3e8b098ff52c137ece8a28b3b40a5893e30e)

tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 19.1.0

This issue was fixed in the openstack/cinder 19.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 18.2.0

This issue was fixed in the openstack/cinder 18.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 20.0.0.0rc1

This issue was fixed in the openstack/cinder 20.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/821893
Committed: https://opendev.org/openstack/cinder/commit/9a28bce3eefb25c02316c01995e88352f8ae9528
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 9a28bce3eefb25c02316c01995e88352f8ae9528
Author: Felipe Rodrigues <email address hidden>
Date: Fri Apr 16 10:05:40 2021 -0300

    NetApp ONTAP: Fix sub-clone zapi call

    The ONTAP documentation states that the `clone-create` ZAPI call
    fails with `block-ranges` and `space-reserve` parameters sent
    together. The sub-clone uses the `block-ranges` and is failing
    because of that restriction.

    This patch fixes the `clone-create` operation by using exactly one
    of `block-ranges` or `space-reserve`.

    Change-Id: I05d83d73de69c57d885e0c417e8a376f7cfb1e4f
    Closes-Bug: #1924643
    (cherry picked from commit dd0b1076df936b9cb1111567f6b4c3b747744b72)
    (cherry picked from commit 344f3e8b098ff52c137ece8a28b3b40a5893e30e)
    (cherry picked from commit fffe9b57b984ba402fbe9f0c578b419926491b5b)

tags: added: in-stable-victoria
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 17.4.0

This issue was fixed in the openstack/cinder 17.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/train)

Change abandoned by "Brian Rosmaita <email address hidden>" on branch: stable/train
Review: https://review.opendev.org/c/openstack/cinder/+/821905
Reason: Train is going EOL [0] and the cinder project team has decided to abandon all open patches on stable/train [1].
[0] https://<email address hidden>/thread/UJ7Y76LRWTZQSSDAGMPWCCVVGGIX3TOG/
[1] https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-12-13-14.00.log.html#l-66

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/ussuri)

Change abandoned by "Brian Rosmaita <email address hidden>" on branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/cinder/+/821894
Reason: Ussuri is going EOL [0] and the cinder project team has decided to abandon all open patches on stable/ussuri [1].
[0] https://<email address hidden>/thread/UJ7Y76LRWTZQSSDAGMPWCCVVGGIX3TOG/
[1] https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-12-13-14.00.log.html#l-66

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.