OpenStack Shared File Systems Service (Manila)

NetApp manila driver does not honor std specs for migration, manage and unmanage

Bug #1707084 reported by Yogesh
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Fix Released
Low
Daniel Tapia

Bug Description

In case of migration and manage/unmanage features -

Instead of using NetApp based extra_specs format (e.g. 'netapp:thin_provisioned' = 'True') if we use standard extra_specs format (e.g. 'thin_privisioning' = '<is> True')

then, the NetApp driver ignores this extra_specs settings and continue the operation with the default option.

Goutham Pacha Ravi (gouthamr)
Changed in manila:
importance: Undecided → Low
status: New → Confirmed
milestone: none → pike-rc1
tags: added: cdot driver drivers netapp ontap vendor
Goutham Pacha Ravi (gouthamr)
Changed in manila:
assignee: nobody → Goutham Pacha Ravi (gouthamr)
Revision history for this message
Ben Swartzlander (bswartz) wrote :

Missed pike, untargeting

Changed in manila:
milestone: pike-rc1 → none
Goutham Pacha Ravi (gouthamr)
Changed in manila:
assignee: Goutham Pacha Ravi (gouthamr) → nobody
Tom Barron (tpb)
tags: removed: drivers
Naresh Kumar Gunjalli (nareshkumarg)
Changed in manila:
assignee: nobody → Naresh Kumar Gunjalli (nareshkumarg)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (master)

Fix proposed to branch: master
Review: https://review.opendev.org/712512

Changed in manila:
assignee: Naresh Kumar Gunjalli (nareshkumarg) → Daniel Tapia (danielarthurt)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (master)

Reviewed: https://review.opendev.org/712512
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=9250e02b56fa4a0d93fe4c32cdfac4eceab8d095
Submitter: Zuul
Branch: master

commit 9250e02b56fa4a0d93fe4c32cdfac4eceab8d095
Author: Naresh Kumar Gunjalli <email address hidden>
Date: Thu Jun 13 07:42:13 2019 -0400

    [NetApp] Fix driver to honor standard extra specs

    During migration and manage/unmanage operations, if we used standard
    extra_specs format (e.g. 'thin_provisioning'='True') instead of
    NetApp based extra_specs format (e.g. 'netapp:thin_provisioned'='True'),
    NetApp driver used to ignore these standard extra_specs settings
    and continued the operation with the default option.
    The driver is now fixed to handle this issue by honouring the
    standard extra_specs during migration and manage/unmanage.

    Change-Id: I48f4d966201fc8d6bd6eec0c37f1575b434310e7
    Closes-Bug: #1707084

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

Fix proposed to branch: stable/train
Review: https://review.opendev.org/718053

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

Reviewed: https://review.opendev.org/718053
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=678ea79f85e53ec21a0fb74758875b3a7ebe1aa7
Submitter: Zuul
Branch: stable/train

commit 678ea79f85e53ec21a0fb74758875b3a7ebe1aa7
Author: Naresh Kumar Gunjalli <email address hidden>
Date: Thu Jun 13 07:42:13 2019 -0400

    [NetApp] Fix driver to honor standard extra specs

    During migration and manage/unmanage operations, if we used standard
    extra_specs format (e.g. 'thin_provisioning'='True') instead of
    NetApp based extra_specs format (e.g. 'netapp:thin_provisioned'='True'),
    NetApp driver used to ignore these standard extra_specs settings
    and continued the operation with the default option.
    The driver is now fixed to handle this issue by honouring the
    standard extra_specs during migration and manage/unmanage.

    Change-Id: I48f4d966201fc8d6bd6eec0c37f1575b434310e7
    Closes-Bug: #1707084
    (cherry picked from commit 9250e02b56fa4a0d93fe4c32cdfac4eceab8d095)

tags: added: in-stable-train
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/719131

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

Reviewed: https://review.opendev.org/719131
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=644e2cfc564208bd5b9b3e1201f8f9718ca65823
Submitter: Zuul
Branch: stable/stein

