PowerMax - check for SG is not case insensitive

Bug #1929429 reported by Helen Walsh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Helen Walsh

Bug Description

When checking if a storage group is a child of a parent storage group the check is currently case sensitive. We should allow for a pattern match that is not case sensitive. For example, myStorageGroup should equal MYSTORAGEGROUP or mystoragegroup.

Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Greetings Helen,

I think we need more information regarding this issue:
- is this a cinder client / api problem or only PowerMax related?
- do you mind adding some log errors.

Thanks
Sofia

tags: added: powermax sg
Changed in cinder:
status: New → Incomplete
Revision history for this message
Helen Walsh (walshh2) wrote :

Certainly Sofia
1. This is PowerMax driver only
2. The error seen is:

A problem occurred modifying the storage group resource: Storage Group [OS-HOSTA-F-PG1-SG] already contains Storage Group [OS-HOSTA-SRP_1-OptimdNONE-PG1]."

Helen Walsh (walshh2)
Changed in cinder:
assignee: nobody → Helen Walsh (walshh2)
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/+/796256

Changed in cinder:
status: Incomplete → In Progress
Changed in cinder:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit 1f65c2a92fe3931791fca81038a27f81e1ca28fd
Author: Helen Walsh <email address hidden>
Date: Mon Jun 14 11:55:15 2021 +0100

    PowerMax Driver - Allow for case mismatch in SGs

    When checking if a storage group is a child of a parent storage
    group the check is currently case sensitive. We should allow for
    a pattern match that is not case sensitive. For example,
    myStorageGroup should equal MYSTORAGEGROUP or mystoragegroup.

    Closes-Bug: #1929429
    Change-Id: I8dd114fedece8e9d8f85c1ed237c31aede907d67

Changed in cinder:
status: In Progress → Fix Released
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/+/809195

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

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

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

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

commit 80ef8c1046c1c786f2e8f04352f9ce7c8f9af588
Author: Helen Walsh <email address hidden>
Date: Mon Jun 14 11:55:15 2021 +0100

    PowerMax Driver - Allow for case mismatch in SGs

    When checking if a storage group is a child of a parent storage
    group the check is currently case sensitive. We should allow for
    a pattern match that is not case sensitive. For example,
    myStorageGroup should equal MYSTORAGEGROUP or mystoragegroup.

    Closes-Bug: #1929429
    Change-Id: I8dd114fedece8e9d8f85c1ed237c31aede907d67
    (cherry picked from commit 1f65c2a92fe3931791fca81038a27f81e1ca28fd)

tags: added: in-stable-wallaby
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/+/810367

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

Reviewed: https://review.opendev.org/c/openstack/cinder/+/810367
Committed: https://opendev.org/openstack/cinder/commit/5dcfe022440e6d6a61b4258b7b0eae1576e1209c
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 5dcfe022440e6d6a61b4258b7b0eae1576e1209c
Author: Helen Walsh <email address hidden>
Date: Mon Jun 14 11:55:15 2021 +0100

    PowerMax Driver - Allow for case mismatch in SGs

    When checking if a storage group is a child of a parent storage
    group the check is currently case sensitive. We should allow for
    a pattern match that is not case sensitive. For example,
    myStorageGroup should equal MYSTORAGEGROUP or mystoragegroup.

    Closes-Bug: #1929429
    Change-Id: I8dd114fedece8e9d8f85c1ed237c31aede907d67
    (cherry picked from commit 1f65c2a92fe3931791fca81038a27f81e1ca28fd)
    (cherry picked from commit 80ef8c1046c1c786f2e8f04352f9ce7c8f9af588)

tags: added: in-stable-victoria
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/+/821749

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 17.3.0

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

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

Reviewed: https://review.opendev.org/c/openstack/cinder/+/821749
Committed: https://opendev.org/openstack/cinder/commit/54c33cf50f9eab9e9151ea9e64b41478adc7e8aa
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 54c33cf50f9eab9e9151ea9e64b41478adc7e8aa
Author: Helen Walsh <email address hidden>
Date: Mon Jun 14 11:55:15 2021 +0100

    PowerMax Driver - Allow for case mismatch in SGs

    When checking if a storage group is a child of a parent storage
    group the check is currently case sensitive. We should allow for
    a pattern match that is not case sensitive. For example,
    myStorageGroup should equal MYSTORAGEGROUP or mystoragegroup.

    Closes-Bug: #1929429
    Change-Id: I8dd114fedece8e9d8f85c1ed237c31aede907d67
    (cherry picked from commit 1f65c2a92fe3931791fca81038a27f81e1ca28fd)
    (cherry picked from commit 80ef8c1046c1c786f2e8f04352f9ce7c8f9af588)
    (cherry picked from commit 5dcfe022440e6d6a61b4258b7b0eae1576e1209c)

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

This issue was fixed in the openstack/cinder ussuri-eol release.

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.