The CinderSCStorageProtocol parameter has no effect

Bug #1978749 reported by Takashi Kajinami
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Manojkatari

Bug Description

Description
===========
The CinderSCStorageProtocol parameter defined in deployment/cinder/cinder-backend-dellemc-sc-puppet.yaml is not used and has no effect.
This parmaeter should be used to set the cinder::backend::dellemc_sc::sc_storage_protocol parameter.

Changed in tripleo:
status: New → Confirmed
importance: Undecided → Medium
importance: Medium → High
status: Confirmed → Triaged
milestone: none → zed-1
Revision history for this message
Takashi Kajinami (kajinamit) wrote :

Also the parameter name isinconsistent with the other parameters, prefixed by CinderS"c" instead of CinderS"C".

Manojkatari (mkatari)
Changed in tripleo:
assignee: nobody → Manojkatari (mkatari)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)
Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (master)

Change abandoned by "Ronelle Landy <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847739
Reason: will reset the gate

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847739
Committed: https://opendev.org/openstack/tripleo-heat-templates/commit/95a02d2419fa279838a5bc3b600b235f41615cac
Submitter: "Zuul (22348)"
Branch: master

commit 95a02d2419fa279838a5bc3b600b235f41615cac
Author: katarimanoj <email address hidden>
Date: Tue Jun 28 19:59:09 2022 +0530

    Fix Cinder Dell EMC SC backend template

    The 'CinderSCStorageProtocol' parameter defined in
    cinder-backend-dellemc-sc-puppet.yaml is not used and
    has no effect. Modified it as 'CinderScStorageProtocol'
    similar to other parameters.

    The parameter 'CinderScExcludedDomainIp' is introduced
    when the backend support was added in the patch
    https://review.opendev.org/c/openstack/tripleo-heat-templates/+/727283
    but it is never used.
    Removed it as the 'CinderScExcludedDomainIps' parameter is already used.

    Closes-Bug: #1978749
    Closes-Bug: #1978750
    Change-Id: I6a82a542855e32bd82c67f34138271b060df5de2

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 17.0.0

This issue was fixed in the openstack/tripleo-heat-templates 17.0.0 release.

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

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/891278

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

Reviewed: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/891278
Committed: https://opendev.org/openstack/tripleo-heat-templates/commit/0cf9c98ab9524e4eca517470ca6118085531fa4a
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 0cf9c98ab9524e4eca517470ca6118085531fa4a
Author: katarimanoj <email address hidden>
Date: Tue Jun 28 19:59:09 2022 +0530

    Fix Cinder Dell EMC SC backend template

    The 'CinderSCStorageProtocol' parameter defined in
    cinder-backend-dellemc-sc-puppet.yaml is not used and
    has no effect. Modified it as 'CinderScStorageProtocol'
    similar to other parameters.

    The parameter 'CinderScExcludedDomainIp' is introduced
    when the backend support was added in the patch
    https://review.opendev.org/c/openstack/tripleo-heat-templates/+/727283
    but it is never used.
    Removed it as the 'CinderScExcludedDomainIps' parameter is already used.

    Closes-Bug: #1978749
    Closes-Bug: #1978750
    Change-Id: I6a82a542855e32bd82c67f34138271b060df5de2
    (cherry picked from commit 95a02d2419fa279838a5bc3b600b235f41615cac)

tags: added: in-stable-wallaby
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.