commit 644e2cfc564208bd5b9b3e1201f8f9718ca65823
Author: Naresh Kumar Gunjalli <email address hidden>
Date: Thu Jun 13 07:42:13 2019 -0400

    [NetApp] Fix driver to honor standard extra specs

    During migration and manage/unmanage operations, if we used standard
    extra_specs format (e.g. 'thin_provisioning'='True') instead of
    NetApp based extra_specs format (e.g. 'netapp:thin_provisioned'='True'),
    NetApp driver used to ignore these standard extra_specs settings
    and continued the operation with the default option.
    The driver is now fixed to handle this issue by honouring the
    standard extra_specs during migration and manage/unmanage.

    Change-Id: I48f4d966201fc8d6bd6eec0c37f1575b434310e7
    Closes-Bug: #1707084
    (cherry picked from commit 9250e02b56fa4a0d93fe4c32cdfac4eceab8d095)
    (cherry picked from commit 678ea79f85e53ec21a0fb74758875b3a7ebe1aa7)

tags: added: in-stable-stein
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/724346

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

Reviewed: https://review.opendev.org/724346
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=fa0b0ec94e319caebe96c4595c60d2c0b803b950
Submitter: Zuul
Branch: stable/rocky

commit fa0b0ec94e319caebe96c4595c60d2c0b803b950
Author: Naresh Kumar Gunjalli <email address hidden>
Date: Thu Jun 13 07:42:13 2019 -0400

    [NetApp] Fix driver to honor standard extra specs

    During migration and manage/unmanage operations, if we used standard
    extra_specs format (e.g. 'thin_provisioning'='True') instead of
    NetApp based extra_specs format (e.g. 'netapp:thin_provisioned'='True'),
    NetApp driver used to ignore these standard extra_specs settings
    and continued the operation with the default option.
    The driver is now fixed to handle this issue by honouring the
    standard extra_specs during migration and manage/unmanage.

    Change-Id: I48f4d966201fc8d6bd6eec0c37f1575b434310e7
    Closes-Bug: #1707084
    (cherry picked from commit 9250e02b56fa4a0d93fe4c32cdfac4eceab8d095)
    (cherry picked from commit 678ea79f85e53ec21a0fb74758875b3a7ebe1aa7)
    (cherry picked from commit 644e2cfc564208bd5b9b3e1201f8f9718ca65823)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/725110

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

Reviewed: https://review.opendev.org/725110
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=c4f952f5a9996a9ddb3daf50529d1a8142c9a23a
Submitter: Zuul
Branch: stable/queens

commit c4f952f5a9996a9ddb3daf50529d1a8142c9a23a
Author: Naresh Kumar Gunjalli <email address hidden>
Date: Thu Jun 13 07:42:13 2019 -0400

    [NetApp] Fix driver to honor standard extra specs

    During migration and manage/unmanage operations, if we used standard
    extra_specs format (e.g. 'thin_provisioning'='True') instead of
    NetApp based extra_specs format (e.g. 'netapp:thin_provisioned'='True'),
    NetApp driver used to ignore these standard extra_specs settings
    and continued the operation with the default option.
    The driver is now fixed to handle this issue by honouring the
    standard extra_specs during migration and manage/unmanage.

    Change-Id: I48f4d966201fc8d6bd6eec0c37f1575b434310e7
    Closes-Bug: #1707084
    (cherry picked from commit 9250e02b56fa4a0d93fe4c32cdfac4eceab8d095)
    (cherry picked from commit 678ea79f85e53ec21a0fb74758875b3a7ebe1aa7)
    (cherry picked from commit 644e2cfc564208bd5b9b3e1201f8f9718ca65823)
    (cherry picked from commit fa0b0ec94e319caebe96c4595c60d2c0b803b950)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/manila queens-eol

This issue was fixed in the openstack/manila queens-eol release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/manila rocky-eol

This issue was fixed in the openstack/manila rocky-eol release.

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

Other bug subscribers

Remote bug watches

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