Code references 'in-use' share state but this state is unreachable

Bug #1346371 reported by Alex Meade
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Fix Released
Low
Valeriy Ponomaryov

Bug Description

Manila code references an 'in-use' status for shares but this status is invalid. I believe this is likely leftover from the cinder fork.

For example.
./manila/share/manager.py: 82 if share['status'] in ['available', 'in-use']:

Revision history for this message
Valeriy Ponomaryov (vponomaryov) wrote :

When you give code links, please, use links to code using commits, for example:

https://github.com/stackforge/manila/blob/0b382f186bb7f834c2ac4a5a2730ff2cc03eb97a/manila/share/manager.py#L82

And 'wishlist' status is more suitable here, described issue is harmless.

Changed in manila:
assignee: nobody → Valeriy Ponomaryov (vponomaryov)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (master)

Reviewed: https://review.openstack.org/109644
Committed: https://git.openstack.org/cgit/stackforge/manila/commit/?id=61316b7adb37cbbcfec0be8cbfca3274b6c19b24
Submitter: Jenkins
Branch: master

commit 61316b7adb37cbbcfec0be8cbfca3274b6c19b24
Author: Valeriy Ponomaryov <email address hidden>
Date: Fri Jul 25 12:06:00 2014 -0400

    Remove reference to 'in-use' state in share manager

    State 'in-use' is leftover from Cinder and is not used by shares,
    hence it should be removed.
    Due to absense of tests for this module at all, as part of bug fix
    were added unittests.

    Change-Id: I768d731bac3eb1f420695082e69598e55e1de90b
    Closes-Bug: #1346371
    Partial-Bug: #1348635

Changed in manila:
status: In Progress → Fix Committed
Changed in manila:
status: Fix Committed → Fix Released
Changed in manila:
milestone: none → juno-3
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.