Missing share type check in manage_existing in share manager

Bug #1493869 reported by Clinton Knight
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Fix Released
Medium
Clinton Knight

Bug Description

In ShareManager::manage_existing(), there is a check for the DHSS mode of the driver but there is no check for the DHSS mode in the specified share type. This omission requires redundant checks in the drivers.

Tags: driver
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (master)

Fix proposed to branch: master
Review: https://review.openstack.org/222257

Changed in manila:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (master)

Reviewed: https://review.openstack.org/222257
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=fd492fc11dc14f39c31f2451cb75e811802cc708
Submitter: Jenkins
Branch: master

commit fd492fc11dc14f39c31f2451cb75e811802cc708
Author: Clinton Knight <email address hidden>
Date: Wed Sep 9 11:27:40 2015 -0400

    Missing check in ShareManager::manage_existing()

    In ShareManager::manage_existing(), there is a check for the DHSS mode of
    the driver but there is no check for the DHSS mode in the specified share
    type. This omission requires redundant checks in the drivers. This patch
    adds the missing check to the share manager, removes the redundant checks
    from the drivers, and updates all relevant unit tests. Also, fix a
    couple minor tempest resource cleanup issues discovered while
    debugging this issue.

    Change-Id: Ib579fd0558e59c28777342bb9d36def12f6bf4da
    Closes-Bug: #1493869

Changed in manila:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in manila:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in manila:
milestone: liberty-rc1 → 1.0.0
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.